[coreboot] [commit] r5843 - in trunk/src: mainboard/getac/p470 mainboard/intel/d945gclf mainboard/roda/rk886ex northbridge/intel/i440lx northbridge/via/vt8601

repository service svn at coreboot.org
Sat Sep 25 16:58:29 CEST 2010


Author: uwe
Date: Sat Sep 25 16:58:28 2010
New Revision: 5843
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5843

Log:
Various CONFIG_DEBUG_RAM_SETUP related fixes (trivial).

Some boards still used the old DEBUG_RAM_SETUP (without _CONFIG prefix).

Also, consistently use "#if CONFIG_DEBUG_RAM_SETUP" (not #ifdef) as we do
elsewhere.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

Modified:
   trunk/src/mainboard/getac/p470/romstage.c
   trunk/src/mainboard/intel/d945gclf/romstage.c
   trunk/src/mainboard/roda/rk886ex/romstage.c
   trunk/src/northbridge/intel/i440lx/Kconfig
   trunk/src/northbridge/intel/i440lx/raminit.c
   trunk/src/northbridge/via/vt8601/Kconfig
   trunk/src/northbridge/via/vt8601/raminit.c

Modified: trunk/src/mainboard/getac/p470/romstage.c
==============================================================================
--- trunk/src/mainboard/getac/p470/romstage.c	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/mainboard/getac/p470/romstage.c	Sat Sep 25 16:58:28 2010	(r5843)
@@ -379,7 +379,7 @@
 
 #if CONFIG_HAVE_ACPI_RESUME == 0
 	/* When doing resume, we must not overwrite RAM */
-#if defined(DEBUG_RAM_SETUP)
+#if CONFIG_DEBUG_RAM_SETUP
 	sdram_dump_mchbar_registers();
 
 	{

Modified: trunk/src/mainboard/intel/d945gclf/romstage.c
==============================================================================
--- trunk/src/mainboard/intel/d945gclf/romstage.c	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/mainboard/intel/d945gclf/romstage.c	Sat Sep 25 16:58:28 2010	(r5843)
@@ -295,7 +295,7 @@
 
 #if !CONFIG_HAVE_ACPI_RESUME
 #if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
-#if defined(DEBUG_RAM_SETUP)
+#if CONFIG_DEBUG_RAM_SETUP
 	sdram_dump_mchbar_registers();
 #endif
 

Modified: trunk/src/mainboard/roda/rk886ex/romstage.c
==============================================================================
--- trunk/src/mainboard/roda/rk886ex/romstage.c	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/mainboard/roda/rk886ex/romstage.c	Sat Sep 25 16:58:28 2010	(r5843)
@@ -355,7 +355,7 @@
 
 #if !CONFIG_HAVE_ACPI_RESUME
 #if CONFIG_DEFAULT_CONSOLE_LOGLEVEL > 8
-#if defined(DEBUG_RAM_SETUP)
+#if CONFIG_DEBUG_RAM_SETUP
 	sdram_dump_mchbar_registers();
 
 	{

Modified: trunk/src/northbridge/intel/i440lx/Kconfig
==============================================================================
--- trunk/src/northbridge/intel/i440lx/Kconfig	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/northbridge/intel/i440lx/Kconfig	Sat Sep 25 16:58:28 2010	(r5843)
@@ -19,4 +19,5 @@
 
 config NORTHBRIDGE_INTEL_I440LX
 	bool
+	select HAVE_DEBUG_RAM_SETUP
 

Modified: trunk/src/northbridge/intel/i440lx/raminit.c
==============================================================================
--- trunk/src/northbridge/intel/i440lx/raminit.c	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/northbridge/intel/i440lx/raminit.c	Sat Sep 25 16:58:28 2010	(r5843)
@@ -31,7 +31,7 @@
 /* Uncomment this to enable debugging output. */
 
 /* Debugging macros. */
-#if defined(DEBUG_RAM_SETUP)
+#if CONFIG_DEBUG_RAM_SETUP
 #define PRINT_DEBUG(x)		print_debug(x)
 #define PRINT_DEBUG_HEX8(x)	print_debug_hex8(x)
 #define PRINT_DEBUG_HEX16(x)	print_debug_hex16(x)
@@ -215,7 +215,7 @@
 	reg32 &= 0xe8000000U;
 	pci_write_config32(NB, APBASE, reg32);
 
-	#ifdef DEBUG_RAM_SETUP
+#if CONFIG_DEBUG_RAM_SETUP
 	/*
 	 * apbase dont get set still, no idea what i have doing wrong yet,
 	 * i am almost sure that somehow i set it by mistake once, but can't
@@ -225,7 +225,7 @@
 	PRINT_DEBUG("APBASE ");
 	PRINT_DEBUG_HEX32(reg32);
 	PRINT_DEBUG("\n");
-	#endif
+#endif
 }
 
 
@@ -265,7 +265,7 @@
 		 * i am not sure if that is needed, but was usefull
 		 * for me to confirm what got written
 		 */
-#ifdef DEBUG_RAM_SETUP
+#if CONFIG_DEBUG_RAM_SETUP
 		PRINT_DEBUG("    Set register 0x");
 		PRINT_DEBUG_HEX8(register_values[i]);
 		PRINT_DEBUG(" to 0x");

Modified: trunk/src/northbridge/via/vt8601/Kconfig
==============================================================================
--- trunk/src/northbridge/via/vt8601/Kconfig	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/northbridge/via/vt8601/Kconfig	Sat Sep 25 16:58:28 2010	(r5843)
@@ -1,3 +1,4 @@
 config NORTHBRIDGE_VIA_VT8601
 	bool
+	select HAVE_DEBUG_RAM_SETUP
 

Modified: trunk/src/northbridge/via/vt8601/raminit.c
==============================================================================
--- trunk/src/northbridge/via/vt8601/raminit.c	Sat Sep 25 16:23:31 2010	(r5842)
+++ trunk/src/northbridge/via/vt8601/raminit.c	Sat Sep 25 16:58:28 2010	(r5843)
@@ -69,7 +69,7 @@
 	}
 }
 
-#ifdef CONFIG_DEBUG_RAM_SETUP
+#if CONFIG_DEBUG_RAM_SETUP
 static void dumpnorth(device_t north)
 {
 	unsigned int r, c;




More information about the coreboot mailing list