From: "gaoliming" <gaoliming@byosoft.com.cn>
To: "'Christopher Jones'" <Christopher.Jones@arm.com>,
<devel@edk2.groups.io>
Cc: <michael.d.kinney@intel.com>, <zhiguang.liu@intel.com>,
<ray.ni@intel.com>, <zhichao.gao@intel.com>,
"'Alexei Fedorov'" <Alexei.Fedorov@arm.com>,
"'Sami Mujawar'" <Sami.Mujawar@arm.com>, "'nd'" <nd@arm.com>
Subject: 回复: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Date: Thu, 4 Nov 2021 14:06:20 +0800 [thread overview]
Message-ID: <008901d7d142$16c20620$44461260$@byosoft.com.cn> (raw)
In-Reply-To: <VE1PR08MB57580A0A372E9CB8E0C413EFEF8C9@VE1PR08MB5758.eurprd08.prod.outlook.com>
[-- Attachment #1: Type: text/plain, Size: 4232 bytes --]
Sure. Liming.
发件人: Christopher Jones <Christopher.Jones@arm.com>
发送时间: 2021年11月4日 0:20
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
抄送: michael.d.kinney@intel.com; zhiguang.liu@intel.com; ray.ni@intel.com;
zhichao.gao@intel.com; Alexei Fedorov <Alexei.Fedorov@arm.com>; Sami Mujawar
<Sami.Mujawar@arm.com>; nd <nd@arm.com>
主题: Re: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Thank you for the review, Liming.
Since uploading the original patch series I noticed some issues myself and
have sent a v2 for the patch series
(https://edk2.groups.io/g/devel/message/83272). I have not made any MdePkg
changes, so I assume your review carries over to the new series?
Thanks,
Chris
_____
From: gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn>
>
Sent: Tuesday, November 2, 2021 1:23 AM
To: devel@edk2.groups.io <mailto:devel@edk2.groups.io>
<devel@edk2.groups.io <mailto:devel@edk2.groups.io> >; Christopher Jones
<Christopher.Jones@arm.com <mailto:Christopher.Jones@arm.com> >
Cc: michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com>
<michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> >; zhiguang.
liu@intel.com <mailto:zhiguang.liu@intel.com> <zhiguang.liu@intel.com
<mailto:zhiguang.liu@intel.com> >; ray.ni@intel.com
<mailto:ray.ni@intel.com> <ray.ni@intel.com <mailto:ray.ni@intel.com> >;
zhichao.gao@intel.com <mailto:zhichao.gao@intel.com> <zhichao.gao@intel.com
<mailto:zhichao.gao@intel.com> >; Alexei Fedorov <Alexei.Fedorov@arm.com
<mailto:Alexei.Fedorov@arm.com> >; Sami Mujawar <Sami.Mujawar@arm.com
<mailto:Sami.Mujawar@arm.com> >; nd <nd@arm.com <mailto:nd@arm.com> >
Subject: 回复: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
For the changes in MdePkg, Reviewed-by: Liming Gao
<gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> >
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <mailto:devel@edk2.groups.io>
<devel@edk2.groups.io <mailto:devel@edk2.groups.io> > 代表 Chris Jones
> 发送时间: 2021年10月18日 23:11
> 收件人: devel@edk2.groups.io <mailto:devel@edk2.groups.io>
> 抄送: michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> ;
gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> ;
> zhiguang.liu@intel.com <mailto:zhiguang.liu@intel.com> ; ray.ni@intel.com
<mailto:ray.ni@intel.com> ; zhichao.gao@intel.com <mailto:zhichao.gao@intel.
com> ;
> Alexei.Fedorov@arm.com <mailto:Alexei.Fedorov@arm.com> ; Sami.Mujawar@arm.
com <mailto:Sami.Mujawar@arm.com> ; nd@arm.com <mailto:nd@arm.com>
> 主题: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
>
> Bugzilla: 3697 (https://bugzilla.tianocore.org/show_bug.cgi?id=3697)
>
> This patch series updates the Acpiview PPTT parser and DynamicTablesPkg
> PPTT generator to support ACPI 6.4. This consists of two main changes:
> - The addition of the 'Cache ID' field.
> - The removal of the PPTT ID (type 2) structure.
>
> In addition add two 'Cache ID' defines and remove the type 2 PPTT
> structure from Acpi64.h as these changes were missing when Acpi64.h was
> introduced.
>
> The changes can be seen at:
> https://github.com/chris-jones-arm/edk2/tree/1632_64_acpi_cache_id_v1
>
> Chris Jones (7):
> MdePkg: Add missing Cache ID (in)valid define
> MdePkg: Remove PPTT ID type structure
> ShellPkg: Update Acpiview PPTT parser to ACPI 6.4
> ShellPkg: Add Cache ID to PPTT parser
> DynamicTablesPkg: Remove PPTT ID structure from ACPI 6.4 generator
> DynamicTablesPkg: Update PPTT generator to ACPI 6.4
> DynamicTablesPkg: Add CacheId to PPTT generator
>
> .../Include/ArmNameSpaceObjects.h | 27 +-
> .../Acpi/Arm/AcpiPpttLibArm/PpttGenerator.c | 297 +++++++-----------
> .../Acpi/Arm/AcpiPpttLibArm/PpttGenerator.h | 4 -
> MdePkg/Include/IndustryStandard/Acpi64.h | 18 +-
> .../Parsers/Pptt/PpttParser.c | 179 ++++++++---
> .../UefiShellAcpiViewCommandLib.c | 2 +-
> 6 files changed, 249 insertions(+), 278 deletions(-)
>
> --
> Guid("CE165669-3EF3-493F-B85D-6190EE5B9759")
>
>
>
>
>
[-- Attachment #2: Type: text/html, Size: 10485 bytes --]
prev parent reply other threads:[~2021-11-04 6:06 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-18 15:10 [PATCH v1 0/7] Support ACPI 6.4 PPTT changes Chris Jones
2021-10-18 15:10 ` [PATCH v1 1/7] MdePkg: Add missing Cache ID (in)valid define Chris Jones
2021-10-18 15:10 ` [PATCH v1 2/7] MdePkg: Remove PPTT ID type structure Chris Jones
2021-10-29 1:34 ` 回复: " gaoliming
2021-10-29 9:18 ` Chris Jones
2021-10-18 15:10 ` [PATCH v1 3/7] ShellPkg: Update Acpiview PPTT parser to ACPI 6.4 Chris Jones
2021-10-18 16:33 ` [edk2-devel] " Jeremy Linton
2021-11-03 17:47 ` Chris Jones
2021-10-18 15:10 ` [PATCH v1 4/7] ShellPkg: Add Cache ID to PPTT parser Chris Jones
2021-10-18 15:10 ` [PATCH v1 5/7] DynamicTablesPkg: Remove PPTT ID structure from ACPI 6.4 generator Chris Jones
2021-10-18 15:10 ` [PATCH v1 6/7] DynamicTablesPkg: Update PPTT generator to ACPI 6.4 Chris Jones
2021-10-18 15:10 ` [PATCH v1 7/7] DynamicTablesPkg: Add CacheId to PPTT generator Chris Jones
2021-11-02 1:23 ` 回复: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes gaoliming
2021-11-03 16:20 ` Chris Jones
2021-11-04 6:06 ` gaoliming [this message]
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='008901d7d142$16c20620$44461260$@byosoft.com.cn' \
--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