Change in coreboot[master]: FSP 2.0: Add fsp_relax_security

[Michael Niewöhner (Code Review)]

Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/25437 )

Change subject: FSP 2.0: Add fsp_relax_security
......................................................................

Patch Set 20:

(1 comment)

https://review.coreboot.org/c/coreboot/+/25437/18/src/drivers/intel/fsp2_0/n... 
File src/drivers/intel/fsp2_0/notify.c:

https://review.coreboot.org/c/coreboot/+/25437/18/src/drivers/intel/fsp2_0/n... 
PS18, Line 70: __attribute__((weak)) bool fsp_relax_security(void)
> Well the purpose was to set it at runtime, not compile time ;)

Huh? Tbh I'm not sure how that would be used, anyway. However, I wonder if we want to always add debug/testing functionality, even in production images. "Relax security" doesn't sound like something that should be always there.

When using CHIPSET_LOCKDOWN_COREBOOT, wouldn't that be the same (setting it at build time)?