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


Sami Mujawar
 

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

Join devel@edk2.groups.io to automatically receive all group messages.