[coreboot-gerrit] Change in ...coreboot[master]: bootmode: display_init_required() to check VBOOT and OPROM_MATTERS

[Julius Werner (Code Review)]

Hello build bot (Jenkins), Furquan Shaikh,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/32112

to look at the new patch set (#2).

Change subject: bootmode: display_init_required() to check VBOOT and OPROM_MATTERS
......................................................................

bootmode: display_init_required() to check VBOOT and OPROM_MATTERS

Skipping display init on normal-mode boot is a vboot feature, not
specific to Chrome OS. Fix the code in display_init_required() to check
CONFIG_VBOOT rather than CONFIG_CHROMEOS now that the two aren't always
the same anymore.

Also add a check to guarantee at compile time that VBOOT_OPROM_MATTERS
is enabled on all platforms that make a check to this function (when
VBOOT is also enabled). The whole display skipping mechanism is based on
the oprom_needed NVRAM flag, and skipping display init without enabling
the option to tell vboot that it needs to pay attention to that flag
would make the whole thing not work right.

Change-Id: I5d6421509bdcdaa61b78015af3fa6241fe75bb7f
Signed-off-by: Julius Werner 
---
M src/lib/bootmode.c
1 file changed, 6 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/12/32112/2