diff options
| author | Richard Spiegel <richard.spiegel@silverbackltd.com> | 2017-12-19 10:24:50 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-12-22 16:56:28 +0000 |
| commit | 9d0921b348cb6fd9f3415ac6e8fd8a41d57d0b3a (patch) | |
| tree | ee323536bc5cc1964265cd39cff380b574d85671 /src/soc/amd/stoneyridge | |
| parent | c50296d997ee9d8643f8bf5d1fce1842194c9cc9 (diff) | |
| download | coreboot-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/soc/amd/stoneyridge')
| -rw-r--r-- | src/soc/amd/stoneyridge/chip.c | 8 |
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) |