Paul Menzel ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2459
-gerrit commit d1adeaa338ff55999d09bab19c7292b9c764de3b Author: Paul Menzel <[email protected]> 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 <[email protected]> Date: Thu Jan 19 13:18:36 2012 +0800 Inagua: Synchronize AMD/inagua mainboard. Reviewed-on: http://review.coreboot.org/542 and commit d91c9b7e3cb9fdaeb9399a21907996130f3120bb Author: efdesign98 <[email protected]> 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 <[email protected]> --- 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 int default 64 +config SMBUS0_BASE_ADDRESS + hex + default 0xB00 + endif 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; + ioBase = CONFIG_SMBUS0_BASE_ADDRESS; setupFch (ioBase); return readspd (ioBase, spdAddress, (void *) info->Buffer, 128); } -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
