[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel: Do CSE sync in romstage, unless ramstage chooses otherwise

[Subrata Banik (Code Review)]

Attention is currently required from: Tarun Tuli, Kapil Porwal, Sridhar Siricilla.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74805 )

Change subject: soc/intel: Do CSE sync in romstage, unless ramstage chooses otherwise
......................................................................

Patch Set 3:

(1 comment)

File src/soc/intel/cannonlake/Kconfig:

https://review.coreboot.org/c/coreboot/+/74805/comment/2f8d800d_183608c6 
PS2, Line 115: if SOC_INTEL_CSE_LITE_SKU
> > When CSE Lite FW update should happen is board choice, i don't think this should part of SoC Kconfig list. This was discussed earlier with Googlers (Furquan) so we pushed it to board configuration.
>
> Are you referring below change which selects CSE lite from mainboard ? as I mentioned/showed there instances already in SoC Kconfig where SOC_INTEL_CSE_LITE_SKU is used as decision point to select other config. For low maintenance it's better to keep it inside SoC (knowing there are 100+ boards to move the romstage section).
> select SOC_INTEL_CSE_LITE_SKU
>
> let me think if there are any better ways. I believe there are still some way.

check the new patchset.