[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Add Kconfig to enable the CSE FW Update feature

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47523 )

Change subject: soc/intel/common: Add Kconfig to enable the CSE FW Update feature
......................................................................

Patch Set 6: Code-Review+2

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/47523/6/src/soc/intel/common/block/... 
PS6, Line 742: #if CONFIG(SOC_INTEL_CSE_RW_UPDATE)
Woops. Looks like my suggestion to add this config caused this addition of #ifs as well.. Humm.. I would really like to get rid of these #ifs, but let's do that as a follow-up. I don't want to make you go back and forth on this change. (Note for self: It is better to have the code compile tested using runtime checks instead of compile time checks here. Will need revisiting the Kconfig definitions to handle this).

We can take that up as follow-up once the entire series is landed.