[coreboot] [commit] r6218 - in trunk/src/northbridge/intel: i440bx i82810

repository service svn at coreboot.org
Mon Dec 27 15:31:05 CET 2010


Author: stepan
Date: Mon Dec 27 15:31:05 2010
New Revision: 6218
URL: https://tracker.coreboot.org/trac/coreboot/changeset/6218

Log:
proper printk handling in src/northbridge/intel/i82810/raminit.c
and drop some romcc relics in 440bx code too

Signed-off-by: Stefan Reinauer <stepan at coreboot.org>
Acked-by: Stefan Reinauer <stepan at coreboot.org>

Modified:
   trunk/src/northbridge/intel/i440bx/raminit.c
   trunk/src/northbridge/intel/i82810/raminit.c

Modified: trunk/src/northbridge/intel/i440bx/raminit.c
==============================================================================
--- trunk/src/northbridge/intel/i440bx/raminit.c	Mon Dec 27 14:30:39 2010	(r6217)
+++ trunk/src/northbridge/intel/i440bx/raminit.c	Mon Dec 27 15:31:05 2010	(r6218)
@@ -39,15 +39,9 @@
 /* Debugging macros. */
 #if CONFIG_DEBUG_RAM_SETUP
 #define PRINT_DEBUG(x...)	printk(BIOS_DEBUG, x)
-#define PRINT_DEBUG_HEX8(x)	printk(BIOS_DEBUG, "%02x", x)
-#define PRINT_DEBUG_HEX16(x)	printk(BIOS_DEBUG, "%04x", x)
-#define PRINT_DEBUG_HEX32(x)	printk(BIOS_DEBUG, "%08x", x)
 #define DUMPNORTH()		dump_pci_device(NB)
 #else
 #define PRINT_DEBUG(x...)
-#define PRINT_DEBUG_HEX8(x)
-#define PRINT_DEBUG_HEX16(x)
-#define PRINT_DEBUG_HEX32(x)
 #define DUMPNORTH()
 #endif
 
@@ -434,11 +428,8 @@
 		addr = (dimm_start * 8 * 1024 * 1024) + addr_offset;
 		if (dimm_end > dimm_start) {
 #if 0
-			PRINT_DEBUG("    Sending RAM command 0x");
-			PRINT_DEBUG_HEX16(reg16);
-			PRINT_DEBUG(" to 0x");
-			PRINT_DEBUG_HEX32(addr);
-			PRINT_DEBUG("\n");
+			PRINT_DEBUG("    Sending RAM command 0x%04x to 0x%08x\n",
+					reg16, addr);
 #endif
 
 			read32(addr);
@@ -648,11 +639,8 @@
 		reg |= register_values[i + 2] & ~(register_values[i + 1]);
 		pci_write_config8(NB, register_values[i], reg);
 #if 0
-		PRINT_DEBUG("    Set register 0x");
-		PRINT_DEBUG_HEX8(register_values[i]);
-		PRINT_DEBUG(" to 0x");
-		PRINT_DEBUG_HEX8(reg);
-		PRINT_DEBUG("\n");
+		PRINT_DEBUG("    Set register 0x%02x to 0x%02x\n",
+				register_values[i], reg);
 #endif
 	}
 }
@@ -885,9 +873,7 @@
 			drb |= (drb + (sz.side2 / 8)) << 8;
 		} else {
 #if 0
-			PRINT_DEBUG("No DIMM found in slot ");
-			PRINT_DEBUG_HEX8(i);
-			PRINT_DEBUG("\n");
+			PRINT_DEBUG("No DIMM found in slot %d\n", i);
 #endif
 
 			/* If there's no DIMM in the slot, set dra to 0x00. */
@@ -900,9 +886,7 @@
 
 		pci_write_config16(NB, DRB + (2 * i), drb);
 #if 0
-		PRINT_DEBUG("DRB has been set to 0x");
-		PRINT_DEBUG_HEX16(drb);
-		PRINT_DEBUG("\n");
+		PRINT_DEBUG("DRB has been set to 0x%04x\n", drb);
 #endif
 
 		/* Brings the upper DRB back down to be base for

Modified: trunk/src/northbridge/intel/i82810/raminit.c
==============================================================================
--- trunk/src/northbridge/intel/i82810/raminit.c	Mon Dec 27 14:30:39 2010	(r6217)
+++ trunk/src/northbridge/intel/i82810/raminit.c	Mon Dec 27 15:31:05 2010	(r6218)
@@ -37,15 +37,9 @@
 /* Debugging macros. */
 #if CONFIG_DEBUG_RAM_SETUP
 #define PRINT_DEBUG(x...)	printk(BIOS_DEBUG, x)
-#define PRINT_DEBUG_HEX8(x)	printk(BIOS_DEBUG, "%02x", x)
-#define PRINT_DEBUG_HEX16(x)	printk(BIOS_DEBUG, "%04x", x)
-#define PRINT_DEBUG_HEX32(x)	printk(BIOS_DEBUG, "%08x", x)
 #define DUMPNORTH()		dump_pci_device(PCI_DEV(0, 0, 0))
 #else
-#define PRINT_DEBUG(x)
-#define PRINT_DEBUG_HEX8(x)
-#define PRINT_DEBUG_HEX16(x)
-#define PRINT_DEBUG_HEX32(x)
+#define PRINT_DEBUG(x...)
 #define DUMPNORTH()
 #endif
 




More information about the coreboot mailing list