erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

include/option: Add verstage stub for UEFI variable store backend

Browse source 
Verstage cannot use the UEFI variable store because it runs before
the SMMSTORE is initialized/available, and because the required EDK2
headers are x86-specific. Provide inline stub that returns fallback
values to satisfy console_init() dependency.

TEST=build google/dewatt with CFR enabled

Change-Id: Icaa493692006cf3e0bb194ee3fdd9caf2f51cda1
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/90295
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Matt DeVillier 2025-11-29 22:55:29 -06:00
commit bae5262c69
1 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

10
src/include/option.h
View file

@ -8,8 +8,14 @@
void sanitize_cmos(void);
/* The CBFS file option backend cannot be used in SMM due to vboot
* dependencies, which are not added to SMM */
#if CONFIG(OPTION_BACKEND_NONE) || (CONFIG(USE_CBFS_FILE_OPTION_BACKEND) && ENV_SMM)
* dependencies, which are not added to SMM.
* The UEFI variable store option backend cannot be used in verstage because:
* - Verstage runs before SMM is available (needed for SMMSTORE access)
* - EDK2 headers required for UEFI variables are x86-specific and not available
* in ARM verstage (PSP verstage). */
#if CONFIG(OPTION_BACKEND_NONE) || \
(CONFIG(USE_CBFS_FILE_OPTION_BACKEND) && ENV_SMM) || \
(CONFIG(USE_UEFI_VARIABLE_STORE) && ENV_SEPARATE_VERSTAGE)
static inline unsigned int get_uint_option(const char *name, const unsigned int fallback)
{