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

["Ard Biesheuvel" <ardb@kernel.org>]

On Wed, 27 Jul 2022 at 02:21, Sami Mujawar <sami.mujawar@arm.com> wrote:
>
> 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/*
>

For the associated fixes for all the above:

Reviewed-by: Ard Biesheuvel <ardb@kernel.org>

Feel free to push all of these changes when you feel it is appropriate.


> 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(-)
>     >
>
>
>
>
>
>
>
>
> 
>
>