Re: [PATCH v2 0/6] ArmPkg/SMBIOS fixes and improvements

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

Hi Nhi,

I was building this patch series and realised that merging this patch series will break the SbsaQemu platform builds.
Would you be submitting a patch to update SbsaQemu platform to match this series, please?

Also, can you check if this patch breaks any other platform.

Regards,

Sami Mujawar

On 05/09/2022, 12:57, "Ard Biesheuvel" <ardb@kernel.org> wrote:

    On Wed, 24 Aug 2022 at 18:51, Nhi Pham <nhi@os.amperecomputing.com> wrote:
    >
    > This patchset is to update the ArmPkg/SMBIOS for fixes and improvements.
    >
    > Changes since v1:
    >     + Change PartNumber to ProcessorVersion [Sami]
    >
    > Minh Nguyen (5):
    >   ArmPkg/ProcessorSubClassDxe: Get processor version from OemMiscLib
    >   ArmPkg: Correct return value of "SMCCC_ARCH_SOC_ID" Function ID call
    >   ArmPkg/SmbiosMiscDxe: Fix typo of "AssetTagType02"
    >   ArmPkg/SmbiosMiscDxe: Remove redundant updates in SMBIOS Type 2
    >   ArmPkg/SmbiosMiscDxe: Get SMBIOS information from OemMiscLib
    >
    > Nhi Pham (1):
    >   ArmPkg/SmbiosMiscDxe: Support fetching System UUID
    >

    As Sami appears to be happy with this series,

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

    >  ArmPkg/Include/Library/OemMiscLib.h                                              | 37 +++++++++++++++-
    >  ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c                              | 46 ++++++++++++++++++++
    >  ArmPkg/Universal/Smbios/ProcessorSubClassDxe/ProcessorSubClass.c                 |  8 +++-
    >  ArmPkg/Universal/Smbios/ProcessorSubClassDxe/SmbiosProcessorArmCommon.c          |  6 +--
    >  ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c            | 32 +++++++++-----
    >  ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type01/MiscSystemManufacturerFunction.c    |  3 +-
    >  ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type02/MiscBaseBoardManufacturerFunction.c | 13 +-----
    >  7 files changed, 117 insertions(+), 28 deletions(-)
    >
    > --
    > 2.25.1
    >