[coreboot-gerrit] Change in coreboot[master]: soc/intel/cannonlake: Allow coreboot to handle SPI lockdown

[Subrata Banik (Code Review)]

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

Change subject: soc/intel/cannonlake: Allow coreboot to handle SPI lockdown
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35299/2/src/soc/intel/cannonlake/fs... 
File src/soc/intel/cannonlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/35299/2/src/soc/intel/cannonlake/fs... 
PS2, Line 421: #
> Is the reason for using the preprocessor, that other SOC have different UPD?

yes you got it right. Other SOCs like CNL, CFL, WHL doesn't have FSP UPD to skip SPI lockdown options and those products are in production now so code changes are not feasible hence added the support in CML and due to compilation issue guarding it.