From: "Krzysztof Koch" <krzysztof.koch@arm.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
"zhichao.gao@intel.com" <zhichao.gao@intel.com>,
"Ni, Ray" <ray.ni@intel.com>
Cc: Sami Mujawar <Sami.Mujawar@arm.com>,
Matteo Carlini <Matteo.Carlini@arm.com>,
Laura Moretta <Laura.Moretta@arm.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH v1 0/3] Check if required ACPI tables are installed
Date: Fri, 1 May 2020 11:26:36 +0000 [thread overview]
Message-ID: <AM5PR0801MB1777CC519FE0BDFFA57C003D84AB0@AM5PR0801MB1777.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <CY4PR1101MB22306FFC9CDBD23510EB1A9FF6D90@CY4PR1101MB2230.namprd11.prod.outlook.com>
Hi Zhichao,
Any idea when this patch set will get merged?
Kind regards,
Krzysztof
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gao, Zhichao via groups.io
Sent: Friday, April 17, 2020 8:20
To: Krzysztof Koch <Krzysztof.Koch@arm.com>; devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>
Cc: Sami Mujawar <Sami.Mujawar@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Laura Moretta <Laura.Moretta@arm.com>; nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH v1 0/3] Check if required ACPI tables are installed
For the patch set:
Reviewed-by: Zhichao Gao <zhichao.gao@intel.com>
Thanks,
Zhichao
> -----Original Message-----
> From: Krzysztof Koch <Krzysztof.Koch@arm.com>
> Sent: Tuesday, April 14, 2020 4:50 PM
> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>; Gao, Zhichao
> <zhichao.gao@intel.com>
> Cc: Sami Mujawar <Sami.Mujawar@arm.com>; Matteo Carlini
> <Matteo.Carlini@arm.com>; Laura Moretta <Laura.Moretta@arm.com>; nd
> <nd@arm.com>
> 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: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Krzysztof Koch via Groups.Io
> Sent: Wednesday, March 25, 2020 9:39
> To: devel@edk2.groups.io
> Cc: ray.ni@intel.com; zhichao.gao@intel.com; Sami Mujawar
> <Sami.Mujawar@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Laura
> Moretta <Laura.Moretta@arm.com>; nd <nd@arm.com>
> 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_tab
> l
> 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)'
>
>
>
next prev parent reply other threads:[~2020-05-01 11:26 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <15FF82990C57A035.32445@groups.io>
2020-04-14 8:50 ` [edk2-devel] [PATCH v1 0/3] Check if required ACPI tables are installed Krzysztof Koch
2020-04-17 7:20 ` Gao, Zhichao
2020-05-01 11:26 ` Krzysztof Koch [this message]
2020-05-07 2:22 ` Gao, Zhichao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AM5PR0801MB1777CC519FE0BDFFA57C003D84AB0@AM5PR0801MB1777.eurprd08.prod.outlook.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox