[coreboot-gerrit] Change in coreboot[master]: soc/amd/stoneyridge/chip.c: Move setup_bsp_ramtop to soc_init()

[Richard Spiegel (Code Review)]

Richard Spiegel has uploaded this change for review. ( https://review.coreboot.org/22948

Change subject: soc/amd/stoneyridge/chip.c: Move setup_bsp_ramtop to soc_init()
......................................................................

soc/amd/stoneyridge/chip.c: Move setup_bsp_ramtop to soc_init()

Issue first reported at commit 1587dc8a2b, the call of functions
setup_bsp_ramtop() and setup_uma_memory() should be moved from enable_dev()
to soc_init(). The function setup_uma_memory() no longer exists, its
functionality transfered to agesawrapper_amdinitpost.

Move setup_bsp_ramtop() as required.

BUG=b:62240756
TEST=Build and boot kahlee.

Change-Id: I44e6cab17a8f7f364fc57657f41b211ec9d17641
Signed-off-by: Richard Spiegel 
---
M src/soc/amd/stoneyridge/chip.c
1 file changed, 7 insertions(+), 7 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/48/22948/1

diff --git a/src/soc/amd/stoneyridge/chip.c b/src/soc/amd/stoneyridge/chip.c
index 2602be5..87f3e15 100644
--- a/src/soc/amd/stoneyridge/chip.c
+++ b/src/soc/amd/stoneyridge/chip.c
@@ -45,13 +45,6 @@

static void enable_dev(device_t dev)
 {
-	static int done = 0;
-
-	if (!done) {
-		setup_bsp_ramtop();
-		done = 1;
-	}
-
    /* Set the operations if it is a special bus type */
    if (dev->path.type == DEVICE_PATH_DOMAIN)
    	dev->ops = &pci_domain_ops;
@@ -63,7 +56,14 @@

static void soc_init(void *chip_info)
 {
+	static int done = 0;
+
    southbridge_init(chip_info);
+	if (!done) {
+		setup_bsp_ramtop();
+		done = 1;
+	}
+
 }

static void soc_final(void *chip_info)