Marshall Dawson has uploaded this change for review. ( https://review.coreboot.org/22939
Change subject: intel/skylake: Perform lockdown in bootstate after MRC write ......................................................................
intel/skylake: Perform lockdown in bootstate after MRC write
The bootstate where MRC data is written to flash is now variable. Make the lockdown procedure's timing variable too, and follow the write by one state.
Change-Id: Ifa4eaba72defc403908fffe3d19dbd3e16e14622 Signed-off-by: Marshall Dawson marshalldawson3rd@gmail.com --- M src/soc/intel/skylake/lockdown.c 1 file changed, 18 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/39/22939/1
diff --git a/src/soc/intel/skylake/lockdown.c b/src/soc/intel/skylake/lockdown.c index 79f6f70..31a9b34 100644 --- a/src/soc/intel/skylake/lockdown.c +++ b/src/soc/intel/skylake/lockdown.c @@ -109,5 +109,21 @@ pmc_lockdown_config(); }
-BOOT_STATE_INIT_ENTRY(BS_DEV_RESOURCES, BS_ON_EXIT, platform_lockdown_config, - NULL); +#define LOCKDOWN_BOOTSTATE (CONFIG_MRC_WRITE_BOOTSTATE + 1) + +#if LOCKDOWN_BOOTSTATE == 3 +BOOT_STATE_INIT_ENTRY(BS_DEV_RESOURCES, BS_ON_EXIT, + platform_lockdown_config, NULL); +#elif LOCKDOWN_BOOTSTATE == 4 +BOOT_STATE_INIT_ENTRY(BS_DEV_ENABLE, BS_ON_EXIT, + platform_lockdown_config, NULL); +#elif LOCKDOWN_BOOTSTATE == 5 +BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_EXIT, + platform_lockdown_config, NULL); +#elif LOCKDOWN_BOOTSTATE == 6 +BOOT_STATE_INIT_ENTRY(BS_POST_DEVICE, BS_ON_EXIT, + platform_lockdown_config, NULL); +#elif LOCKDOWN_BOOTSTATE == 7 +BOOT_STATE_INIT_ENTRY(BS_OS_RESUME_CHECK, BS_ON_EXIT, + platform_lockdown_config, NULL); +#endif