[S] Change in coreboot[main]: commonlib: Simplify FSP header inclusion

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81624?usp=email )

Change subject: commonlib: Simplify FSP header inclusion
......................................................................

commonlib: Simplify FSP header inclusion

Include `fsp_header.h` from vendorcode for dynamic FSP_INFO_HEADER
selection.

BUG=b:242829490
TEST=google/rex0 builds successfully with 64-bit FSP.

Change-Id: If165e0517752f320d898cf82f298aa9f5699ae86
Signed-off-by: Subrata Banik 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/81624
Reviewed-by: Arthur Heymans 
Tested-by: build bot (Jenkins) 
---
M src/commonlib/fsp_relocate.c
1 file changed, 1 insertion(+), 14 deletions(-)

Approvals:
  Arthur Heymans: Looks good to me, approved
  build bot (Jenkins): Verified

diff --git a/src/commonlib/fsp_relocate.c b/src/commonlib/fsp_relocate.c
index 96d31b3..f57e278 100644
--- a/src/commonlib/fsp_relocate.c
+++ b/src/commonlib/fsp_relocate.c
@@ -4,24 +4,11 @@
 #include <commonlib/endian.h>
 #include <commonlib/fsp.h>
 #include <inttypes.h>
-/*
- * Intel's code does not have a handle on changing global packing state.
- * Therefore, one needs to protect against packing policies that are set
- * globally for a compilation unit just by including a header file.
- */
-#pragma pack(push)
-
-/* Default bind FSP 1.1 API to edk2 UEFI 2.4 types. */
-#include <vendorcode/intel/edk2/uefi_2.4/uefi_types.h>
-#include <vendorcode/intel/fsp/fsp1_1/IntelFspPkg/Include/FspInfoHeader.h>
-
-/* Restore original packing policy. */
-#pragma pack(pop)
-
 #include <commonlib/helpers.h>
 #include <stddef.h>
 #include <stdint.h>
 #include <string.h>
+#include <vendorcode/intel/fsp/fsp_header.h>

#define FSP_DBG_LVL BIOS_NEVER
 #define MASK_24BITS  0x00FFFFFF