[coreboot] Patch set updated for coreboot: ac6652c exynos5250: add RAM resource beginning at physical address

Ronald G. Minnich (rminnich@gmail.com) gerrit at coreboot.org
Wed Mar 13 23:16:03 CET 2013


Ronald G. Minnich (rminnich at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2698

-gerrit

commit ac6652c6ac73574e1f946059171743c1a77f88e3
Author: David Hendricks <dhendrix at chromium.org>
Date:   Tue Mar 12 20:16:44 2013 -0700

    exynos5250: add RAM resource beginning at physical address
    
    The original code attempted to reserve a space in RAM for coreboot to
    remain resident. This turns out not to be needed, and breaks things
    for the kernel since the exynos5250-smdk5250 kernel device tree starts
    RAM at 0x40000000.
    
    (This patch was originally by Gabe, I'm just uploading it)
    
    Change-Id: I4536edaf8785d81a3ea008216a2d57549ce5edfb
    Signed-off-by: Gabe Black <gabeblack at chromium.org>
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
---
 src/cpu/samsung/exynos5250/cpu.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/cpu/samsung/exynos5250/cpu.c b/src/cpu/samsung/exynos5250/cpu.c
index 0a49e1e..3b0ae58 100644
--- a/src/cpu/samsung/exynos5250/cpu.c
+++ b/src/cpu/samsung/exynos5250/cpu.c
@@ -1,9 +1,8 @@
 #include <console/console.h>
 #include <device/device.h>
 
-#define RAM_BASE ((CONFIG_SYS_SDRAM_BASE >> 10) + (CONFIG_COREBOOT_ROMSIZE_KB))
-#define RAM_SIZE (((CONFIG_DRAM_SIZE_MB << 10UL) * CONFIG_NR_DRAM_BANKS) \
-		- CONFIG_COREBOOT_ROMSIZE_KB)
+#define RAM_BASE (CONFIG_SYS_SDRAM_BASE >> 10)
+#define RAM_SIZE (CONFIG_DRAM_SIZE_MB << 10UL)
 
 static void domain_read_resources(device_t dev)
 {



More information about the coreboot mailing list