Re: [PATCH v4 0/7] Add definitions introduced in UEFI 2.8

["Liming Gao" <liming.gao@intel.com>]

Oleksiy:
  I see you contribute three patch set for UEFI2.8/UEFI2.8a/PI1.7a. With this change, please also update spec version in MdePkg.dec

# EFI1.10/UEFI2.7/PI1.7 and some Industry Standards.
==>
# EFI1.10/UEFI2.8a/PI1.7a and some Industry Standards.

  For this patch set, I have minor comments on 1/7 and 7/7. Other patches are good to me. Reviewed-by: Liming Gao <liming.gao@intel.com>

Thanks
Liming
> -----Original Message-----
> From: Oleksiy Yakovlev <oleksiyy@ami.com>
> Sent: Saturday, May 9, 2020 5:46 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: [PATCH v4 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
>   BaseTools:  REST style formset
>   MdePkg:  REST style formset
>   BaseTools: Bootable NVDIMM namespaces
>   MdePkg: Bootable NVDIMM namespaces
>   MdePkg: Add UEFI Spec Revision 2.8
>   MdePkg: UEFI JSON Capsule Support
> 
>  .../Common/UefiInternalFormRepresentation.h        |  1 +
>  BaseTools/Source/C/Include/Protocol/DevicePath.h   | 12 +++
>  MdePkg/Include/Guid/CapsuleReport.h                | 29 +++++++
>  MdePkg/Include/Guid/HiiPlatformSetupFormset.h      |  4 +
>  MdePkg/Include/Guid/JsonCapsule.h                  | 97 ++++++++++++++++++++++
>  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                                  | 11 +++
>  10 files changed, 171 insertions(+), 1 deletion(-)
>  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.