[coreboot-gerrit] Patch set updated for coreboot: quick_ram_check: Remove reference to RAMBASE

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Tue Nov 8 00:50:15 CET 2016


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15239

-gerrit

commit 27b95af3d201400a5a6122d8653c4bd07120d97d
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Fri Jun 17 23:31:42 2016 +0300

    quick_ram_check: Remove reference to RAMBASE
    
    Change-Id: Ieb8f11690fb0e9b287d866be56010bb9adefd21d
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 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);
+}



More information about the coreboot-gerrit mailing list