diff options
author | Yu-Ping Wu <yupingso@chromium.org> | 2022-12-26 17:16:53 +0800 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2022-12-31 00:34:13 +0000 |
commit | 8112c95a068dc064dc4bcf6a1bccc57211767cb9 (patch) | |
tree | a5184c476ba5eb14c2890e246a67beeacc85d7a0 | |
parent | d9b646d96a23d6366b9bdc6dff84ef1c3f6cc06c (diff) |
Enable VBOOT_VBNV_FLASH for SOC_INTEL_BRASWELL
To deprecate VBOOT_VBNV_CMOS [1], replace VBOOT_VBNV_CMOS with
VBOOT_VBNV_FLASH for boards using SOC_INTEL_BRASWELL.
Currently BOOT_DEVICE_SPI_FLASH_NO_EARLY_WRITES is selected for
CPU_INTEL_HASWELL, SOC_INTEL_BRASWELL and others (see [2]). However,
there seems to be no particular reason on those platforms. We've dropped
the config for haswell. Now do the same for SOC_INTEL_BRASWELL, so that
VBOOT_VBNV_FLASH can be enabled.
VBOOT_VBNV_FLASH is enabled for the following boards:
- facebook/fbg1701: A 0x2000 RW_NVRAM region is allocated, with the
FW_MAIN_A(CBFS) size reduced by 0x2000.
- google/cyan, intel/strago: Repurpose RW_UNUSED as RW_NVRAM.
[1] https://issuetracker.google.com/issues/235293589
[2] commit 6c2568f4f58b9a1b209c9af36d7f980fde784f08
("drivers/spi: Add BOOT_DEVICE_SPI_FLASH_NO_EARLY_WRITES config")
BUG=b:235293589
TEST=./util/abuild/abuild -t FACEBOOK_FBG1701 -a (with VBOOT selected)
TEST=./util/abuild/abuild -x -t GOOGLE_CYAN -a
TEST=./util/abuild/abuild -x -t INTEL_STRAGO -a
Change-Id: I46542c2887b254f59245f20b8642b023a7871708
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71544
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-by: Erik van den Bogaert <ebogaert@eltan.com>
-rw-r--r-- | src/mainboard/facebook/fbg1701/Kconfig | 2 | ||||
-rw-r--r-- | src/mainboard/facebook/fbg1701/vboot-rw.fmd | 9 | ||||
-rw-r--r-- | src/mainboard/google/cyan/Kconfig | 2 | ||||
-rw-r--r-- | src/mainboard/google/cyan/chromeos.fmd | 2 | ||||
-rw-r--r-- | src/mainboard/intel/strago/Kconfig | 2 | ||||
-rw-r--r-- | src/mainboard/intel/strago/chromeos.fmd | 2 | ||||
-rw-r--r-- | src/soc/intel/braswell/Kconfig | 1 |
7 files changed, 10 insertions, 10 deletions
diff --git a/src/mainboard/facebook/fbg1701/Kconfig b/src/mainboard/facebook/fbg1701/Kconfig index 15528642e0..1218102406 100644 --- a/src/mainboard/facebook/fbg1701/Kconfig +++ b/src/mainboard/facebook/fbg1701/Kconfig @@ -24,7 +24,7 @@ config USE_VENDORCODE_ELTAN if VBOOT -config VBOOT_VBNV_CMOS +config VBOOT_VBNV_FLASH default y config VBOOT_SLOTS_RW_A diff --git a/src/mainboard/facebook/fbg1701/vboot-rw.fmd b/src/mainboard/facebook/fbg1701/vboot-rw.fmd index 172256f7ac..a00c0c9d92 100644 --- a/src/mainboard/facebook/fbg1701/vboot-rw.fmd +++ b/src/mainboard/facebook/fbg1701/vboot-rw.fmd @@ -1,12 +1,13 @@ FLASH@0xff800000 8M { SI_BIOS@0x200000 0x600000 { - MISC_RW@0x0 0x08000 { - RW_MRC_CACHE@0 0x08000 + MISC_RW@0x0 0x0a000 { + RW_MRC_CACHE@0x0 0x08000 + RW_NVRAM@0x8000 0x2000 } - RW_SECTION_A@0x08000 0x558000 { + RW_SECTION_A@0x0a000 0x556000 { VBLOCK_A@0x0 0x10000 RW_FWID_A@0x10000 0x40 - FW_MAIN_A(CBFS)@0x10040 0x547FC0 + FW_MAIN_A(CBFS)@0x10040 0x545FC0 } WP_RO@0x560000 0x0a0000 { RO_SECTION@0x0000 0xa0000 { diff --git a/src/mainboard/google/cyan/Kconfig b/src/mainboard/google/cyan/Kconfig index ac679c765f..710ecc2c06 100644 --- a/src/mainboard/google/cyan/Kconfig +++ b/src/mainboard/google/cyan/Kconfig @@ -57,7 +57,7 @@ if BOARD_GOOGLE_BASEBOARD_CYAN config VBOOT select EC_GOOGLE_CHROMEEC_SWITCHES select VBOOT_LID_SWITCH - select VBOOT_VBNV_CMOS + select VBOOT_VBNV_FLASH config DISPLAY_SPD_DATA bool "Display Memory Serial Presence Detect Data" diff --git a/src/mainboard/google/cyan/chromeos.fmd b/src/mainboard/google/cyan/chromeos.fmd index 923c8ee18a..bc9bac3344 100644 --- a/src/mainboard/google/cyan/chromeos.fmd +++ b/src/mainboard/google/cyan/chromeos.fmd @@ -21,7 +21,7 @@ FLASH@0xff800000 0x800000 { VBLOCK_DEV@0x2000 0x2000 } RW_VPD(PRESERVE)@0x1f8000 0x2000 - RW_UNUSED@0x1fa000 0x6000 + RW_NVRAM@0x1fa000 0x6000 SMMSTORE(PRESERVE)@0x200000 0x40000 RW_LEGACY(CBFS)@0x240000 0x1c0000 WP_RO@0x400000 0x200000 { diff --git a/src/mainboard/intel/strago/Kconfig b/src/mainboard/intel/strago/Kconfig index 144ba5aaa3..b15984f448 100644 --- a/src/mainboard/intel/strago/Kconfig +++ b/src/mainboard/intel/strago/Kconfig @@ -20,7 +20,7 @@ config BOARD_SPECIFIC_OPTIONS config VBOOT select EC_GOOGLE_CHROMEEC_SWITCHES select VBOOT_LID_SWITCH - select VBOOT_VBNV_CMOS + select VBOOT_VBNV_FLASH config MAINBOARD_DIR default "intel/strago" diff --git a/src/mainboard/intel/strago/chromeos.fmd b/src/mainboard/intel/strago/chromeos.fmd index 923c8ee18a..bc9bac3344 100644 --- a/src/mainboard/intel/strago/chromeos.fmd +++ b/src/mainboard/intel/strago/chromeos.fmd @@ -21,7 +21,7 @@ FLASH@0xff800000 0x800000 { VBLOCK_DEV@0x2000 0x2000 } RW_VPD(PRESERVE)@0x1f8000 0x2000 - RW_UNUSED@0x1fa000 0x6000 + RW_NVRAM@0x1fa000 0x6000 SMMSTORE(PRESERVE)@0x200000 0x40000 RW_LEGACY(CBFS)@0x240000 0x1c0000 WP_RO@0x400000 0x200000 { diff --git a/src/soc/intel/braswell/Kconfig b/src/soc/intel/braswell/Kconfig index c35fa7473d..4d3d4df6ea 100644 --- a/src/soc/intel/braswell/Kconfig +++ b/src/soc/intel/braswell/Kconfig @@ -9,7 +9,6 @@ config CPU_SPECIFIC_OPTIONS def_bool y select ACPI_INTEL_HARDWARE_SLEEP_VALUES select ARCH_X86 - select BOOT_DEVICE_SPI_FLASH_NO_EARLY_WRITES select BOOT_DEVICE_SUPPORTS_WRITES select CACHE_MRC_SETTINGS select SUPPORT_CPU_UCODE_IN_CBFS |