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 /src/mainboard/facebook/fbg1701 | |
| parent | d9b646d96a23d6366b9bdc6dff84ef1c3f6cc06c (diff) | |
| download | coreboot-8112c95a068dc064dc4bcf6a1bccc57211767cb9.tar.gz coreboot-8112c95a068dc064dc4bcf6a1bccc57211767cb9.tar.bz2 coreboot-8112c95a068dc064dc4bcf6a1bccc57211767cb9.zip | |
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>
Diffstat (limited to 'src/mainboard/facebook/fbg1701')
| -rw-r--r-- | src/mainboard/facebook/fbg1701/Kconfig | 2 | ||||
| -rw-r--r-- | src/mainboard/facebook/fbg1701/vboot-rw.fmd | 9 |
2 files changed, 6 insertions, 5 deletions
diff --git a/src/mainboard/facebook/fbg1701/Kconfig b/src/mainboard/facebook/fbg1701/Kconfig index 15528642e0d8..121810240670 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 172256f7ac7d..a00c0c9d92db 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 { |