summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarshall Dawson <marshalldawson3rd@gmail.com>2017-12-13 21:53:09 -0700
committerMartin Roth <martinroth@google.com>2017-12-18 16:51:23 +0000
commitff4da93f4e2a5279622b3669b845139f02adc166 (patch)
treec3416e6da1fd2bfba10bdbafb45cea6399d73ab6
parent8f6cd22087c4a3536b2d0ade97e255df6e910bb2 (diff)
downloadcoreboot-ff4da93f4e2a5279622b3669b845139f02adc166.tar.xz
soc/amd/common: Remove #ifndef/#endif from AGESA wrapper
There isn't a good reason to keep the checks for __PRE_RAM__. The global variables are not used outside of ramstage and the linker removes them cleanly in other stages. BUG=b:70671590 TEST=Build and boot Kahlee Change-Id: I7a35141f212f340c157d57fde8daf93c0c383af8 Signed-off-by: Marshall Dawson <marshalldawson3rd@gmail.com> Reviewed-on: https://review.coreboot.org/22885 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
-rw-r--r--src/soc/amd/common/block/pi/agesawrapper.c6
1 files changed, 0 insertions, 6 deletions
diff --git a/src/soc/amd/common/block/pi/agesawrapper.c b/src/soc/amd/common/block/pi/agesawrapper.c
index b9931d2d9c..467206e3b7 100644
--- a/src/soc/amd/common/block/pi/agesawrapper.c
+++ b/src/soc/amd/common/block/pi/agesawrapper.c
@@ -26,7 +26,6 @@
void __attribute__((weak)) SetMemParams(AMD_POST_PARAMS *PostParams) {}
void __attribute__((weak)) OemPostParams(AMD_POST_PARAMS *PostParams) {}
-#ifndef __PRE_RAM__
/* ACPI table pointers returned by AmdInitLate */
static void *DmiTable;
static void *AcpiPstate;
@@ -38,7 +37,6 @@ static void *AcpiWheaCmc;
static void *AcpiAlib;
static void *AcpiIvrs;
static void *AcpiCrat;
-#endif /* #ifndef __PRE_RAM__ */
static AGESA_STATUS agesawrapper_readeventlog(UINT8 HeapStatus)
{
@@ -244,7 +242,6 @@ AGESA_STATUS agesawrapper_amdinitenv(void)
return status;
}
-#ifndef __PRE_RAM__
VOID* agesawrapper_getlateinitptr (int pick)
{
switch (pick) {
@@ -270,7 +267,6 @@ VOID* agesawrapper_getlateinitptr (int pick)
return NULL;
}
}
-#endif /* #ifndef __PRE_RAM__ */
AGESA_STATUS agesawrapper_amdinitmid(void)
{
@@ -313,7 +309,6 @@ AGESA_STATUS agesawrapper_amdinitmid(void)
return status;
}
-#ifndef __PRE_RAM__
AGESA_STATUS agesawrapper_amdinitlate(void)
{
AGESA_STATUS Status;
@@ -362,7 +357,6 @@ AGESA_STATUS agesawrapper_amdinitlate(void)
/* AmdReleaseStruct (&AmdParamStruct); */
return Status;
}
-#endif /* #ifndef __PRE_RAM__ */
AGESA_STATUS agesawrapper_amdlaterunaptask (
UINT32 Func,