diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2020-06-08 06:05:03 +0300 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-06-10 18:33:34 +0000 |
commit | d8bf22a8635b300e93d0cda83ee87d1d8ad39b72 (patch) | |
tree | 31d4fd14c555d820a9a310c59686e62d7828b34e | |
parent | 9d1cbf1033165486c43b23fad5dcb90473153168 (diff) |
binaryPI: Replace CONFIG(ARCH_xx) test
Once we support building stages for different architectures,
such CONFIG(ARCH_xx) tests do not evaluate correctly anymore.
Not strictly required for binaryPI boards, but do this for
consistency.
Change-Id: Id0bbbfb6f695c4bb920bc57a1e9362a23884efb3
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42161
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
-rw-r--r-- | src/drivers/amd/agesa/def_callouts.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/drivers/amd/agesa/def_callouts.c b/src/drivers/amd/agesa/def_callouts.c index 3b2d35c1a3..07f717324e 100644 --- a/src/drivers/amd/agesa/def_callouts.c +++ b/src/drivers/amd/agesa/def_callouts.c @@ -12,12 +12,9 @@ #include <northbridge/amd/agesa/BiosCallOuts.h> #include <northbridge/amd/agesa/dimmSpd.h> -#if CONFIG(NORTHBRIDGE_AMD_PI) -#if CONFIG(ARCH_ROMSTAGE_X86_64) || \ - CONFIG(ARCH_RAMSTAGE_X86_64) +#if ENV_X86_64 && CONFIG(NORTHBRIDGE_AMD_PI) #error "FIXME: CALLOUT_ENTRY is UINT32 Data, not UINT Data" #endif -#endif AGESA_STATUS GetBiosCallout (UINT32 Func, UINTN Data, VOID *ConfigPtr) { |