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

login
register
about
Submitter gerrit@coreboot.org
Date 2011-06-19 16:42:34
Message ID <E1QYL50-0004Ud-8R@ra.coresystems.de>
Download mbox | patch
Permalink /patch/3172/
State New, archived
Headers show

Comments

gerrit@coreboot.org - 2011-06-19 16:42:34
Marshall Buschman (mbuschman@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@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@notabs.org>
    Acked-by: Marshall Buschman <mbuschman@lucidmachines.com>
---
 .../amd/agesa_wrapper/family14/northbridge.c       |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Patch

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. */