public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Sami Mujawar" <sami.mujawar@arm.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"zhichao.gao@intel.com" <zhichao.gao@intel.com>
Cc: "Ni, Ray" <ray.ni@intel.com>,
	Matteo Carlini <Matteo.Carlini@arm.com>,
	Ben Adderson <Ben.Adderson@arm.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg/Acpiview: Add support for PCCT parser
Date: Fri, 23 Oct 2020 12:23:17 +0000	[thread overview]
Message-ID: <DB7PR08MB309780AF492676C61F92D700841A0@DB7PR08MB3097.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <MWHPR11MB1647826A14EA9069A243009DF61A0@MWHPR11MB1647.namprd11.prod.outlook.com>

Hi Zhichao,

Please find my response inline marked [SAMI]

Regards,

Sami Mujawar

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gao, Zhichao via groups.io
Sent: 23 October 2020 06:53 AM
To: devel@edk2.groups.io; Sami Mujawar <Sami.Mujawar@arm.com>
Cc: Ni, Ray <ray.ni@intel.com>; Matteo Carlini <Matteo.Carlini@arm.com>; Ben Adderson <Ben.Adderson@arm.com>; nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg/Acpiview: Add support for PCCT parser

Hi Sami,

I am OK with the patch #1 and #2. I hope you already tested the patch, did you?
[SAMI] The patch has been tested by creating a PCCT table with dummy values to test various conditions. [/SAMI]

Patch #2 is a bug fix and not related to the PCCT parser. You can put it in this patch set but it should be put before the first one. Because its fix also benefit patch #1.
[SAMI] I will resend this series with the order changed. [/SAMI]

And remind as I said before, it is highly suggested to file a BZ for a patch. For this patch set, BZ is required indeed because you put two different patches into one patch set. Two different BZ ID clearly descript that.
[SAMI] I will raise 2 bugzilla tickets and reword the commit messages for tracking when I send the updated series. [/SAMI]
I have no other comments.

Thanks,
Zhichao

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sami
> Mujawar
> Sent: Wednesday, October 21, 2020 9:59 PM
> To: Sami Mujawar <Sami.Mujawar@arm.com>; devel@edk2.groups.io
> Cc: Ni, Ray <ray.ni@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Matteo
> Carlini <Matteo.Carlini@arm.com>; Ben Adderson <Ben.Adderson@arm.com>;
> nd <nd@arm.com>
> Subject: Re: [edk2-devel] [PATCH v2 0/2] ShellPkg/Acpiview: Add support for PCCT
> parser
> 
> Hi Zhichao,
> 
> Can you let me know anything is needed for this patch series, please?
> 
> Regards,
> 
> Sami Mujawar
> 
> -----Original Message-----
> From: Sami Mujawar <sami.mujawar@arm.com>
> Sent: 24 September 2020 12:49 PM
> To: devel@edk2.groups.io
> Cc: Sami Mujawar <Sami.Mujawar@arm.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 0/2] ShellPkg/Acpiview: Add support for PCCT parser
> 
> This patch series:
>  - Addresses the feedback for v1 patch for PCCT parser.
>  - Fixes an issue wherein the field validation is not
>    invoked when a print formatter is present.
> 
> The changes can be seen at:
> https://github.com/samimujawar/edk2/tree/840_pcct_parser_v2
> 
> Marc Moisson-Franckhauser (1):
>   ShellPkg/AcpiView: PCCT Parser
> 
> Sami Mujawar (1):
>   ShellPkg/AcpiView: Fix field validator invocation
> 
>  ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c                    |  15 +-
>  ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h                    |  24 +-
>  ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiTableParser.h               |   4
> +-
>  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.c       |
> 615 ++++++++++++++++++++
>  ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.h       |
> 33 ++
> 
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.c
> |   4 +-
> 
> ShellPkg/Library/UefiShellAcpiViewCommandLib/UefiShellAcpiViewCommandLib.i
> nf |   4 +-
>  7 files changed, 686 insertions(+), 13 deletions(-)  create mode 100644
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.c
>  create mode 100644
> ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Pcct/PcctParser.h
> 
> --
> 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
> 
> 
> 
> 
> 







      reply	other threads:[~2020-10-23 12:23 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-24 11:49 [PATCH v2 0/2] ShellPkg/Acpiview: Add support for PCCT parser Sami Mujawar
2020-09-24 11:49 ` [PATCH v2 1/2] ShellPkg/AcpiView: PCCT Parser Sami Mujawar
2020-09-24 11:49 ` [PATCH v2 2/2] ShellPkg/AcpiView: Fix field validator invocation Sami Mujawar
2020-10-21 13:58 ` [PATCH v2 0/2] ShellPkg/Acpiview: Add support for PCCT parser Sami Mujawar
2020-10-23  5:53   ` [edk2-devel] " Gao, Zhichao
2020-10-23 12:23     ` Sami Mujawar [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=DB7PR08MB309780AF492676C61F92D700841A0@DB7PR08MB3097.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