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 <subratabanik@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/81624 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
		| @@ -4,24 +4,11 @@ | |||||||
| #include <commonlib/endian.h> | #include <commonlib/endian.h> | ||||||
| #include <commonlib/fsp.h> | #include <commonlib/fsp.h> | ||||||
| #include <inttypes.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 <commonlib/helpers.h> | ||||||
| #include <stddef.h> | #include <stddef.h> | ||||||
| #include <stdint.h> | #include <stdint.h> | ||||||
| #include <string.h> | #include <string.h> | ||||||
|  | #include <vendorcode/intel/fsp/fsp_header.h> | ||||||
|  |  | ||||||
| #define FSP_DBG_LVL BIOS_NEVER | #define FSP_DBG_LVL BIOS_NEVER | ||||||
| #define MASK_24BITS  0x00FFFFFF | #define MASK_24BITS  0x00FFFFFF | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user