From patchwork Tue Nov  2 21:33:22 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Define unused DIMM1 to 0xFF on Geode GX2 boards
Date: Tue, 02 Nov 2010 21:33:22 -0000
From: Nils <njacobs8@hetnet.nl>
X-Patchwork-Id: 2229
Message-Id: <201011022233.22980.njacobs8@hetnet.nl>
To: coreboot@coreboot.org

This patch defines the unused DIMM1 to 0xFF to make it obvious it is a bogus 
value.

Signed-off-by: Nils Jacobs <njacobs8@hetnet.nl>

This was requested by Myles.
This is Abuild and boot tested.

Thanks, Nils.


Index: src/mainboard/wyse/s50/romstage.c
===================================================================
--- src/mainboard/wyse/s50/romstage.c	(revision 6011)
+++ src/mainboard/wyse/s50/romstage.c	(working copy)
@@ -35,7 +35,7 @@
 #include "southbridge/amd/cs5536/cs5536_early_setup.c"
 
 #define DIMM0 0xA0
-#define DIMM1 0xA2
+#define DIMM1 0xFF
 
 static inline int spd_read_byte(unsigned int device, unsigned int address)
 {
Index: src/mainboard/amd/rumba/romstage.c
===================================================================
--- src/mainboard/amd/rumba/romstage.c	(revision 6011)
+++ src/mainboard/amd/rumba/romstage.c	(working copy)
@@ -16,7 +16,7 @@
 #include "southbridge/amd/cs5536/cs5536_early_setup.c"
 
 #define DIMM0 0xA0
-#define DIMM1 0xA2
+#define DIMM1 0xFF
 
 static inline int spd_read_byte(unsigned device, unsigned address)
 {
Index: src/mainboard/lippert/frontrunner/romstage.c
===================================================================
--- src/mainboard/lippert/frontrunner/romstage.c	(revision 6011)
+++ src/mainboard/lippert/frontrunner/romstage.c	(working copy)
@@ -18,7 +18,7 @@
 #include "southbridge/amd/cs5535/cs5535_early_setup.c"
 
 #define DIMM0 0xA0
-#define DIMM1 0xA2
+#define DIMM1 0xFF
 
 static const unsigned char spdbytes[] = {	/* 4x Qimonda HYB25DC512160CF-6 */
 	0xFF, 0xFF,				/* only values used by raminit.c are set */