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

[Paul Menzel (Code Review)]

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

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

Patch Set 10: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/39710/10//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/39710/10//COMMIT_MSG@11 
PS10, Line 11: to
for?

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 116: PchUnlockGpioPads
Unrelated, it is very confusing to have *Unlock* here and not consistently *Lock*.