[coreboot-gerrit] Patch set updated for coreboot: amd/pi/stoney: Change to early cbmem init

[Marshall Dawson]

Marshall Dawson ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18433

-gerrit

commit 9f4d143cad4a2fdf27700bb65fe06cf15c1d92c0
Author: Marshall Dawson 
Date:   Sun Jan 29 18:27:02 2017 -0700

amd/pi/stoney: Change to early cbmem init

Remove the flexibility of using early or late cbmem init.  Use early
    100% of the time.

Signed-off-by: Marshall Dawson 
    Reviewed-by: Marc Jones 
    (cherry picked from commit 4c1b4615516dfbd46a768968c2be017cdeb51ee4)

Change-Id: Id1c22a15752a07eebf39530ba8549ef1f74265ec
    Signed-off-by: Marc Jones 
---
 src/cpu/amd/pi/00670F00/romstage.c            |  3 +++
 src/northbridge/amd/pi/00670F00/Kconfig       |  2 +-
 src/northbridge/amd/pi/00670F00/memmap.c      | 13 -------------
 src/northbridge/amd/pi/00670F00/northbridge.c |  9 ---------
 4 files changed, 4 insertions(+), 23 deletions(-)

diff --git a/src/cpu/amd/pi/00670F00/romstage.c b/src/cpu/amd/pi/00670F00/romstage.c
index e305a05..79fb351 100644
--- a/src/cpu/amd/pi/00670F00/romstage.c
+++ b/src/cpu/amd/pi/00670F00/romstage.c
@@ -52,5 +52,8 @@ void romstage_common(void)
    AGESAWRAPPER(amdinitpost);

post_code(0x41);
+	cbmem_initialize_empty();
+
+	post_code(0x42);
    AGESAWRAPPER(amdinitenv);
 }
diff --git a/src/northbridge/amd/pi/00670F00/Kconfig b/src/northbridge/amd/pi/00670F00/Kconfig
index c8ce668..b47a3e6 100644
--- a/src/northbridge/amd/pi/00670F00/Kconfig
+++ b/src/northbridge/amd/pi/00670F00/Kconfig
@@ -14,7 +14,7 @@
 ##
 config NORTHBRIDGE_AMD_PI_00670F00
    bool
-	select LATE_CBMEM_INIT
+	select EARLY_CBMEM_INIT

if NORTHBRIDGE_AMD_PI_00670F00

diff --git a/src/northbridge/amd/pi/00670F00/memmap.c b/src/northbridge/amd/pi/00670F00/memmap.c
index 02bfc13..982ea62 100644
--- a/src/northbridge/amd/pi/00670F00/memmap.c
+++ b/src/northbridge/amd/pi/00670F00/memmap.c
@@ -82,20 +82,7 @@ uint32_t uma_base_auto(void)
    return (uint32_t)topmem - uma_size_auto();
 }

-#if IS_ENABLED(CONFIG_LATE_CBMEM_INIT)
-unsigned long get_top_of_ram(void)
-{
-	AMD_CONFIG_PARAMS             StdHeader;
-	UINT64                        MsrReg;
-
-	LibAmdMsrRead (0xC001001A, &MsrReg, &StdHeader);
-	return (unsigned long)MsrReg;
-}
-#endif
-
-#if !IS_ENABLED(CONFIG_LATE_CBMEM_INIT)
 void *cbmem_top(void)
 {
    return (void *)uma_base_auto();
 }
-#endif
diff --git a/src/northbridge/amd/pi/00670F00/northbridge.c b/src/northbridge/amd/pi/00670F00/northbridge.c
index 69478d6..e1b3862 100644
--- a/src/northbridge/amd/pi/00670F00/northbridge.c
+++ b/src/northbridge/amd/pi/00670F00/northbridge.c
@@ -805,16 +805,7 @@ static void domain_set_resources(device_t dev)
    		ramtop = limitk * 1024;
    }

-#if IS_ENABLED(CONFIG_LATE_CBMEM_INIT)
-#if CONFIG_GFXUMA
-	set_top_of_ram(uma_memory_base);
    uma_resource(dev, 7, uma_memory_base >> 10, uma_memory_size >> 10);
-#else
-	set_top_of_ram(ramtop);
-#endif
-#else /* IS_ENABLED(CONFIG_LATE_CBMEM_INIT) */
-	uma_resource(dev, 7, uma_memory_base >> 10, uma_memory_size >> 10);
-#endif

for (link = dev->link_list; link; link = link->next) {
    	if (link->children) {