Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39221 )
Change subject: vboot: Clean up pre-RAM use of vboot_recovery_mode_enabled() ......................................................................
Patch Set 3:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39221/3/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/raminit_mrc.c:
https://review.coreboot.org/c/coreboot/+/39221/3/src/northbridge/intel/sandy... PS3, Line 142: SANDYBRIDGE_VBOOT_IN_BOOTBLOCK nit: for consistency over platforms, offering also a VBOOT_STARTS_IN_ROMSTAGE option for legacy reasons, just using VBOOT_STARTS_IN_BOOTBLOCK plainly seems better?