[coreboot] New patch to review for coreboot: 8f85c75 exynos5250: add RAM resource beginning at physical address

David Hendricks (dhendrix@chromium.org) gerrit at coreboot.org
Wed Mar 13 04:30:52 CET 2013


David Hendricks (dhendrix at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2698

-gerrit

commit 8f85c75edaabfa057a73f60cd0d44b3b3a938576
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 is
    set start 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>
---
 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..7a565d2 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) * CONFIG_NR_DRAM_BANKS)
 
 static void domain_read_resources(device_t dev)
 {



More information about the coreboot mailing list