From: "Guomin Jiang" <guomin.jiang@intel.com>
To: Oleksiy Yakovlev <Oleksiyy@ami.com>,
"devel@edk2.groups.io" <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>,
Felix Polyudov <Felixp@ami.com>
Subject: Re: [edk2-devel] [PATCH v7 1/7] MdePkg: Extend SERIAL_IO with DeviceTypeGuid
Date: Wed, 20 May 2020 00:57:59 +0000 [thread overview]
Message-ID: <DM6PR11MB29553230781209D4212538129DB60@DM6PR11MB2955.namprd11.prod.outlook.com> (raw)
In-Reply-To: <6BD875C24FFE2A4E8D6EFC142C67836F03FFFFA33D@atlms1.us.megatrends.com>
Sorry for reply later.
I understood and the series is good to me.
Reviewed-by: Guomin Jiang <guomin.jiang@intel.com>
> -----Original Message-----
> From: Oleksiy Yakovlev <Oleksiyy@ami.com>
> Sent: Thursday, May 14, 2020 10:17 PM
> To: Jiang, Guomin <guomin.jiang@intel.com>; 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>;
> Felix Polyudov <Felixp@ami.com>
> Subject: RE: [edk2-devel] [PATCH v7 1/7] MdePkg: Extend SERIAL_IO with
> DeviceTypeGuid
>
> There was a mistake in all previous v here.
> I took it from original ECR and misted correction.
> Now it is up to spec.
>
> Regards, Oleksiy.
>
> -----Original Message-----
> From: Jiang, Guomin [mailto:guomin.jiang@intel.com]
> Sent: Wednesday, May 13, 2020 11:43 PM
> To: devel@edk2.groups.io; Oleksiy Yakovlev
> Cc: Feng, Bob C; Gao, Liming; Kinney, Michael D; Felix Polyudov
> Subject: RE: [edk2-devel] [PATCH v7 1/7] MdePkg: Extend SERIAL_IO with
> DeviceTypeGuid
>
> Add some comments.
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Oleksiy
> > Yakovlev
> > Sent: Thursday, May 14, 2020 3:53 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 v7 1/7] MdePkg: Extend SERIAL_IO with
> > DeviceTypeGuid
> >
> > EFI_SERIAL_IO_PROTOCOL_REVISION incremented to 0x00010001.
> > CONST EFI_GUID *DeviceTypeGuid field added to the protocol structure.
> > UEFI 2.8 mantis 1832.
> >
> > Signed-off-by: Oleksiy Yakovlev <oleksiyy@ami.com>
> >
> > Reviewed-by: Liming Gao <liming.gao@intel.com>
> > ---
> > MdePkg/Include/Protocol/SerialIo.h | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/MdePkg/Include/Protocol/SerialIo.h
> > b/MdePkg/Include/Protocol/SerialIo.h
> > index 51eaa53..e2e0c61 100644
> > --- a/MdePkg/Include/Protocol/SerialIo.h
> > +++ b/MdePkg/Include/Protocol/SerialIo.h
> > @@ -263,6 +263,7 @@ typedef struct {
> > } EFI_SERIAL_IO_MODE;
> >
> > #define EFI_SERIAL_IO_PROTOCOL_REVISION 0x00010000
> > +#define EFI_SERIAL_IO_PROTOCOL_REVISION1p1 0x00010001
>
> Why add this new definition, it is not exist in patch v5 but exist in patch v6?
>
> > #define SERIAL_IO_INTERFACE_REVISION
> > EFI_SERIAL_IO_PROTOCOL_REVISION
> >
> > ///
> > @@ -287,6 +288,14 @@ struct _EFI_SERIAL_IO_PROTOCOL {
> > /// Pointer to SERIAL_IO_MODE data.
> > ///
> > EFI_SERIAL_IO_MODE *Mode;
> > + ///
> > + /// Pointer to a GUID identifying the device connected to the serial port.
> > + /// This field is NULL when the protocol is installed by the serial
> > + port /// driver and may be populated by a platform driver for a
> > + serial port /// with a known device attached. The field will remain
> > + NULL if there is /// no platform serial device identification
> > + information
> > available.
> > + ///
> > + CONST EFI_GUID *DeviceTypeGuid; // Revision 1.1
> > };
> >
> > extern EFI_GUID gEfiSerialIoProtocolGuid;
> > --
> > 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.
> >
> >
>
>
> 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.
next prev parent reply other threads:[~2020-05-20 0:58 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-13 19:52 [PATCH v7 0/7] Add definitions introduced in UEFI 2.8 Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 1/7] MdePkg: Extend SERIAL_IO with DeviceTypeGuid Oleksiy Yakovlev
2020-05-14 3:42 ` [edk2-devel] " Guomin Jiang
2020-05-14 6:29 ` Liming Gao
2020-05-14 14:16 ` Oleksiy Yakovlev
2020-05-20 0:57 ` Guomin Jiang [this message]
2020-05-13 19:52 ` [PATCH v7 2/7] BaseTools: REST style formset Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 3/7] MdePkg: " Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 4/7] BaseTools: Bootable NVDIMM namespaces Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 5/7] MdePkg: " Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 6/7] MdePkg: Add UEFI Spec Revision 2.8 Oleksiy Yakovlev
2020-05-13 19:52 ` [PATCH v7 7/7] MdePkg: UEFI JSON Capsule Support Oleksiy Yakovlev
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=DM6PR11MB29553230781209D4212538129DB60@DM6PR11MB2955.namprd11.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