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

Marshall Buschman (mbuschman@lucidmachines.com) gerrit at coreboot.org
Sun Jun 19 09:12:14 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/47

-gerrit

commit 8cfdef386203ab8fdba72786e784689b6ea6fadf
Author: Scott Duplichan <scott at 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 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