Fix compilation of VIA CN700 northbridge code with gcc 4.6
authorStefan Reinauer <stefan.reinauer@coreboot.org>
Fri, 14 Oct 2011 00:03:04 +0000 (17:03 -0700)
committerStefan Reinauer <stefan.reinauer@coreboot.org>
Fri, 14 Oct 2011 17:12:10 +0000 (19:12 +0200)
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

index df4010eafd3f63abd6fcd4a734e7245b06ec6547..2c387bb4f6e015121a19b91388d302d99a5f759e 100644 (file)
@@ -179,12 +179,20 @@ static void sdram_set_size(const struct mem_controller *ctrl)
        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);