Kconfig: Set VBOOT_OPROM_MATTERS for relevant non-x86 devices
The VBOOT_OPROM_MATTERS configuration option signals to vboot that the board can skip display initialization in the normal boot path. It's name is a left-over from a time when this could only happen by avoiding loading the VGA option ROM on x86 devices. Now we have other boards that can skip their native display initialization paths too, and the effect to vboot is the same. (Really, we should rename oprom_matters and oprom_loaded to display_skippable and display_initialized or something, but I don't think that's worth the amount of repositories this would need to touch.) The only effect this still has in today's vboot is to reboot and explicitly request display initialization for EC software sync on VBOOT_EC_SLOW_UPDATE devices (which we haven't had yet on ARM). Still, the vboot flag just declares the capability (for skipping display init), and it should be set correctly regardless of whether that actually makes a difference on a given platform (right now). This patch updates all boards/SoCs that have a conditional path based on display_init_required() accordingly. BRANCH=None BUG=chrome-os-partner:51145 TEST=Booted Oak, confirmed that there's no notable boot time impact. Change-Id: Ic7c77dbd8356d67af7aee54e7869f9ac35241b99 Signed-off-by: Martin Roth <martinroth@chromium.org> Original-Commit-Id: 9c242f7 Original-Change-Id: I75e5cdda2ba2d111ea50ed2c7cdf94322679f1cd Original-Signed-off-by: Julius Werner <jwerner@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/348786 Original-Reviewed-by: Vadim Bendebury <vbendeb@chromium.org> Original-Reviewed-by: Duncan Laurie <dlaurie@chromium.org> Reviewed-on: https://review.coreboot.org/15113 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
parent
76655cb82c
commit
0a0f9c5d68
|
@ -36,6 +36,7 @@ config BOARD_SPECIFIC_OPTIONS
|
||||||
config CHROMEOS
|
config CHROMEOS
|
||||||
select CHROMEOS_VBNV_FLASH
|
select CHROMEOS_VBNV_FLASH
|
||||||
select EC_SOFTWARE_SYNC
|
select EC_SOFTWARE_SYNC
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
select VIRTUAL_DEV_SWITCH
|
select VIRTUAL_DEV_SWITCH
|
||||||
|
|
||||||
config MAINBOARD_DIR
|
config MAINBOARD_DIR
|
||||||
|
|
|
@ -16,6 +16,9 @@ config SOC_NVIDIA_TEGRA124
|
||||||
|
|
||||||
if SOC_NVIDIA_TEGRA124
|
if SOC_NVIDIA_TEGRA124
|
||||||
|
|
||||||
|
config CHROMEOS
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
|
|
||||||
config TEGRA124_MODEL_TD570D
|
config TEGRA124_MODEL_TD570D
|
||||||
bool "TD570D"
|
bool "TD570D"
|
||||||
|
|
||||||
|
|
|
@ -17,6 +17,9 @@ config SOC_NVIDIA_TEGRA132
|
||||||
|
|
||||||
if SOC_NVIDIA_TEGRA132
|
if SOC_NVIDIA_TEGRA132
|
||||||
|
|
||||||
|
config CHROMEOS
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
|
|
||||||
config MAINBOARD_DO_DSI_INIT
|
config MAINBOARD_DO_DSI_INIT
|
||||||
bool "Use dsi graphics interface"
|
bool "Use dsi graphics interface"
|
||||||
depends on MAINBOARD_DO_NATIVE_VGA_INIT
|
depends on MAINBOARD_DO_NATIVE_VGA_INIT
|
||||||
|
|
|
@ -19,6 +19,7 @@ if SOC_NVIDIA_TEGRA210
|
||||||
|
|
||||||
config CHROMEOS
|
config CHROMEOS
|
||||||
select CHROMEOS_RAMOOPS_NON_ACPI
|
select CHROMEOS_RAMOOPS_NON_ACPI
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
|
|
||||||
config MAINBOARD_DO_DSI_INIT
|
config MAINBOARD_DO_DSI_INIT
|
||||||
bool "Use dsi graphics interface"
|
bool "Use dsi graphics interface"
|
||||||
|
|
|
@ -32,6 +32,7 @@ config SOC_ROCKCHIP_RK3288
|
||||||
if SOC_ROCKCHIP_RK3288
|
if SOC_ROCKCHIP_RK3288
|
||||||
|
|
||||||
config CHROMEOS
|
config CHROMEOS
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
select VBOOT_STARTS_IN_BOOTBLOCK
|
select VBOOT_STARTS_IN_BOOTBLOCK
|
||||||
select SEPARATE_VERSTAGE
|
select SEPARATE_VERSTAGE
|
||||||
select RETURN_FROM_VERSTAGE
|
select RETURN_FROM_VERSTAGE
|
||||||
|
|
|
@ -19,6 +19,7 @@ if SOC_ROCKCHIP_RK3399
|
||||||
config CHROMEOS
|
config CHROMEOS
|
||||||
select RETURN_FROM_VERSTAGE
|
select RETURN_FROM_VERSTAGE
|
||||||
select SEPARATE_VERSTAGE
|
select SEPARATE_VERSTAGE
|
||||||
|
select VBOOT_OPROM_MATTERS
|
||||||
select VBOOT_STARTS_IN_BOOTBLOCK
|
select VBOOT_STARTS_IN_BOOTBLOCK
|
||||||
|
|
||||||
config PMIC_BUS
|
config PMIC_BUS
|
||||||
|
|
|
@ -111,11 +111,16 @@ config VBOOT_EC_SLOW_UPDATE
|
||||||
screen that informs the user the update is happening.
|
screen that informs the user the update is happening.
|
||||||
|
|
||||||
config VBOOT_OPROM_MATTERS
|
config VBOOT_OPROM_MATTERS
|
||||||
bool "Video option ROM matters"
|
bool "Video option ROM matters (= can skip display init)"
|
||||||
default n
|
default n
|
||||||
depends on VBOOT_VERIFY_FIRMWARE
|
depends on VBOOT_VERIFY_FIRMWARE
|
||||||
help
|
help
|
||||||
Whether the video option ROM has run matters on this platform.
|
Set this option to indicate to vboot that this platform will skip its
|
||||||
|
display initialization on a normal (non-recovery, non-developer) boot.
|
||||||
|
Vboot calls this "oprom matters" because on x86 devices this
|
||||||
|
traditionally meant that the video option ROM will not be loaded, but
|
||||||
|
it works functionally the same for other platforms that can skip their
|
||||||
|
native display initialization code instead.
|
||||||
|
|
||||||
config VIRTUAL_DEV_SWITCH
|
config VIRTUAL_DEV_SWITCH
|
||||||
bool "Virtual developer switch support"
|
bool "Virtual developer switch support"
|
||||||
|
|
Loading…
Reference in New Issue