Yes, you are right. I will help commit it in after tuning the commit log.

Reviewed-by: Feng Tian <[email protected]>

-----Original Message-----
From: Anbazhagan, Baraneedharan [mailto:[email protected]] 
Sent: Monday, November 30, 2015 05:45
To: [email protected]; Tian, Feng
Subject: MdeModulePkg: Nvme Controller Name

Hi,
With recent changes to Nvme module, NvmExpressComponentNameGetControllerName 
returns EFI_UNSUPPORTED if child handle is passed. Seems like 
gEfiNvmExpressPassThruProtocolGuid needs to be passed to EfiTestChildHandle 
instead of gEfiPciIoProtocolGuid.

-Baranee

MdeModulePkg/Bus/Pci/NvmExpressDxe/ComponentName.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MdeModulePkg/Bus/Pci/NvmExpressDxe/ComponentName.c 
b/MdeModulePkg/Bus/Pci/NvmExpressDxe/ComponentName.c
index 60849ae..855a1b1 100644
--- a/MdeModulePkg/Bus/Pci/NvmExpressDxe/ComponentName.c
+++ b/MdeModulePkg/Bus/Pci/NvmExpressDxe/ComponentName.c
@@ -199,7 +199,7 @@ NvmExpressComponentNameGetControllerName (
     Status = EfiTestChildHandle (
                ControllerHandle,
                ChildHandle,
-               &gEfiPciIoProtocolGuid
+               &gEfiNvmExpressPassThruProtocolGuid
                );
     if (EFI_ERROR (Status)) {
       return Status;
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to