[coreboot] r1014 - in coreboot-v3: include lib
svn at coreboot.org
svn at coreboot.org
Thu Nov 13 18:04:11 CET 2008
Author: mraudsepp
Date: 2008-11-13 18:04:11 +0100 (Thu, 13 Nov 2008)
New Revision: 1014
Modified:
coreboot-v3/include/lib.h
coreboot-v3/lib/ramtest.c
Log:
Return the count of failures from ram_check.
Will be necessary for DBE61 automatic memory size selection.
Signed-off-by: Mart Raudsepp <mart.raudsepp at artecdesign.ee>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Modified: coreboot-v3/include/lib.h
===================================================================
--- coreboot-v3/include/lib.h 2008-11-13 01:53:55 UTC (rev 1013)
+++ coreboot-v3/include/lib.h 2008-11-13 17:04:11 UTC (rev 1014)
@@ -50,6 +50,6 @@
void beep_long(void);
/* Optional ramtest. */
-void ram_check(unsigned long start, unsigned long stop);
+int ram_check(unsigned long start, unsigned long stop);
#endif /* LIB_H */
Modified: coreboot-v3/lib/ramtest.c
===================================================================
--- coreboot-v3/lib/ramtest.c 2008-11-13 01:53:55 UTC (rev 1013)
+++ coreboot-v3/lib/ramtest.c 2008-11-13 17:04:11 UTC (rev 1014)
@@ -84,7 +84,7 @@
* @param start The beginning of the RAM area.
* @param stop The end of the RAM area.
*/
-static void ram_verify(unsigned long start, unsigned long stop)
+static int ram_verify(unsigned long start, unsigned long stop)
{
unsigned long addr, value;
int i = 0;
@@ -112,6 +112,7 @@
/* Print whether or not the verify failed. */
printk(BIOS_DEBUG, "\nDRAM range %sverified.", i ? "_NOT_ " : "");
+ return i;
}
/**
@@ -123,11 +124,14 @@
*
* @param start The beginning of the RAM area.
* @param stop The end of the RAM area.
+ * @return verify failure count
*/
-void ram_check(unsigned long start, unsigned long stop)
+int ram_check(unsigned long start, unsigned long stop)
{
+ int result;
printk(BIOS_DEBUG, "Testing DRAM: %lx-%lx\n", start, stop);
ram_fill(start, stop);
- ram_verify(start, stop);
+ result = ram_verify(start, stop);
printk(BIOS_DEBUG, "Done.\n");
+ return result;
}
More information about the coreboot
mailing list