Patch set updated for coreboot: quick_ram_check: Remove reference to RAMBASE (WIP)

[Kyösti Mälkki]

Kyösti Mälkki ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15239

-gerrit

commit 10d94c6303aafcd563fc490237e7aa853d609ef6
Author: Kyösti Mälkki 
Date:   Fri Jun 17 23:31:42 2016 +0300

quick_ram_check: Remove reference to RAMBASE (WIP)

TODO: Test at address cbmem_top() instead?
    TODO: lib/ramtest.c builds for arch/x86-only

Change-Id: Ieb8f11690fb0e9b287d866be56010bb9adefd21d
    Signed-off-by: Kyösti Mälkki 
---
 src/lib/ramtest.c | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/src/lib/ramtest.c b/src/lib/ramtest.c
index a29fa25..6d12060 100644
--- a/src/lib/ramtest.c
+++ b/src/lib/ramtest.c
@@ -181,32 +181,37 @@ int ram_check_noprint_nodie(unsigned long start, unsigned long stop)
    return failures;
 }

-void quick_ram_check(void)
+static void __quick_ram_check(uintptr_t dst)
 {
    int fail = 0;
    u32 backup;
-	backup = read_phys(CONFIG_RAMBASE);
-	write_phys(CONFIG_RAMBASE, 0x55555555);
+	backup = read_phys(dst);
+	write_phys(dst, 0x55555555);
    phys_memory_barrier();
-	if (read_phys(CONFIG_RAMBASE) != 0x55555555)
+	if (read_phys(dst) != 0x55555555)
    	fail=1;
-	write_phys(CONFIG_RAMBASE, 0xaaaaaaaa);
+	write_phys(dst, 0xaaaaaaaa);
    phys_memory_barrier();
-	if (read_phys(CONFIG_RAMBASE) != 0xaaaaaaaa)
+	if (read_phys(dst) != 0xaaaaaaaa)
    	fail=1;
-	write_phys(CONFIG_RAMBASE, 0x00000000);
+	write_phys(dst, 0x00000000);
    phys_memory_barrier();
-	if (read_phys(CONFIG_RAMBASE) != 0x00000000)
+	if (read_phys(dst) != 0x00000000)
    	fail=1;
-	write_phys(CONFIG_RAMBASE, 0xffffffff);
+	write_phys(dst, 0xffffffff);
    phys_memory_barrier();
-	if (read_phys(CONFIG_RAMBASE) != 0xffffffff)
+	if (read_phys(dst) != 0xffffffff)
    	fail=1;

-	write_phys(CONFIG_RAMBASE, backup);
+	write_phys(dst, backup);
    if (fail) {
    	post_code(0xea);
    	die("RAM INIT FAILURE!\n");
    }
    phys_memory_barrier();
 }
+
+void quick_ram_check(void)
+{
+	__quick_ram_check(0x100000);
+}