diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-04-11 15:40:02 +0300 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-04-15 11:16:10 +0200 |
| commit | 59b23a2fae6502da0172ae69a3734902c03e5faa (patch) | |
| tree | 09729abe658da7d928a0a32476a50fba1b2fadf9 | |
| parent | 472d5111adc037b8ffeeac9ae9362eaa8ec89e45 (diff) | |
| download | coreboot-59b23a2fae6502da0172ae69a3734902c03e5faa.tar.xz | |
AGESA: Unify heap location
HEAP management is identical enough to move heap away from
first 1MiB for all platforms.
Change-Id: I4128fc084fe072fef6194d260c05592582b7b0d0
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/19267
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
| -rw-r--r-- | src/cpu/amd/agesa/heapmanager.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/src/cpu/amd/agesa/heapmanager.c b/src/cpu/amd/agesa/heapmanager.c index 827c262e9c..87864743bf 100644 --- a/src/cpu/amd/agesa/heapmanager.c +++ b/src/cpu/amd/agesa/heapmanager.c @@ -23,22 +23,10 @@ #include <arch/acpi.h> #include <string.h> -#if IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY15_TN) || \ - IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY15_RL) || \ - IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY16_KB) - /* BIOS_HEAP_START_ADDRESS is only for cold boots. */ #define BIOS_HEAP_SIZE 0x30000 #define BIOS_HEAP_START_ADDRESS 0x010000000 -#else - -/* BIOS_HEAP_START_ADDRESS is only for cold boots. */ -#define BIOS_HEAP_SIZE 0x20000 -#define BIOS_HEAP_START_ADDRESS (BSP_STACK_BASE_ADDR - BIOS_HEAP_SIZE) - -#endif - #if IS_ENABLED(CONFIG_HAVE_ACPI_RESUME) && (HIGH_MEMORY_SCRATCH < BIOS_HEAP_SIZE) #error Increase HIGH_MEMORY_SCRATCH allocation #endif |