David Hendricks ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2698
-gerrit commit 89e859b28fb3ce3b4e8ddd31550763bb8962cbf2 Author: David Hendricks <[email protected]> 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 <[email protected]> Signed-off-by: David Hendricks <[email protected]> --- src/cpu/samsung/exynos5250/cpu.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/cpu/samsung/exynos5250/cpu.c b/src/cpu/samsung/exynos5250/cpu.c index 0a49e1e..bcf4d22 100644 --- a/src/cpu/samsung/exynos5250/cpu.c +++ b/src/cpu/samsung/exynos5250/cpu.c @@ -1,13 +1,12 @@ #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_KB (CONFIG_SYS_SDRAM_BASE >> 10) +#define RAM_SIZE_KB (CONFIG_DRAM_SIZE_MB << 10UL) static void domain_read_resources(device_t dev) { - ram_resource(dev, 0, RAM_BASE, RAM_SIZE); + ram_resource(dev, 0, RAM_BASE_KB, RAM_SIZE_KB); } static void domain_set_resources(device_t dev) -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
