public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Rohit Mathew" <rohit.mathew@arm.com>
To: Sami Mujawar <Sami.Mujawar@arm.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: Michael D Kinney <michael.d.kinney@intel.com>,
	Liming Gao <gaoliming@byosoft.com.cn>,
	Thomas Abraham <thomas.abraham@arm.com>,
	James Morse <James.Morse@arm.com>, Ray Ni <ray.ni@intel.com>,
	Zhichao Gao <zhichao.gao@intel.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH V6 0/6] MPAM ACPI definitions and parser
Date: Tue, 9 Jul 2024 08:03:40 +0000	[thread overview]
Message-ID: <AM6PR08MB37838AF58CEAB3A90A0666CC8FDB2@AM6PR08MB3783.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <743102E1-553C-4DF5-AD98-F8047D64BAE3@arm.com>

Thank you, Sami, for letting me know. I had created a PR as well for making sure the CI goes through here - https://github.com/tianocore/edk2/pull/5888

Could you please let me know if this would be sufficient?

Regards,
Rohit

> -----Original Message-----
> From: Sami Mujawar <Sami.Mujawar@arm.com>
> Sent: Tuesday, July 9, 2024 8:16 AM
> To: Rohit Mathew <Rohit.Mathew@arm.com>; devel@edk2.groups.io
> Cc: Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Thomas Abraham
> <thomas.abraham@arm.com>; James Morse <James.Morse@arm.com>; Ray
> Ni <ray.ni@intel.com>; Zhichao Gao <zhichao.gao@intel.com>; nd
> <nd@arm.com>
> Subject: Re: [PATCH V6 0/6] MPAM ACPI definitions and parser
> 
> Hi Rohit,
> 
> Edk2 has moved to a pull request model for code review and merge.
> See the latest guidelines at
> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-
> Process
> I replied to your previous patch series on the mailing list as it was sent before
> the new process was introduced.
> You will need to create a PR for the new series to progress.
> 
> Regards,
> 
> Sami Mujawar
> 
> On 08/07/2024, 22:35, "Rohit Mathew" <rohit.mathew@arm.com
> <mailto:rohit.mathew@arm.com>> wrote:
> 
> 
> This series adds the following
> 
> 
> * definitions corresponding to MPAM ACPI 2.0 specification.
> * MPAM parser An MPAM
> 
> 
> ACPI table formulated using the newly added MPAM ACPI definitions were
> validated on the linux kernel tree at [1]. The same table was parsed via
> acpiview using the newly added parser. Certain aspects of the MPAM ACPI
> specification are still not implemented by the kernel tree. These aspects were
> verified only using acpiview.
> 
> 
> Changes since V1:
> Addressed comments on MPAM ACPI definitions from Sami.
> V1 did not incorporate the parser. V2 has this implemented.
> 
> 
> Changes since V2:
> Addressed comments related to Interface/Link type defintions from Sami.
> 
> 
> Changes since V3:
> Addressed comments from Pierre
> 
> 
> Changes since V4:
> Addressed comments from Sami regarding Parser.
> 
> 
> Changes since V5:
> Addressed comments from Sami regarding Parser.
> 
> 
> Changes can be found at:
> https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6
> <https://github.com/rohit-arm/edk2/tree/mpam_acpi_v6>
> 
> 
> CI results/PR can be found at:
> https://github.com/tianocore/edk2/pull/5888
> <https://github.com/tianocore/edk2/pull/5888>
> 
> 
> [1]
> https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?h=mpam
> /snapshot/v6.7-rc2
> <https://git.kernel.org/pub/scm/linux/kernel/git/morse/linux.git/log/?h=mpa
> m/snapshot/v6.7-rc2>
> 
> 
> Rohit Mathew (6):
> MdePkg/IndustryStandard: Add definitions for MPAM ACPI specification
> ShellPkg/AcpiView: Update field-validator prototype
> ShellPkg/AcpiView: Update print-formatter prototype
> ShellPkg: acpiview: Add routine to print 16 chars
> ShellPkg: acpiview: Add routines to print reserved fields
> ShellPkg/AcpiView: Add MPAM Parser
> 
> 
> MdePkg/Include/IndustryStandard/Acpi65.h | 7 +-
> MdePkg/Include/IndustryStandard/Mpam.h | 246 ++++
> ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c | 207 +++-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h | 113 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Aest/AestParser.c |
> 36 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Dbg2/Dbg2Parser.c |
> 8 +- ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Erst/ErstParser.c
> | 28 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Fadt/FadtParser.c |
> 24 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Gtdt/GtdtParser.c |
> 14 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Hmat/HmatParser.c |
> 12 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Hpet/HpetParser.c |
> 19 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c | 32
> +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Madt/MadtParser.c |
> 24 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Mpam/MpamParser.
> c | 1241 ++++++++++++++++++++
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.c |
> 56 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pptt/PpttParser.c |
> 38 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Rsdp/RsdpParser.c |
> 14 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Spcr/SpcrParser.c |
> 14 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Srat/SratParser.c | 26
> +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Wsmt/WsmtParser.c
> | 20 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> Lib.c | 3 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> Lib.inf | 3 +-
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommand
> Lib.uni | 3 +-
> 23 files changed, 2040 insertions(+), 148 deletions(-) create mode 100644
> MdePkg/Include/IndustryStandard/Mpam.h
> create mode 100644
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Mpam/MpamParser.
> c
> 
> 
> --
> 2.34.1
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119828): https://edk2.groups.io/g/devel/message/119828
Mute This Topic: https://groups.io/mt/107112126/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2024-07-09  8:04 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-08 21:35 [edk2-devel] [PATCH V6 0/6] MPAM ACPI definitions and parser Rohit Mathew
2024-07-09  7:15 ` Sami Mujawar
2024-07-09  8:03   ` Rohit Mathew [this message]
2024-07-09  8:10     ` Sami Mujawar
2024-07-15  7:56       ` Rohit Mathew
     [not found]       ` <17E253DECB9ED3EF.4882@groups.io>
2024-07-29 19:50         ` Rohit Mathew

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=AM6PR08MB37838AF58CEAB3A90A0666CC8FDB2@AM6PR08MB3783.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