[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Implement CHIPSET_LOCKDOWN

[Wonkyu Kim (Code Review)]

Wonkyu Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39710 )

Change subject: soc/intel/tigerlake: Implement CHIPSET_LOCKDOWN
......................................................................

Patch Set 13:

(1 comment)

https://review.coreboot.org/c/coreboot/+/39710/10/src/soc/intel/tigerlake/fs... 
File src/soc/intel/tigerlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/39710/10/src/soc/intel/tigerlake/fs... 
PS10, Line 113: CHIPSET_LOCKDOWN_COREBOOT
> Why does the CHIPSET_LOCKDOWN_COREBOOT case appear to unlock everything and the else case lock down  […]

The condistion CHIPSET_LOCKDOWN_COREBOOT hear means do not lockdown inside FSP and Coreboot will lockdown.  And this is code for providing parameters to FSP.
Actual lockdown implementation is in src/soc/intel/common/pch/lockdown/lockdown.c.