Hi, Gentle reminder on the series.
Regards, Rohit > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rohit > Mathew via groups.io > Sent: Monday, July 15, 2024 8:57 AM > To: Sami Mujawar <sami.muja...@arm.com>; devel@edk2.groups.io > Cc: Michael D Kinney <michael.d.kin...@intel.com>; Liming Gao > <gaolim...@byosoft.com.cn>; Thomas Abraham > <thomas.abra...@arm.com>; James Morse <james.mo...@arm.com>; Ray > Ni <ray...@intel.com>; Zhichao Gao <zhichao....@intel.com>; nd > <n...@arm.com> > Subject: Re: [edk2-devel] [PATCH V6 0/6] MPAM ACPI definitions and parser > > Hi Sami, > > I have added these as well. > > Regards, > Rohit > > > -----Original Message----- > > From: Sami Mujawar <sami.muja...@arm.com> > > Sent: Tuesday, July 9, 2024 9:11 AM > > To: Rohit Mathew <rohit.mat...@arm.com>; devel@edk2.groups.io > > Cc: Michael D Kinney <michael.d.kin...@intel.com>; Liming Gao > > <gaolim...@byosoft.com.cn>; Thomas Abraham > <thomas.abra...@arm.com>; > > James Morse <james.mo...@arm.com>; Ray Ni <ray...@intel.com>; Zhichao > > Gao <zhichao....@intel.com>; nd <n...@arm.com> > > Subject: Re: [PATCH V6 0/6] MPAM ACPI definitions and parser > > > > Hi Rohit, > > > > I think the pull request template must be used. In addition to the > > patch series description you have added, it has questions like if the > > change is a breaking change, how was the patch tested etc. > > It would be good if you can update the pull request to use the PR template. > > > > Regards, > > > > Sami Mujawar > > > > On 09/07/2024, 09:03, "Rohit Mathew" <rohit.mat...@arm.com > > <mailto:rohit.mat...@arm.com>> wrote: > > > > > > 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 > > <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.muja...@arm.com > > > <mailto:sami.muja...@arm.com>> > > > Sent: Tuesday, July 9, 2024 8:16 AM > > > To: Rohit Mathew <rohit.mat...@arm.com > > <mailto:rohit.mat...@arm.com>>; > > > devel@edk2.groups.io <mailto:devel@edk2.groups.io> > > > Cc: Michael D Kinney <michael.d.kin...@intel.com > > > <mailto:michael.d.kin...@intel.com>>; Liming Gao > > > <gaolim...@byosoft.com.cn <mailto:gaolim...@byosoft.com.cn>>; > Thomas > > > Abraham <thomas.abra...@arm.com > > <mailto:thomas.abra...@arm.com>>; > > > James Morse <james.mo...@arm.com > <mailto:james.mo...@arm.com>>; > > Ray Ni > > > <ray...@intel.com <mailto:ray...@intel.com>>; Zhichao Gao > > > <zhichao....@intel.com <mailto:zhichao....@intel.com>>; nd > > <n...@arm.com > > > <mailto:n...@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-Develop > > > me > > > nt- > > > <https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Develo > > > pm > > > ent-> > > > 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.mat...@arm.com > > > <mailto:rohit.mat...@arm.com> <mailto:rohit.mat...@arm.com > > <mailto:rohit.mat...@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> > > > <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> > > > <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 > > > <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 > > > <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 (#120065): https://edk2.groups.io/g/devel/message/120065 Mute This Topic: https://groups.io/mt/107112126/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
