[coreboot-gerrit] New patch to review for coreboot: amd/pi: Increase assumption for maximum CBFS file header size

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jul 16 15:36:33 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10942

-gerrit

commit 31807badef2a5515e0715af33aa4631590b49900
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Thu Jul 16 15:34:26 2015 +0200

    amd/pi: Increase assumption for maximum CBFS file header size
    
    The new attributes increase the header size, breaking this assumption.
    
    Change-Id: Ib23862f27650b39133deafb74a24327b098b6e86
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
---
 src/southbridge/amd/pi/hudson/Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/southbridge/amd/pi/hudson/Makefile.inc b/src/southbridge/amd/pi/hudson/Makefile.inc
index d2fd7dc..bf67505 100644
--- a/src/southbridge/amd/pi/hudson/Makefile.inc
+++ b/src/southbridge/amd/pi/hudson/Makefile.inc
@@ -68,7 +68,7 @@ endif
 ifneq ($(HUDSON_FWM_INSIDE_CBFS), 1)
 $(warning ##### WARNING: CONFIG_HUDSON_FWM_POSITION ($(HUDSON_FWM_POSITION)) is outside of CBFS area ($(CBFS_BASE_ADDRESS) to 4294967295) ##### )
 endif
-CBFS_HEADER_SIZE=0x40
+CBFS_HEADER_SIZE=0x60
 
 #assume the cbfs header is less than $(CBFS_HEADER_SIZE) bytes.
 ROMSIG_SIZE=4096



More information about the coreboot-gerrit mailing list