diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-07-02 13:54:52 +0300 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-07-07 20:24:39 +0000 |
commit | 3c2305f162d24cf07fa6079bd20fd2839e5d672c (patch) | |
tree | 686b479a6b90005b495b5845fb5b23c31720b8cd | |
parent | 57f2188d8610bb7fd57e30b562cdb672691f18da (diff) | |
download | coreboot-3c2305f162d24cf07fa6079bd20fd2839e5d672c.tar.gz coreboot-3c2305f162d24cf07fa6079bd20fd2839e5d672c.tar.bz2 coreboot-3c2305f162d24cf07fa6079bd20fd2839e5d672c.zip |
drivers/amd/agesa: Drop redundant stack allocation
The removed call was there to support case LATE_CBMEM_INIT=y,
HAVE_ACPI_RESUME=y. Same stack space is already allocated
with postcar_frame_init() call.
Change-Id: I03a44bc3252f553b1769d362b2f442d3e6ab73f4
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33956
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Mike Banon <mikebdp2@gmail.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
-rw-r--r-- | src/drivers/amd/agesa/oem_s3.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/drivers/amd/agesa/oem_s3.c b/src/drivers/amd/agesa/oem_s3.c index acd34b8edfc8..9514bcdf5eea 100644 --- a/src/drivers/amd/agesa/oem_s3.c +++ b/src/drivers/amd/agesa/oem_s3.c @@ -18,7 +18,6 @@ #include <string.h> #include <cbmem.h> #include <console/console.h> -#include <program_loading.h> #include <northbridge/amd/agesa/state_machine.h> #include <AGESA.h> #include <northbridge/amd/agesa/agesa_helper.h> @@ -122,8 +121,6 @@ AGESA_STATUS OemS3Save(AMD_S3_PARAMS *dataBlock) u32 MTRRStorageSize = 0; uintptr_t pos, size; - romstage_ram_stack_base(HIGH_ROMSTAGE_STACK_SIZE, ROMSTAGE_STACK_CBMEM); - /* To be consumed in AmdInitResume. */ get_s3nv_data(S3DataTypeNonVolatile, &pos, &size); if (size && dataBlock->NvStorageSize) |