[coreboot] Patch set updated for coreboot: d1adeaa AMD Fam15tn: dimmSpd.c: Introduce Kconfig variable `SMBUS0_BASE_ADDRESS`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Wed Feb 20 12:21:11 CET 2013

Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2459


commit d1adeaa338ff55999d09bab19c7292b9c764de3b
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Tue Feb 19 14:46:31 2013 +0100

    AMD Fam15tn: dimmSpd.c: Introduce Kconfig variable `SMBUS0_BASE_ADDRESS`
    For AMD Inagua, the following two commits
        commit 01f7ab93359ae0fee5784d35effbcbe0b596df18
        Author: Kerry Sheh <shekairui at gmail.com>
        Date:   Thu Jan 19 13:18:36 2012 +0800
            Inagua: Synchronize AMD/inagua mainboard.
            Reviewed-on: http://review.coreboot.org/542
        commit d91c9b7e3cb9fdaeb9399a21907996130f3120bb
        Author: efdesign98 <efdesign98 at gmail.com>
        Date:   Thu Sep 15 10:59:55 2011 -0600
            AMD Inagua platform updates
            Reviewed-on: http://review.coreboot.org/136
    replaced the constant, `iobase` is set to, by the define
    `SMBUS0_BASE_ADDRESS` from `OEM.h`.
    Do the something similar for AMD Fam15tn used by AMD Parmer and
    Thatcher. Though SB900 CIMX does not ship this header file.
    Therefore set it in Kconfig.
    Change-Id: I70dbf5b398e8abae1c20dc401f547abfe5b37a69
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
 src/northbridge/amd/agesa/family15tn/Kconfig   | 4 ++++
 src/northbridge/amd/agesa/family15tn/dimmSpd.c | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/northbridge/amd/agesa/family15tn/Kconfig b/src/northbridge/amd/agesa/family15tn/Kconfig
index 5f08593..3b67ada 100644
--- a/src/northbridge/amd/agesa/family15tn/Kconfig
+++ b/src/northbridge/amd/agesa/family15tn/Kconfig
@@ -39,4 +39,8 @@ config MMCONF_BUS_NUMBER
 	default 64
+	hex
+	default 0xB00
diff --git a/src/northbridge/amd/agesa/family15tn/dimmSpd.c b/src/northbridge/amd/agesa/family15tn/dimmSpd.c
index 8b62a39..6552da2 100644
--- a/src/northbridge/amd/agesa/family15tn/dimmSpd.c
+++ b/src/northbridge/amd/agesa/family15tn/dimmSpd.c
@@ -154,7 +154,7 @@ AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PA
 		[info->SocketId] [info->MemChannelId] [info->DimmId];
 	if (spdAddress == 0) return AGESA_ERROR;
-	ioBase = 0xB00;
 	setupFch (ioBase);
 	return readspd (ioBase, spdAddress, (void *) info->Buffer, 128);

More information about the coreboot mailing list