On Wed, 10 Nov 2021 at 12:01, Sami Mujawar <sami.muja...@arm.com> wrote: > > The first 4 patches in this series fix build breaks in upstream > edk2-platforms code for Socionext, Lemaker and SynQuacerEvalBoard. > > The IORT Rev E.b specification updates the IORT table revision > to 3. 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.
I *really* don't like this. All other ACPI tables are set up in a forward compatible way, where an existing definition in code is guaranteed to comply with later versions of the spec, and can always claim to be the newest revision. This is the reason why xxx_TABLE_REVISION macros exist in the first place. So I suppose that means that the IORT was modified in a non-forward compatible fashion? Does that mean that newer firmware has to stick with the old revision in order to remain able to boot older OS versions? > Therefore, > the last 7 patches in this series are added to reflect this > change for the following platforms: > - FVP > - Morello FVP > - SGI > - AMD/Styx > - Socionext/SynQuacer > - RaspberryPi > - N1SDP > > Note: This patch series is dependent on the edk2 patch series: > "[PATCH v3 0/8] IORT Rev E.b specification updates" > (https://edk2.groups.io/g/devel/message/83545), 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_eb_platforms_v1 > > Sami Mujawar (11): > Platform/Socionext: Fix build break for SynQuacerEvalBoard > Platform/LeMaker: Fix build break for CelloBoard > Platform/Socionext/SynQuacerEvalBoard: Add missing ArmSoftFloatLib > Platform/Socionext/SynQuacerEvalBoard: Disable EbcDxe for AArch32 > builds > 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 > > > 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/LeMaker/CelloBoard/CelloBoard.dsc > | 4 +++- > Platform/RaspberryPi/AcpiTables/Iort.aslc > | 2 +- > Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc > | 11 +++++++++-- > Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.fdf > | 8 +++++++- > Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc > | 2 +- > Silicon/Socionext/SynQuacer/AcpiTables/Iort.aslc > | 2 +- > 10 files changed, 26 insertions(+), 11 deletions(-) > > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#83567): https://edk2.groups.io/g/devel/message/83567 Mute This Topic: https://groups.io/mt/86954380/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
