diff options
| author | Jeremy Compostella <jeremy.compostella@intel.com> | 2024-09-17 10:18:50 -0700 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2024-09-20 23:25:30 +0000 |
| commit | 7e67198e4103c0139e7f2657f7c8b5077d0e0b3a (patch) | |
| tree | 515eab460a551aaece314ca9d66ffb990085139c | |
| parent | dd7b671f460eab0cdba3ab60fee39843e6cbada5 (diff) | |
efi: Set EFIAPI to 32-bit ABI for FSP1_1
Because PLATFORM_USES_FSP2_X86_32 default to false when
PLATFORM_USES_FSP1_1, efi_datatype.h wrongly defines EFI as
__attribute__((__ms_abi__)).
TEST=When some code involved in the build of a platform using
FSP 1.1 such as Google/CYAN includes efi_datatype.h, it does
not hit the following error: '__ms_abi__' calling convention
is not supported for this target
Change-Id: I914f73ff06bfb801fc319b45b23d7ce4cb7a6d5d
Signed-off-by: Jeremy Compostella <jeremy.compostella@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/84402
Reviewed-by: Subrata Banik <subratabanik@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/include/efi/efi_datatype.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/include/efi/efi_datatype.h b/src/include/efi/efi_datatype.h index d4152af754..a5e8ec3014 100644 --- a/src/include/efi/efi_datatype.h +++ b/src/include/efi/efi_datatype.h @@ -13,7 +13,7 @@ * * Fortunately, EDK2 header allows to override EFIAPI. */ -#if CONFIG(PLATFORM_USES_FSP2_X86_32) +#if CONFIG(PLATFORM_USES_FSP1_1) || CONFIG(PLATFORM_USES_FSP2_X86_32) #define EFIAPI __attribute__((regparm(0))) #else #define EFIAPI __attribute__((__ms_abi__)) |