Re: [PATCH v2-resend 0/2] Add AEST parser support to Acpiview

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

Hi Zhichao,

May I have your feedback for this series, please?

I have raised two Bugzilla tickets for these patches:
 - Bug 3049 - Add defintion for Arm Error Source Table (AEST) [https://bugzilla.tianocore.org/show_bug.cgi?id=3049]
 - Bug 3048 - Acpiview: Add support for parsing Arm Error Source Table (AEST) [https://bugzilla.tianocore.org/show_bug.cgi?id=3048]

The links for these patches on the mailing list are as below:
 - https://edk2.groups.io/g/devel/message/66928
- https://edk2.groups.io/g/devel/message/66929
- https://edk2.groups.io/g/devel/message/66930

Regards,

Sami Mujawar

-----Original Message-----
From: Sami Mujawar <sami.mujawar@arm.com> 
Sent: 03 November 2020 02:32 PM
To: devel@edk2.groups.io
Cc: Sami Mujawar <Sami.Mujawar@arm.com>; michael.d.kinney@intel.com; gaoliming@byosoft.com.cn; zhiguang.liu@intel.com; ray.ni@intel.com; zhichao.gao@intel.com; Matteo Carlini <Matteo.Carlini@arm.com>; Ben Adderson <Ben.Adderson@arm.com>; nd <nd@arm.com>
Subject: [PATCH v2-resend 0/2] Add AEST parser support to Acpiview

Resending v2 patch series with Bugzilla IDs added. No code change
since v2 patch seiries.

Arm Error Source Table (AEST) enables kernel-first handling of
errors in a system that supports the Armv8 RAS extensions. AEST
is described in the ACPI for the Armv8 RAS Extensions 1.1,
Platform Design Document, dated 28 September 2020.
(https://developer.arm.com/documentation/den0085/0101/)

This v2 patch series:
 - adds AEST parser support to Acpiview.
 - addresses the review feedback for v1 series.

The changes can be seen at:
https://github.com/samimujawar/edk2/tree/1429_aest_parser_v2

Marc Moisson-Franckhauser (2):
  MdePkg/IndustryStandard: AEST Table definition
  ShellPkg/Acpiview: AEST Parser

 MdePkg/Include/IndustryStandard/ArmErrorSourceTable.h                        | 357 +++++++++
 ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h                    |  21 +
 ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiTableParser.h               |   4 +-
 ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Aest/AestParser.c       | 755 ++++++++++++++++++++
 ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c   |   4 +-
 ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.inf |   3 +-
 ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.uni |   3 +-
 7 files changed, 1142 insertions(+), 5 deletions(-)
 create mode 100644 MdePkg/Include/IndustryStandard/ArmErrorSourceTable.h
 create mode 100644 ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Aest/AestParser.c

-- 
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'