Hi Sami, I think Silicon/NXP/NxpQoriqLs.dsc.inc also requires the VariableFlashInfoLib library. Otherwise:
Reviewed-by: Pierre Gondois <pierre.gond...@arm.com> On 7/12/22 16:40, Sami Mujawar wrote:
This v3 patch series renames the EFI_ACPI_IO_REMAPPING_TABLE_REV0 macro to EFI_ACPI_IO_REMAPPING_TABLE_REVISION_00 based on the feedback received. It also includes an additional patch that fixes the build break in CelloBoard due to missing VariablePolicyHelperLib dependency. The v2 patch series fixes the build break in Ampere/Jade and Pythium/FT2000-4 due to missing VariableFlashInfoLib dependency and also updates the IORT revision macro. The IORT Rev E.d specification updates the IORT table revision to 5. Following this the IORT table revision macro EFI_ACPI_IO_REMAPPING_TABLE_REVISION which was at Rev 0 has been renamed to EFI_ACPI_IO_REMAPPING_TABLE_REV0. Therefore, this series updates the following platforms to reflect this renaming: - FVP - Morello FVP - SGI - AMD/Styx - Socionext/SynQuacer - RaspberryPi - N1SDP - Ampere/Jade - Pythium/FT2000-4 Note: This patch series is dependent on the edk2 patch series: "[PATCH v5 0/8] IORT Rev E.d specification updates" (https://edk2.groups.io/g/devel/message/91264), which must be merged in edk2 before this patch series can be integrated. The changes can be seen at: https://github.com/samimujawar/edk2-platforms/tree/1527_iort_rev_ed_platforms_v3 Sami Mujawar (12): Platform/ARM: FVP: Update for IORT revision macro renaming Platform/ARM: Morello: Update for IORT revision macro renaming Platform/ARM: SGI: Update for IORT revision macro renaming Silicon/AMD/Styx: Update for IORT revision macro renaming Silicon/Socionext/SynQuacer: Update for IORT revision macro renaming Platform/RaspberryPi: Update for IORT revision macro renaming Platform/ARM: N1SDP: Update for IORT revision macro renaming Silicon/Ampere: Add VariableFlashInfoLib Platform/Ampere: JadePkg: Update for IORT revision macro renaming Silicon/Phytium: Add VariableFlashInfoLib Silicon/Phytium: FT2000-4Pkg: Update for IORT revision macro renaming Platform/LeMaker: Fix missing dependency on VariablePolicyHelperLib Platform/ARM/Morello/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerFvp.c | 2 +- Platform/ARM/N1Sdp/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c | 2 +- Platform/ARM/SgiPkg/AcpiTables/Iort.aslc | 2 +- Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c | 2 +- Platform/Ampere/JadePkg/Drivers/AcpiPlatformDxe/AcpiIort.c | 2 +- Platform/LeMaker/CelloBoard/CelloBoard.dsc | 1 + Platform/RaspberryPi/AcpiTables/Iort.aslc | 2 +- Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc | 2 +- Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc | 1 + Silicon/Phytium/FT2000-4Pkg/Drivers/AcpiTables/Iort.aslc | 2 +- Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc | 1 + Silicon/Socionext/SynQuacer/AcpiTables/Iort.aslc | 2 +- 12 files changed, 12 insertions(+), 9 deletions(-)
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#91312): https://edk2.groups.io/g/devel/message/91312 Mute This Topic: https://groups.io/mt/92334296/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
