[coreboot-gerrit] Change in coreboot[master]: soc/tigerlake: Update early TCSS code to not call back and forth betw...

[Brandon Breitenstein (Code Review)]

Attention is currently required from: Furquan Shaikh, Tim Wawrzynczak.
Brandon Breitenstein has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47684 )

Change subject: soc/tigerlake: Update early TCSS code to not call back and forth between mainboard and soc
......................................................................

Patch Set 4:

(2 comments)

Patchset:

PS4:
> I see some comments from earlier patchsets are still not addressed. […]

I addressed most of them just hadn't gone back to mark them as such marked them appropriately

File src/soc/intel/tigerlake/early_tcss.c:

https://review.coreboot.org/c/coreboot/+/47684/comment/e27897cb_7e0155a4 
PS2, Line 296: __weak
> This is what I am thinking: […]

looks good to me I'll add the changes related to port map in the next patch but this flow looks good. I will also rename the functions in the next patch because this was doing the only updates prior to the new changes. Is that appropriate? This patch is only meant to remove calling back and forth so I don't want to unnecessarily complicate that