[coreboot-gerrit] Change in coreboot[master]: soc/intel/{skylake,cannonlake}: Co-ordinate lockdown configuration

[Nico Huber (Code Review)]

Attention is currently required from: Benjamin Doron, Angel Pons, Patrick Rudolph.
Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/50048 )

Change subject: soc/intel/{skylake,cannonlake}: Co-ordinate lockdown configuration
......................................................................

Patch Set 5:

(1 comment)

File src/soc/intel/skylake/chip.c:

https://review.coreboot.org/c/coreboot/+/50048/comment/9a8ee7d1_9a703b83 
PS5, Line 332: 	}
The (implicit) else case would change depending on the defaults in
the FSP binary. Is this change intended?

If yes, how to test / verify it?

If we can't, why do we drag get_lockdown_config() around when only
one setting is supported? (IIRC, it was already broken when added.)