[coreboot-gerrit] Change in coreboot[master]: soc/intel/skylake: Always use `CHIPSET_LOCKDOWN_COREBOOT`

[Angel Pons (Code Review)]

Attention is currently required from: Nico Huber, Michael Niewöhner, Patrick Rudolph.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52096 )

Change subject: soc/intel/skylake: Always use `CHIPSET_LOCKDOWN_COREBOOT`
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> Uhm, this looks more like a hack than a real solution to me. […]

I agree this isn't elegant. However, FSP lockdown is tied into soc/intel/common, so it's not like I can drop it in one patch. This change is a stopgap to prevent new boards from accidentally using CHIPSET_LOCKDOWN_FSP by omission.

Another approach would be to invert the enum values for chipset lockdown to make CHIPSET_LOCKDOWN_COREBOOT the default value, but that also requires checking each and every board. I'll see what I can do.