Re: [edk2-devel] [PATCH v1 0/7] Add definitions introduced in UEFI 2.8

["Zhiguang Liu" <zhiguang.liu@intel.com>]

Hi Oleksiy,

There are tabs and Trailing whitespace in this patch set, which is not consistent with edk2 coding type. 
Please run ECC tool to check your code.
https://github.com/tianocore/tianocore.github.io/wiki/ECC-tool

Thanks
Zhiguang

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Oleksiy Yakovlev
Sent: Friday, April 10, 2020 4:26 AM
To: devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Gao, Liming <liming.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Felixp@ami.com; oleksiyy@ami.com
Subject: [edk2-devel] [PATCH v1 0/7] Add definitions introduced in UEFI 2.8

Following patches add/update structures/definitions, that was introduced/modified by UEFI Spec v.2.8.
Oleksiy Yakovlev (7):
  MdePkg: Extend SERIAL_IO with DeviceTypeGuid (UEFI 2.8 mantis 1832)
  BaseTools:  REST style formset (UEFI 2.8 mantis 1853)
  MdePkg:  REST style formset (UEFI 2.8 mantis 1853)
  BaseTools: Bootable NVDIMM namespaces (UEFI 2.8 mantis 1858)
  MdePkg: Bootable NVDIMM namespaces (UEFI 2.8 mantis 1858)
  MdePkg: Add UEFI Spec Revision 2.8 (UEFI 2.8 mantis 1926)
  MdePkg: UEFI JSON Capsule Support (UEFI 2.8 mantis 1935)

 .../Common/UefiInternalFormRepresentation.h        |  1 +
 BaseTools/Source/C/Include/Protocol/DevicePath.h   | 12 +++
 MdePkg/Include/Guid/CapsuleReport.h                | 32 ++++++-
 MdePkg/Include/Guid/HiiPlatformSetupFormset.h      |  4 +
 MdePkg/Include/Guid/JsonCapsule.h                  | 99 ++++++++++++++++++++++
 MdePkg/Include/Protocol/DevicePath.h               | 12 +++
 MdePkg/Include/Protocol/SerialIo.h                 |  3 +-
 .../Include/Uefi/UefiInternalFormRepresentation.h  |  1 +
 MdePkg/Include/Uefi/UefiSpec.h                     |  2 +
 MdePkg/MdePkg.dec                                  | 12 +++
 10 files changed, 176 insertions(+), 2 deletions(-)  create mode 100644 MdePkg/Include/Guid/JsonCapsule.h

--
2.9.0.windows.1


Please consider the environment before printing this email.

The information contained in this message may be confidential and proprietary to American Megatrends (AMI).  This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited.  Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission.