summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorRichard Spiegel <richard.spiegel@silverbackltd.com>2017-12-19 10:24:50 -0700
committerMartin Roth <martinroth@google.com>2017-12-22 16:56:28 +0000
commit9d0921b348cb6fd9f3415ac6e8fd8a41d57d0b3a (patch)
treeee323536bc5cc1964265cd39cff380b574d85671 /src
parentc50296d997ee9d8643f8bf5d1fce1842194c9cc9 (diff)
downloadcoreboot-9d0921b348cb6fd9f3415ac6e8fd8a41d57d0b3a.tar.xz
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 <richard.spiegel@silverbackltd.com> Reviewed-on: https://review.coreboot.org/22948 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Diffstat (limited to 'src')
-rw-r--r--src/soc/amd/stoneyridge/chip.c8
1 files changed, 1 insertions, 7 deletions
diff --git a/src/soc/amd/stoneyridge/chip.c b/src/soc/amd/stoneyridge/chip.c
index 2602be5087..3d82c37099 100644
--- a/src/soc/amd/stoneyridge/chip.c
+++ b/src/soc/amd/stoneyridge/chip.c
@@ -45,13 +45,6 @@ struct device_operations pci_domain_ops = {
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;
@@ -64,6 +57,7 @@ static void enable_dev(device_t dev)
static void soc_init(void *chip_info)
{
southbridge_init(chip_info);
+ setup_bsp_ramtop();
}
static void soc_final(void *chip_info)