diff --git a/arch/x86/geodelx/geodelx.c b/arch/x86/geodelx/geodelx.c
index 99a8f59b43..e97b5fb070 100644
--- a/arch/x86/geodelx/geodelx.c
+++ b/arch/x86/geodelx/geodelx.c
@@ -386,6 +386,7 @@ static void set_delay_control(u8 dimm0, u8 dimm1)
 				msr.hi |= delay_control_table[i].fast_hi;
 				msr.lo |= delay_control_table[i].fast_low;
 			}
+			break;
 		}
 	}
 	wrmsr(GLCP_DELAY_CONTROLS, msr);
diff --git a/northbridge/amd/geodelx/raminit.c b/northbridge/amd/geodelx/raminit.c
index 2feda2f908..13161f6278 100644
--- a/northbridge/amd/geodelx/raminit.c
+++ b/northbridge/amd/geodelx/raminit.c
@@ -113,7 +113,7 @@ static void auto_size_dimm(unsigned int dimm, u8 dimm0, u8 dimm1)
 	/* EEPROM byte usage: (5) Number of DIMM Banks */
 	banner(BIOS_DEBUG, "MODBANKS");
 	spd_byte = spd_read_byte(dimm, SPD_NUM_DIMM_BANKS);
-	if ((MIN_MOD_BANKS > spd_byte) && (spd_byte > MAX_MOD_BANKS)) {
+	if ((MIN_MOD_BANKS > spd_byte) || (spd_byte > MAX_MOD_BANKS)) {
 		printk(BIOS_EMERG, "Number of module banks not compatible\n");
 		post_code(ERROR_BANK_SET);
 		hlt();
@@ -124,7 +124,7 @@ static void auto_size_dimm(unsigned int dimm, u8 dimm0, u8 dimm1)
 	/* EEPROM byte usage: (17) Number of Banks on SDRAM Device */
 	banner(BIOS_DEBUG, "FIELDBANKS");
 	spd_byte = spd_read_byte(dimm, SPD_NUM_BANKS_PER_SDRAM);
-	if ((MIN_DEV_BANKS > spd_byte) && (spd_byte > MAX_DEV_BANKS)) {
+	if ((MIN_DEV_BANKS > spd_byte) || (spd_byte > MAX_DEV_BANKS)) {
 		printk(BIOS_EMERG, "Number of device banks not compatible\n");
 		post_code(ERROR_BANK_SET);
 		hlt();