From 409860687baf68ac3f4f27d234f6e08afa6ad309 Mon Sep 17 00:00:00 2001 From: Yu-Ping Wu Date: Fri, 28 Jun 2024 16:26:43 +0800 Subject: security/vboot: Set VBOOT_ALWAYS_ENABLE_DISPLAY if BMP_LOGO If BMP_LOGO is set, currently display_init_required() will always return 1, so that platform code will always initialize display. However, that information isn't passed to vboot, which may result in unnecessary extra reboots, for example when the payload needs to request display init (by vb2api_need_reboot_for_display()). Since there is already a Kconfig option VBOOT_ALWAYS_ENABLE_DISPLAY to tell vboot that "display is available on this boot", enable it by default if BMP_LOGO is set. BUG=b:345085042 TEST=none BRANCH=none Change-Id: I20113ec464aa036d0498dedb50f0e82cb677ae93 Signed-off-by: Yu-Ping Wu Reviewed-on: https://review.coreboot.org/c/coreboot/+/83256 Tested-by: build bot (Jenkins) Reviewed-by: Subrata Banik Reviewed-by: Julius Werner --- src/security/vboot/Kconfig | 1 + 1 file changed, 1 insertion(+) (limited to 'src/security') diff --git a/src/security/vboot/Kconfig b/src/security/vboot/Kconfig index d42dc74c51..e30e8ee372 100644 --- a/src/security/vboot/Kconfig +++ b/src/security/vboot/Kconfig @@ -156,6 +156,7 @@ config VBOOT_MUST_REQUEST_DISPLAY config VBOOT_ALWAYS_ENABLE_DISPLAY bool "Force to always enable display" + default y if BMP_LOGO default n help Set this option to indicate to vboot that display should always be enabled. -- cgit v1.2.3