Hello,

Could you help to follow the instructions at:
https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process#the-developer-process-for-the-edk-ii-project
and send the patch mail to the edk2 mail list?

For the patch, could you help to refine the check to:
if ((Private->Cap.Css & BIT0) == 0) {

Thanks in advance.

Best Regards,
Hao Wu

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Mara
> Sophie Grosch via groups.io
> Sent: Sunday, March 20, 2022 6:31 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A <hao.a...@intel.com>; Ni, Ray <ray...@intel.com>; Wang, Jian
> J <jian.j.w...@intel.com>; Gao, Liming <gaolim...@byosoft.com.cn>
> Subject: Re: [edk2-devel] [PATCH] MdeModulePkg/NvmExpressDxe: fix check
> for Cap.Css
> 
> Hi,
> 
> sending again, because I forgot to CC reviewers.
> 
> Best regards,
> Mara Sophie Grosch
> 
> Am Wed, Mar 16, 2022 at 12:56:25PM -0700 schrieb littlefox via groups.io:
> >Fix the check for NVMe command set being supported by the controller.
> >
> >Was problematic with qemu (6.2.0, Debian 1:6.2+dfsg-3), which sets 0xC1 in
> that register, making the OVMF think the NVMe controller does not support
> NVMe.
> >
> >Uncovered by commit 9dd14fc
> ( https://github.com/tianocore/edk2/commit/9dd14fc91c174eae87fd122c7ac7
> 0073a363527f ) , which changed the number of bits included in the Css register
> from 4 to 8.
> >
> >Signed-off-by: Mara Sophie Grosch little...@lf-net.org
> >
> >Patch is on GitHub: https://github.com/tianocore/edk2/pull/2637
> >
> >$ git diff --stat master
> >
> >MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressHci.c | 2 +-
> >
> >1 file changed, 1 insertion(+), 1 deletion(-)
> >
> >
> >
> >
> >
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#87775): https://edk2.groups.io/g/devel/message/87775
Mute This Topic: https://groups.io/mt/89833412/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to