From 6a7531431d1f9a8225529fc7b71ab57e0da984c3 Mon Sep 17 00:00:00 2001 From: Jonathan Zhang Date: Mon, 17 Aug 2020 15:22:54 -0700 Subject: [PATCH] mb/facebook/watson: increase size of RO_VPD and RW_VPD The current size of RO_VPD (and RW_VPD) is too small. We have case that adding VPD parameters silently corrupts the coreboot region next to RO_VPD. Increase the size of both RO_VPD and RW_VPD to 0x4000 bytes. TESTED=build coreboot image for watson, add large size VPD parameter to the image, boot watson server into target OS. Signed-off-by: Jonathan Zhang Change-Id: I428b7de6462b47492d9526042018395d2f99cb2a Reviewed-on: https://review.coreboot.org/c/coreboot/+/44531 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons Reviewed-by: Paul Menzel --- src/mainboard/facebook/watson/board.fmd | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mainboard/facebook/watson/board.fmd b/src/mainboard/facebook/watson/board.fmd index e2b8f58d06..6c58d12ebd 100644 --- a/src/mainboard/facebook/watson/board.fmd +++ b/src/mainboard/facebook/watson/board.fmd @@ -9,8 +9,8 @@ FLASH@0xff000000 0x1000000 { FMAP@0x0 0x1000 RW_MISC@0x1000 0xe000 { RW_ELOG@0x0 0x4000 - RW_VPD@0x4000 0x2000 - RW_MISC_UNUSED@0x6000 0x5000 + RW_VPD@0x4000 0x4000 + RW_MISC_UNUSED@0x8000 0x4000 RW_NVRAM@0xc000 0x2000 } UNIFIED_MRC_CACHE@0x10000 0x20000 { @@ -19,7 +19,7 @@ FLASH@0xff000000 0x1000000 { } # This only exists to satisfy tools that specifically # look for RO_VPD. - RO_VPD@0x30000 0x1000 - COREBOOT(CBFS)@0x31000 0x9cf000 + RO_VPD@0x30000 0x4000 + COREBOOT(CBFS)@0x34000 } }