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

login
register
about
Submitter gerrit@coreboot.org
Date 2011-06-19 07:12:14
Message ID <E1QYCB4-0007ka-NQ@ra.coresystems.de>
Download mbox | patch
Permalink /patch/3170/
State New, archived
Headers show

Comments

gerrit@coreboot.org - 2011-06-19 07:12:14
Marshall Buschman (mbuschman@lucidmachines.com) just uploaded a new patch set to gerrit, which you can find at
http://review.coreboot.org/47

-gerrit

commit 8cfdef386203ab8fdba72786e784689b6ea6fadf
Author: Scott Duplichan <scott@notabs.org>
Date:   Sun Jun 19 02:02:04 2011 -0500

    Fix memory size reporting on AMD family 14h systems for >= 4GB
    
    Applying Scott Duplichan's fix for memory >=4GB
    
    Change-Id: Id687e6412a6bd600b07cef23220b0b6ecfcb1870
    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. */