From: "Krzysztof Koch" <krzysztof.koch@arm.com>
To: "devel@edk2.groups.io" <devel@edk2.groups.io>,
Krzysztof Koch <Krzysztof.Koch@arm.com>
Cc: "jaben.carsey@intel.com" <jaben.carsey@intel.com>,
"ray.ni@intel.com" <ray.ni@intel.com>,
"zhichao.gao@intel.com" <zhichao.gao@intel.com>,
Sami Mujawar <Sami.Mujawar@arm.com>, nd <nd@arm.com>
Subject: Re: [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring
Date: Mon, 22 Jul 2019 08:24:32 +0000 [thread overview]
Message-ID: <VE1PR08MB47836B6013F0B72563E1F0D184C40@VE1PR08MB4783.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <15B3AB4E8413227A.7925@groups.io>
Hi Everyone,
Just to let you know, only "ShellPkg: acpiview: Allow passing buffer length to DumpGasStruct()" is changed compared to v1. I had a wrong understanding of how to post revised patches to the upstream community.
Kind regards,
Krzysztof
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Krzysztof Koch via Groups.Io
Sent: Monday, July 22, 2019 8:50
To: devel@edk2.groups.io
Cc: jaben.carsey@intel.com; ray.ni@intel.com; zhichao.gao@intel.com; Sami Mujawar <Sami.Mujawar@arm.com>; Matteo Carlini <Matteo.Carlini@arm.com>; nd <nd@arm.com>
Subject: [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring
This set of patches consists of a number of changes which make the code structure consistent across the existing ACPI table parsers. These are all refactoring changes which do not modify the existing functionality of the acpiview UEFI shell tool.
Changes can be seen at: https://github.com/KrzysztofKoch1/edk2/tree/612_acpiview_code_style_enhance_v2
Krzysztof Koch (6):
ShellPkg: acpiview: Allow passing buffer length to DumpGasStruct()
ShellPkg: acpiview: XSDT: Remove redundant ParseAcpi() call
ShellPkg: acpiview: RSDP: Make code consistent with other parsers
ShellPkg: acpiview: SRAT: Minor code style enhancements
ShellPkg: acpiview: MADT: Split structure length validation
ShellPkg: acpiview: IORT: Refactor PMCG node mapping count validation
ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.c | 26 +++++++++-------
ShellPkg/Library/UefiShellAcpiViewCommandLib/AcpiParser.h | 8 +++--
ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Dbg2/Dbg2Parser.c | 2 +- ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Iort/IortParser.c | 32 ++++++++++++++------ ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Madt/MadtParser.c | 30 +++++++++++++----- ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Rsdp/RsdpParser.c | 11 +++++-- ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Srat/SratParser.c | 3 +- ShellPkg/Library/UefiShellAcpiViewCommandLib/Parsers/Xsdt/XsdtParser.c | 18 +++--------
8 files changed, 82 insertions(+), 48 deletions(-)
--
'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)'
next parent reply other threads:[~2019-07-22 8:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <15B3AB4E8413227A.7925@groups.io>
2019-07-22 8:24 ` Krzysztof Koch [this message]
2019-07-23 9:56 ` [edk2-devel] [PATCH v2 0/6] Acpiview table parsers code style enhancements and refactoring Sami Mujawar
2019-07-26 16:33 ` Krzysztof Koch
2019-07-29 6:29 ` Gao, Zhichao
2019-08-01 2:32 ` Gao, Zhichao
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=VE1PR08MB47836B6013F0B72563E1F0D184C40@VE1PR08MB4783.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