[coreboot] r3456 - trunk/coreboot-v2/src/ram

svn at coreboot.org svn at coreboot.org
Fri Aug 1 14:12:37 CEST 2008


Author: stepan
Date: 2008-08-01 14:12:37 +0200 (Fri, 01 Aug 2008)
New Revision: 3456

Modified:
   trunk/coreboot-v2/src/ram/ramtest.c
Log:
use printk, when possible. (trivial)

Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>



Modified: trunk/coreboot-v2/src/ram/ramtest.c
===================================================================
--- trunk/coreboot-v2/src/ram/ramtest.c	2008-08-01 12:11:00 UTC (rev 3455)
+++ trunk/coreboot-v2/src/ram/ramtest.c	2008-08-01 12:12:37 UTC (rev 3456)
@@ -29,22 +29,34 @@
 	/* 
 	 * Fill.
 	 */
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("DRAM fill: 0x%08x-0x%08x\r\n", start, stop);
+#else
 	print_debug("DRAM fill: ");
 	print_debug_hex32(start);
 	print_debug("-");
 	print_debug_hex32(stop);
 	print_debug("\r\n");
+#endif
 	for(addr = start; addr < stop ; addr += 4) {
 		/* Display address being filled */
-		if (!(addr & 0xffff)) {
+		if (!(addr & 0xfffff)) {
+#if CONFIG_USE_PRINTK_IN_CAR
+			printk_debug("%08x \r", addr);
+#else
 			print_debug_hex32(addr);
 			print_debug(" \r");
+#endif
 		}
 		write_phys(addr, addr);
 	};
 	/* Display final address */
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("%08x\r\nDRAM filled\r\n", addr);
+#else
 	print_debug_hex32(addr);
 	print_debug("\r\nDRAM filled\r\n");
+#endif
 }
 
 static void ram_verify(unsigned long start, unsigned long stop)
@@ -54,40 +66,70 @@
 	/* 
 	 * Verify.
 	 */
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("DRAM verify: 0x%08x-0x%08x\r\n", start, stop);
+#else
 	print_debug("DRAM verify: ");
 	print_debug_hex32(start);
 	print_debug_char('-');
 	print_debug_hex32(stop);
 	print_debug("\r\n");
+#endif
 	for(addr = start; addr < stop ; addr += 4) {
 		unsigned long value;
 		/* Display address being tested */
-		if (!(addr & 0xffff)) {
+		if (!(addr & 0xfffff)) {
+#if CONFIG_USE_PRINTK_IN_CAR
+			printk_debug("%08x \r", addr);
+#else
 			print_debug_hex32(addr);
 			print_debug(" \r");
+#endif
 		}
 		value = read_phys(addr);
 		if (value != addr) {
 			/* Display address with error */
+#if CONFIG_USE_PRINTK_IN_CAR
+			printk_err("Fail: @0x%08x Read value=0x%08x\r\n", addr, value);
+#else
 			print_err("Fail: @0x");
 			print_err_hex32(addr);
 			print_err(" Read value=0x");
 			print_err_hex32(value);
 			print_err("\r\n");
+#endif
 			i++;
 			if(i>256) {
+#if CONFIG_USE_PRINTK_IN_CAR
+				printk_debug("Aborting.\n\r");
+#else
 				print_debug("Aborting.\n\r");
+#endif
 				break;
 			}
 		}
 	}
 	/* Display final address */
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("%08x", addr);
+#else
 	print_debug_hex32(addr);
+#endif
+
 	if (i) {
+#if CONFIG_USE_PRINTK_IN_CAR
+		printk_debug("\r\nDRAM did _NOT_ verify!\r\n");
+#else
 		print_debug("\r\nDRAM did _NOT_ verify!\r\n");
+#endif
+		die("DRAM ERROR");
 	}
 	else {
+#if CONFIG_USE_PRINTK_IN_CAR
+		printk_debug("\r\nDRAM range verified.\r\n");
+#else
 		print_debug("\r\nDRAM range verified.\r\n");
+#endif
 	}
 }
 
@@ -99,13 +141,21 @@
 	 * test than a "Is my DRAM faulty?" test.  Not all bits
 	 * are tested.   -Tyson
 	 */
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("Testing DRAM : %08x - %08x\r\n", start, stop);
+#else
 	print_debug("Testing DRAM : ");
 	print_debug_hex32(start);
 	print_debug("-");	
 	print_debug_hex32(stop);
 	print_debug("\r\n");
+#endif
 	ram_fill(start, stop);
 	ram_verify(start, stop);
+#if CONFIG_USE_PRINTK_IN_CAR
+	printk_debug("Done.\r\n");
+#else
 	print_debug("Done.\r\n");
+#endif
 }
 





More information about the coreboot mailing list