[coreboot-gerrit] New patch to review for coreboot: bd05142 AMD Kabini: change memory layout

WANG Siyuan (wangsiyuanbuaa@gmail.com) gerrit at coreboot.org
Mon Mar 10 09:22:56 CET 2014


WANG Siyuan (wangsiyuanbuaa at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5358

-gerrit

commit bd051426a9e5a3a4766c8509f6cf7c8cbe63efab
Author: WANG Siyuan <wangsiyuanbuaa at gmail.com>
Date:   Mon Mar 10 15:39:57 2014 +0800

    AMD Kabini: change memory layout
    
    According to BKDG, from TOM to 4G is MMIO space. In the current setting,
    TOM is 0xE0000000 and MMCONF_BASE_ADDRESS is 0xA0000000. This is not
    correct.Take 4G memory as an example, the current memory layout is:
      0. 0000000000000000-0000000000000fff: CONFIGURATION TABLES
      1. 0000000000001000-000000000009ffff: RAM
      2. 00000000000c0000-000000009fffffff: RAM
      3. 00000000a0000000-00000000afffffff: RESERVED (PCIe configration space)
      4. 00000000b0000000-00000000bf13efff: RAM
      5. 00000000bf13f000-00000000bfffffff: CONFIGURATION TABLES
      6. 00000000c0000000-00000000dfffffff: RESERVED (UMA memory)
      7. 0000000100000000-000000011fffffff: RAM
    
    In this patch, change TOM to 0xC0000000 and change MMCONF_BASE_ADDRESS
    to 0xC0000000. Take 4G memory as an example, the new memory layout is:
      0. 0000000000000000-0000000000000fff: CONFIGURATION TABLES
      1. 0000000000001000-000000000009ffff: RAM
      2. 00000000000c0000-000000009f12ffff: RAM
      3. 000000009f130000-000000009fffffff: CONFIGURATION TABLES
      4. 00000000a0000000-00000000cfffffff: RESERVED (UMA memory and PCIe configration space)
      5. 0000000100000000-000000013fffffff: RAM
    
    I have tested on OliveHill with 2G, 4G and 8G memory. coreboot+seabios
    can boot to Windows 7 and Ubuntu 13.10. S3 is good in both of them.
    
    Change-Id: Icdb36bca384c34e9c2c6798162b36c07df0149f8
    Signed-off-by: WANG Siyuan <SiYuan.Wang at amd.com>
    Signed-off-by: WANG Siyuan <wangsiyuanbuaa at gmail.com>
---
 src/mainboard/amd/olivehill/agesawrapper.c   | 1 +
 src/mainboard/asrock/imb-a180/agesawrapper.c | 1 +
 src/northbridge/amd/agesa/family16kb/Kconfig | 2 +-
 3 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/mainboard/amd/olivehill/agesawrapper.c b/src/mainboard/amd/olivehill/agesawrapper.c
index a1cf7de..00997fb 100644
--- a/src/mainboard/amd/olivehill/agesawrapper.c
+++ b/src/mainboard/amd/olivehill/agesawrapper.c
@@ -291,6 +291,7 @@ agesawrapper_amdinitpost (
 
 	AmdCreateStruct (&AmdParamStruct);
 	PostParams = (AMD_POST_PARAMS *)AmdParamStruct.NewStructPtr;
+	PostParams->MemConfig.BottomIo = 0xC0;
 	status = AmdInitPost (PostParams);
 	if (status != AGESA_SUCCESS) agesawrapper_amdreadeventlog(PostParams->StdHeader.HeapStatus);
 	AmdReleaseStruct (&AmdParamStruct);
diff --git a/src/mainboard/asrock/imb-a180/agesawrapper.c b/src/mainboard/asrock/imb-a180/agesawrapper.c
index 0f9212f..c64cfe1 100644
--- a/src/mainboard/asrock/imb-a180/agesawrapper.c
+++ b/src/mainboard/asrock/imb-a180/agesawrapper.c
@@ -291,6 +291,7 @@ agesawrapper_amdinitpost (
 
 	AmdCreateStruct (&AmdParamStruct);
 	PostParams = (AMD_POST_PARAMS *)AmdParamStruct.NewStructPtr;
+	PostParams->MemConfig.BottomIo = 0xC0;
 	status = AmdInitPost (PostParams);
 	if (status != AGESA_SUCCESS) agesawrapper_amdreadeventlog(PostParams->StdHeader.HeapStatus);
 	AmdReleaseStruct (&AmdParamStruct);
diff --git a/src/northbridge/amd/agesa/family16kb/Kconfig b/src/northbridge/amd/agesa/family16kb/Kconfig
index 259195b..db186f9 100644
--- a/src/northbridge/amd/agesa/family16kb/Kconfig
+++ b/src/northbridge/amd/agesa/family16kb/Kconfig
@@ -32,7 +32,7 @@ config HW_MEM_HOLE_SIZE_AUTO_INC
 
 config MMCONF_BASE_ADDRESS
 	hex
-	default 0xA0000000
+	default 0xC0000000
 
 config MMCONF_BUS_NUMBER
 	int



More information about the coreboot-gerrit mailing list