[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Switch to CSE Lite RW at BS_DEV_INIT_CHIPS entry

[Rizwan Qureshi (Code Review)]

Rizwan Qureshi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42790 )

Change subject: soc/intel/tigerlake: Switch to CSE Lite RW at BS_DEV_INIT_CHIPS entry
......................................................................

Patch Set 5:

(1 comment)
> Patch Set 5: -Code-Review
>
> Ok, I had missed some of the background here. Thank for your analysis Rizwan. A straight-up revert when this is not needed would be great.

Yes, we can just revert when 159884143 is fixed

https://review.coreboot.org/c/coreboot/+/42790/5/src/soc/intel/common/block/... 
File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/42790/5/src/soc/intel/common/block/... 
PS5, Line 364: #if CONFIG(SOC_INTEL_TIGERLAKE)
             : BOOT_STATE_INIT_ENTRY(BS_DEV_INIT_CHIPS, BS_ON_ENTRY, cse_fw_sync, NULL);
             : #else
Although mute, please add a TODO to indicate that this should be removed once the issue is fixed in the CSE firmware i.e when we have a true fix for b:159884143