[coreboot-gerrit] New patch to review for coreboot: cbfs: look for CBFS header in a predefined place

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Wed Jul 8 10:13:44 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/10846

-gerrit

commit ce26a6fcd586a724c54047e1809e8ff3572140e3
Author: Vadim Bendebury <vbendeb at chromium.org>
Date:   Thu Dec 25 15:07:22 2014 -0800

    cbfs: look for CBFS header in a predefined place
    
    This patch adds a facility to set the CBFS header offset. The offset
    value of zero means default. i.e. the CBFS initialization code still
    discovers the offset through the value saved at the top of the ROM.
    
    [pg: stripped to cover only the CBFS_HEADER_OFFSET memory region since
    CBFS diverged too much and will be worked on further in short order.]
    
    Change-Id: I9f2ca1842b479ea4371cfcce02bce7dd28e77ce2
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: e57a3a15bba7cdcca4a5d684ed78f8ac6dbbc95e
    Original-Change-Id: I4c026389ec4fbaa19bd11b2160202282d2f9283c
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/237569
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/include/memlayout.h | 2 ++
 src/include/symbols.h   | 1 +
 2 files changed, 3 insertions(+)

diff --git a/src/include/memlayout.h b/src/include/memlayout.h
index a529628..caae25a 100644
--- a/src/include/memlayout.h
+++ b/src/include/memlayout.h
@@ -105,4 +105,6 @@
 		. += sz;
 #endif
 
+#define CBFS_HEADER_OFFSET(addr) REGION(cbfs_header_offset, addr, 4, 4)
+
 #endif /* __MEMLAYOUT_H */
diff --git a/src/include/symbols.h b/src/include/symbols.h
index 3fbf819..5f59f33 100644
--- a/src/include/symbols.h
+++ b/src/include/symbols.h
@@ -27,6 +27,7 @@ extern u8 _esram[];
 #define _sram_size (_esram - _sram)
 
 extern u8 _dram[];
+extern u32 _cbfs_header_offset[];
 
 extern u8 _timestamp[];
 extern u8 _etimestamp[];



More information about the coreboot-gerrit mailing list