For the patch set: Reviewed-by: Zhichao Gao <[email protected]> Thanks, Zhichao
> -----Original Message----- > From: Krzysztof Koch <[email protected]> > Sent: Tuesday, April 14, 2020 4:50 PM > To: [email protected]; Ni, Ray <[email protected]>; Gao, Zhichao > <[email protected]> > Cc: Sami Mujawar <[email protected]>; Matteo Carlini > <[email protected]>; Laura Moretta <[email protected]>; nd > <[email protected]> > Subject: RE: [edk2-devel] [PATCH v1 0/3] Check if required ACPI tables are > installed > > Hi, > > Can I get a review for this patch series? > > Thanks, > > Krzysztof > > -----Original Message----- > From: [email protected] <[email protected]> On Behalf Of > Krzysztof Koch via Groups.Io > Sent: Wednesday, March 25, 2020 9:39 > To: [email protected] > Cc: [email protected]; [email protected]; Sami Mujawar > <[email protected]>; Matteo Carlini <[email protected]>; > Laura Moretta <[email protected]>; nd <[email protected]> > Subject: [edk2-devel] [PATCH v1 0/3] Check if required ACPI tables are > installed > > This patch series adds a new capability to the Acpiview UEFI shell tool. > Using the -r command line parameter, it is now possible to choose a > specification which lists mandatory ACPI tables. The parameter value is then > consumed by a library which validates ACPI tables identified on the platform > against these requirements. > > The -r parameter is architecture agnostic. However, as of now, the possible > values for the parameter are only defined in the context of the Arm > architecture. > > For Arm-based platforms, it is now possible to validate that Server Base Boot > Requirements (SBBR) mandatory ACPI tables are present on the platform. > > Changes can be seen at: > https://github.com/KrzysztofKoch1/edk2/tree/617_sbbr_validate_acpi_tabl > e_counts_v1 > > Krzysztof Koch (3): > ShellPkg: acpiview: Add -r parameter for table requirements validation > ShellPkg: acpiview: Add library for SBBR ACPI requirements validation > ShellPkg: acpiview: Check if SBBR mandatory ACPI tables are installed > > ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiTableParser.c > | > 16 ++ > ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiView.c > | 104 > ++++++++- > ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiView.h > | 44 > +++- > ShellPkg/Library/UefiShellAcpiViewCommandLib/Arm/SbbrValidator.c > | 222 ++++++++++++++++++++ > ShellPkg/Library/UefiShellAcpiViewCommandLib/Arm/SbbrValidator.h > | 91 ++++++++ > > ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComman > dLib.inf | 6 +- > > ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewComman > dLib.uni | 14 +- > 7 files changed, 492 insertions(+), 5 deletions(-) create mode 100644 > ShellPkg/Library/UefiShellAcpiViewCommandLib/Arm/SbbrValidator.c > create mode 100644 > ShellPkg/Library/UefiShellAcpiViewCommandLib/Arm/SbbrValidator.h > > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#57495): https://edk2.groups.io/g/devel/message/57495 Mute This Topic: https://groups.io/mt/72537308/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
