From 9d0921b348cb6fd9f3415ac6e8fd8a41d57d0b3a Mon Sep 17 00:00:00 2001 From: Richard Spiegel Date: Tue, 19 Dec 2017 10:24:50 -0700 Subject: 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 Reviewed-on: https://review.coreboot.org/22948 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Marshall Dawson --- src/soc/amd/stoneyridge/chip.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) (limited to 'src') 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) -- cgit v1.2.3