Re: [edk2-devel] [PATCH edk2-platforms v4 00/17] Fix build breaks and update IORT revision macro

["Sami Mujawar" <sami.mujawar@arm.com>]

Hi All,

Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=3458

The edk2 patch series for IORT Rev E.d updates has been submitted to the mailing list and has been reviewed at 
https://edk2.groups.io/g/devel/message/91361

The corresponding edk2-platforms series is posted on the mailing list at https://edk2.groups.io/g/devel/message/91543

The edk2-platforms series fixes a number of build breaks as well as makes the necessary updates to reflect the changes due to IORT Rev E.d updates.

I would prefer this patch series to be merged in the edk2-stable202208 tag. Therefore, requesting review for the edk2-platform patches from the respective maintainers for the following modules:
- Silicon/AMD/Styx
- Silicon/Socionext/SynQuacer
- Platform/LeMaker
- Platform/Socionext
- Silicon/NXP & Platform/NXP/*

Regards,

Sami Mujawar


On 19/07/2022, 17:24, "devel@edk2.groups.io on behalf of Sami Mujawar via groups.io" <devel@edk2.groups.io on behalf of sami.mujawar=arm.com@groups.io> wrote:

    Hi All,

    The IORT Rev E.d seriesfor edk2 has received the r-b from the respective 
    maintainers.

    Can the following package maintainers review this series and provide 
    feedback, please?

    - Silicon/AMD/Styx

    - Silicon/Socionext/SynQuacer

    - Platform/LeMaker

    - Platform/Socionext

    - Silicon/NXP & Platform/NXP/*

    Regards,

    Sami Mujawar

    On 14/07/2022 05:55 pm, Sami Mujawar wrote:
    > This v4 series includes 5 additional patches (13-17) that
    > fix various build breaks caused by changes in edk2 repo.
    > The patches 1-12 have not changed between v3 series, except
    > updates to the r-b received.
    >
    > The 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 v6 0/8] IORT Rev E.d specification updates"
    >    (https://edk2.groups.io/g/devel/message/91361), 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_v4
    >
    > Sami Mujawar (17):
    >    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/Socionext: Fix missing dependency on VariableFlashInfoLib
    >    Silicon/NXP: Fix missing dependency on VariableFlashInfoLib
    >    Platform/NXP/LS1043aRdbPk: Fix build break due to change in
    >      ARM_CORE_INFO
    >    Platform/NXP/LS1046aFrwyPkg: Fix build break due to change in
    >      ARM_CORE_INFO
    >    Platform/NXP/LX2160aRdbPkg: Fix build break due to change in
    >      ARM_CORE_INFO
    >
    >   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/NXP/LS1043aRdbPkg/Library/ArmPlatformLib/ArmPlatformLib.c                           | 2 +-
    >   Platform/NXP/LS1046aFrwyPkg/Library/ArmPlatformLib/ArmPlatformLib.c                          | 2 +-
    >   Platform/NXP/LX2160aRdbPkg/Library/ArmPlatformLib/ArmPlatformLib.c                           | 2 +-
    >   Platform/RaspberryPi/AcpiTables/Iort.aslc                                                    | 2 +-
    >   Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc                                         | 1 +
    >   Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/Iort.aslc                                           | 2 +-
    >   Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                                         | 1 +
    >   Silicon/NXP/NxpQoriqLs.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 +-
    >   17 files changed, 17 insertions(+), 12 deletions(-)
    >