1. Patchew
  2. EDK2
  3. View series

[edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes

Chris Jones posted 7 patches 2 years, 6 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
.../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(-)
[edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Posted by Chris Jones 2 years, 6 months ago 
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")



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#82229): https://edk2.groups.io/g/devel/message/82229
Mute This Topic: https://groups.io/mt/86416265/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
回复: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Posted by gaoliming 2 years, 5 months ago 
For the changes in MdePkg, Reviewed-by: Liming Gao
<gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chris Jones
> 发送时间: 2021年10月18日 23:11
> 收件人: devel@edk2.groups.io
> 抄送: michael.d.kinney@intel.com; gaoliming@byosoft.com.cn;
> zhiguang.liu@intel.com; ray.ni@intel.com; zhichao.gao@intel.com;
> Alexei.Fedorov@arm.com; Sami.Mujawar@arm.com; 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")
> 
> 
> 
> 
> 





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83074): https://edk2.groups.io/g/devel/message/83074
Mute This Topic: https://groups.io/mt/86757363/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Posted by Chris Jones 2 years, 5 months ago 
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>
Sent: Tuesday, November 2, 2021 1:23 AM
To: devel@edk2.groups.io <devel@edk2.groups.io>; Christopher Jones <Christopher.Jones@arm.com>
Cc: michael.d.kinney@intel.com <michael.d.kinney@intel.com>; zhiguang.liu@intel.com <zhiguang.liu@intel.com>; ray.ni@intel.com <ray.ni@intel.com>; zhichao.gao@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

For the changes in MdePkg, Reviewed-by: Liming Gao
<gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chris Jones
> 发送时间: 2021年10月18日 23:11
> 收件人: devel@edk2.groups.io
> 抄送: michael.d.kinney@intel.com; gaoliming@byosoft.com.cn;
> zhiguang.liu@intel.com; ray.ni@intel.com; zhichao.gao@intel.com;
> Alexei.Fedorov@arm.com; Sami.Mujawar@arm.com; 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")
>
>
>
> 
>





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83281): https://edk2.groups.io/g/devel/message/83281
Mute This Topic: https://groups.io/mt/86795239/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
回复: [edk2-devel] [PATCH v1 0/7] Support ACPI 6.4 PPTT changes
Posted by gaoliming 2 years, 5 months ago 
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")
> 
> 
> 
> 
> 






-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83318): https://edk2.groups.io/g/devel/message/83318
Mute This Topic: https://groups.io/mt/86810312/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.