Any additional thoughts on this patch?
> -----Original Message-----
> From: Jeff Brasen
> Sent: Tuesday, July 19, 2022 9:14 AM
> To: gaoliming <gaolim...@byosoft.com.cn>; devel@edk2.groups.io
> Cc: jian.j.w...@intel.com; zhichao....@intel.com; ray...@intel.com
> Subject: RE: [edk2-devel] [PATCH] MdeModulePkg/UefiBootManagerLib:
> Add Disk Info support for Ufs
>
> Yes, Universal Flash Storage is based on the SCSI architectural model and
> installs gEfiExtScsiPassThruProtocolGuid to function.
>
> Thanks,
> Jeff
>
> > -----Original Message-----
> > From: gaoliming <gaolim...@byosoft.com.cn>
> > Sent: Monday, July 18, 2022 7:44 PM
> > To: devel@edk2.groups.io; Jeff Brasen <jbra...@nvidia.com>
> > Cc: jian.j.w...@intel.com; zhichao....@intel.com; ray...@intel.com
> > Subject: 回复: [edk2-devel] [PATCH]
> MdeModulePkg/UefiBootManagerLib:
> > Add Disk Info support for Ufs
> >
> > External email: Use caution opening links or attachments
> >
> >
> > Jeff:
> > I want to confirm why UFS apply the same rule to SCSI. Does UFS
> > follows SCSI spec?
> >
> > Thanks
> > Liming
> > > -----邮件原件-----
> > > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Jeff
> Brasen
> > via
> > > groups.io
> > > 发送时间: 2022年6月15日 1:54
> > > 收件人: devel@edk2.groups.io
> > > 抄送: jian.j.w...@intel.com; gaolim...@byosoft.com.cn;
> > > zhichao....@intel.com; ray...@intel.com; Jeff Brasen
> > > <jbra...@nvidia.com>
> > > 主题: [edk2-devel] [PATCH] MdeModulePkg/UefiBootManagerLib: Add
> > Disk
> > > Info support for Ufs
> > >
> > > Add support for getting disk info from UFS devices.
> > >
> > > Signed-off-by: Jeff Brasen <jbra...@nvidia.com>
> > > ---
> > > MdeModulePkg/Library/UefiBootManagerLib/BmBootDescription.c | 3
> > > ++-
> > > MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf |
> 1
> > +
> > > 2 files changed, 3 insertions(+), 1 deletion(-)
> > >
> > > diff --git
> > > a/MdeModulePkg/Library/UefiBootManagerLib/BmBootDescription.c
> > > b/MdeModulePkg/Library/UefiBootManagerLib/BmBootDescription.c
> > > index fac33b9ee9..87b82f299f 100644
> > > --- a/MdeModulePkg/Library/UefiBootManagerLib/BmBootDescription.c
> > > +++
> b/MdeModulePkg/Library/UefiBootManagerLib/BmBootDescription.c
> > > @@ -195,7 +195,8 @@ BmGetDescriptionFromDiskInfo (
> > >
> > >
> > > BmEliminateExtraSpaces (Description);
> > >
> > > }
> > >
> > > - } else if (CompareGuid (&DiskInfo->Interface,
> > > &gEfiDiskInfoScsiInterfaceGuid)) {
> > >
> > > + } else if (CompareGuid (&DiskInfo->Interface,
> > > &gEfiDiskInfoScsiInterfaceGuid) ||
> > >
> > > + CompareGuid (&DiskInfo->Interface,
> > > &gEfiDiskInfoUfsInterfaceGuid)) {
> > >
> > > BufferSize = sizeof (EFI_SCSI_INQUIRY_DATA);
> > >
> > > Status = DiskInfo->Inquiry (
> > >
> > > DiskInfo,
> > >
> > > diff --git
> > > a/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> > > b/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> > > index fe05d5f1cc..2fc0a80a4e 100644
> > > ---
> > a/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> > > +++
> > b/MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
> > > @@ -85,6 +85,7 @@
> > > gEfiDiskInfoIdeInterfaceGuid ##
> > > SOMETIMES_CONSUMES ## GUID
> > >
> > > gEfiDiskInfoScsiInterfaceGuid ##
> > > SOMETIMES_CONSUMES ## GUID
> > >
> > > gEfiDiskInfoSdMmcInterfaceGuid ##
> > > SOMETIMES_CONSUMES ## GUID
> > >
> > > + gEfiDiskInfoUfsInterfaceGuid ##
> > > SOMETIMES_CONSUMES ## GUID
> > >
> > >
> > >
> > > [Protocols]
> > >
> > > gEfiPciRootBridgeIoProtocolGuid ## CONSUMES
> > >
> > > --
> > > 2.25.1
> > >
> > >
> > >
> > >
> > >
> > >
> >
> >
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#94793): https://edk2.groups.io/g/devel/message/94793
Mute This Topic: https://groups.io/mt/92484604/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
