commit 89fcdec9721231a2a6faf96462359bb9a2cdda63
author Stefan Reinauer <stefan.reinauer@coreboot.org> Thu Oct 13 17:03:04 2011 -0700
committer Stefan Reinauer <stefan.reinauer@coreboot.org> Fri Oct 14 19:12:10 2011 +0200
tree 776ef7004367a32b7b1672cfe6f1b6b335e608e2
parent b9d60c9ac8f878ebd5a69a0f87d9a9406b89c606

Fix compilation of VIA CN700 northbridge code with gcc 4.6

Change-Id: Ia52d21c5c467ec08bc7b958ee1a8e37e7d3e025b
Signed-off-by: Stefan Reinauer <reinauer@google.com>
Reviewed-on: http://review.coreboot.org/265
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>

src/northbridge/via/cn700/raminit.c

diff --git a/src/northbridge/via/cn700/raminit.c b/src/northbridge/via/cn700/raminit.c
index df4010e..2c387bb 100644
--- a/src/northbridge/via/cn700/raminit.c
+++ b/src/northbridge/via/cn700/raminit.c
@@ -179,12 +179,20 @@
 	case 0x01:
 		result = 0x10;	/* 1GB */
 		break;
+	default:
+		result = 0;
 	}
 
-	if (result == 0xff)
+	switch (result) {
+	case 0xff:
 		die("DRAM module size too big, not supported by CN700\n");
-	else
+		break;
+	case 0:
+		die("DRAM module has unknown density\n");
+		break;
+	default:
 		printk(BIOS_DEBUG, "Found %iMB of ram\n", result * ranks * 64);
+	}
 
 	pci_write_config8(ctrl->d0f3, 0x40, result);
 	pci_write_config8(ctrl->d0f3, 0x48, 0x00);