David Hendricks ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/2355

-gerrit

commit 6584cf683e71fc12c5a87cff1102ff8c367850a5
Author: David Hendricks <[email protected]>
Date:   Mon Feb 11 11:53:42 2013 -0800

    snow: fix high_tables_base calculation
    
    It was off by a few orders of magnitude. D'oh.
    
    Change-Id: I9c8a3d5bd9ce261f914cfc7d05d86a1c61519b81
    Signed-off-by: David Hendricks <[email protected]>
---
 src/mainboard/google/snow/ramstage.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/google/snow/ramstage.c 
b/src/mainboard/google/snow/ramstage.c
index b779999..a3e9236 100644
--- a/src/mainboard/google/snow/ramstage.c
+++ b/src/mainboard/google/snow/ramstage.c
@@ -33,7 +33,7 @@ void main(void)
        /* place at top of physical memory */
        high_tables_size = CONFIG_COREBOOT_TABLES_SIZE;
        high_tables_base = CONFIG_SYS_SDRAM_BASE +
-                       ((CONFIG_DRAM_SIZE_MB * 1024) * CONFIG_NR_DRAM_BANKS) -
+                       ((CONFIG_DRAM_SIZE_MB << 20UL) * CONFIG_NR_DRAM_BANKS) -
                        CONFIG_COREBOOT_TABLES_SIZE;
 #endif
 

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to