Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/44531 )
Change subject: mb/facebook/watson: increase size of RO_VPD and RW_VPD ......................................................................
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 jonzhang@fb.com Change-Id: I428b7de6462b47492d9526042018395d2f99cb2a Reviewed-on: https://review.coreboot.org/c/coreboot/+/44531 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net --- M src/mainboard/facebook/watson/board.fmd 1 file changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Angel Pons: Looks good to me, approved
diff --git a/src/mainboard/facebook/watson/board.fmd b/src/mainboard/facebook/watson/board.fmd index e2b8f58..6c58d12 100644 --- a/src/mainboard/facebook/watson/board.fmd +++ b/src/mainboard/facebook/watson/board.fmd @@ -9,8 +9,8 @@ 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 @@ } # 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 } }