Re: [edk2-devel] [edk2-platforms][PATCH V1 00/11] Add SMBIOS tables for SGI/RD platforms

["Pranav Madhu" <pranav.madhu@arm.com>]

> -----Original Message-----
> From: Thomas Abraham <thomas.abraham@arm.com>
> Sent: Tuesday, January 26, 2021 8:04 PM
> To: devel@edk2.groups.io; Pranav Madhu <Pranav.Madhu@arm.com>
> Cc: 'Ard Biesheuvel' <ardb+tianocore@kernel.org>; Ard Biesheuvel
> <Ard.Biesheuvel@arm.com>; Leif Lindholm <leif@nuviainc.com>; nd
> <nd@arm.com>
> Subject: RE: [edk2-devel] [edk2-platforms][PATCH V1 00/11] Add SMBIOS
> tables for SGI/RD platforms
>
>
> On Friday, January 15, 2021 11:57 PM, Pranav Madhu wrote:
> > SMBIOS provides basic hardware and firmware configuration information
> > through table-driven data structure. This patch series adds SMBIOS
> > support for Arm's SGI/RD platforms.
> >
> > The first patch in this series defines platform-id values for the
> > RD-N2 platform library header. The second patch add SgiGetProductId
> > API, which is used by the SMBIOS driver to distinguish between the
> > platforms, and install the right table. The third patch in this series
> > adds SMBIOS driver support that allows for installation of multiple SMBIOS
> tables.
> > And subsequent patches in this series add SMBIOS tables, which are
> > mandatory as per Arm serverready SBBR specification.
> >
> > This patch should be kept on top of 'Update ACPI revision' patch to
> > avoid conflict.
> >
> > Pranav Madhu (11):
> >   Platform/ARM/SgiPkg: Define RD-N2 platform id values
> >   Platform/ARM/SgiPkg: Add GetProductId API for SGI/RD Platforms
> >   Platform/ARM/SgiPkg: Add Initial SMBIOS support
> >   Platform/ARM/SgiPkg: Add SMBIOS Type1 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type3 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type4 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type7 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type16 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type17 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type19 Table
> >   Platform/ARM/SgiPkg: Add SMBIOS Type32 Table
>
> For the series:
> Reviewed-by: Thomas Abraham <thomas.abraham@arm.com>
>
Hi Ard, Leif,

Could you please have a look at this patch series and let me know if there are any comments.

Thanks,
Pranav
> [...]
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.