[coreboot] New patch to review: caa9edc Fix memory size reporting on AMD family 14h systems for >= 4GB

Marshall Buschman (mbuschman@lucidmachines.com) gerrit at coreboot.org
Sun Jun 19 18:42:34 CEST 2011


Marshall Buschman (mbuschman at lucidmachines.com) just uploaded a new patch set to gerrit, which you can find at
http://review.coreboot.org/48

-gerrit

commit caa9edc3e49be41dbcd48ae5e594c0bf8dcae34f
Author: Scott Duplichan <scott at notabs.org>
Date:   Sun Jun 19 11:27:21 2011 -0500

    Fix memory size reporting on AMD family 14h systems for >= 4GB
    
    Applying Scott Duplichan's fix for memory >=4GB
    
    Change-Id: I31d66a488a7811d214d84653860b3e0116f67d19
    Signed-off-by: Scott Duplichan <scott at notabs.org>
    Acked-by: Marshall Buschman <mbuschman at lucidmachines.com>
---
 .../amd/agesa_wrapper/family14/northbridge.c       |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/northbridge/amd/agesa_wrapper/family14/northbridge.c b/src/northbridge/amd/agesa_wrapper/family14/northbridge.c
index af7d130..4eb589d 100644
--- a/src/northbridge/amd/agesa_wrapper/family14/northbridge.c
+++ b/src/northbridge/amd/agesa_wrapper/family14/northbridge.c
@@ -652,8 +652,8 @@ printk(BIOS_DEBUG, "adsr - CONFIG_PCI_64BIT_PREF_MEM is true.\n");
     d = get_dram_base_mask(0);
 
     if (d.mask & 1) {
-        basek = ((resource_t)(d.base)) << 8;
-        limitk = (resource_t)((d.mask << 8) | 0xFFFFFF);
+        basek = ((resource_t)((u64)d.base)) << 8;
+        limitk = (resource_t)(((u64)d.mask << 8) | 0xFFFFFF);
 printk(BIOS_DEBUG, "adsr: (before) basek = %llx, limitk = %llx.\n",basek,limitk);
 
         /* Convert these values to multiples of 1K for ease of math. */




More information about the coreboot mailing list