Add back the trick logic on EFI_PCI_DEVICE_ENABLE in PciBus driver so that some old...
authorlgao4 <lgao4@de2fecce-e211-0410-80a6-f3fac2684e05>
Tue, 16 Oct 2007 09:58:55 +0000 (09:58 +0000)
committerlgao4 <lgao4@de2fecce-e211-0410-80a6-f3fac2684e05>
Tue, 16 Oct 2007 09:58:55 +0000 (09:58 +0000)
git-svn-id: https://edk2.tianocore.org/svn/edk2/trunk@4131 de2fecce-e211-0410-80a6-f3fac2684e05

edk2/IntelFrameworkModulePkg/Bus/Pci/IdeBusDxe/ata.c
edk2/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciHotPlugSupport.c
edk2/IntelFrameworkModulePkg/Bus/Pci/PciBusDxe/PciIo.c

index 3a7271e..a315953 100644 (file)
@@ -191,6 +191,8 @@ ATAIdentify (
   the capacity is below 120G, 48bit addressing is not needed\r
 \r
   @retval  EFI_DEVICE_ERROR The identify data in IdeDev is incorrect\r
+  \r
+  @retval  EFI_INVALID_PARAMETER The identify data in IdeDev is NULL.\r
 \r
   @note\r
   This function must be called after DEVICE_IDENTITY command has been\r
@@ -208,7 +210,7 @@ AtaAtapi6Identify (
   EFI_IDENTIFY_DATA *Atapi6IdentifyStruct;\r
 \r
   if (IdeDev->pIdData == NULL) {\r
-    return EFI_UNSUPPORTED;\r
+    return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   Atapi6IdentifyStruct = IdeDev->pIdData;\r
index 8593c52..6128542 100644 (file)
@@ -275,9 +275,6 @@ Returns:
   EFI_TIMEOUT - Time out\r
 \r
 --*/\r
-// TODO:    TimeoutInMilliSeconds - add argument and description to function comment\r
-// TODO:    EFI_SUCCESS - add return value to function comment\r
-// TODO:    EFI_TIMEOUT - add return value to function comment\r
 {\r
   UINT32  Delay;\r
   UINTN   Index;\r
index 63ab8e5..765cf4c 100644 (file)
@@ -1464,19 +1464,20 @@ Returns:
   // Just a trick for ENABLE attribute\r
   // EFI_PCI_DEVICE_ENABLE is not defined in UEFI spec, which is the internal usage.\r
   // So, this logic doesn't confrom to UEFI spec, which should be removed.\r
+  // But this trick logic is still kept for some binary drivers that depend on it.\r
   //\r
-  //  if ((Attributes & EFI_PCI_DEVICE_ENABLE) == EFI_PCI_DEVICE_ENABLE) {\r
-  //    Attributes &= (PciIoDevice->Supports);\r
-  //\r
-  //    //\r
-  //    // Raise the EFI_P_PC_ENABLE Status code\r
-  //    //\r
-  //    REPORT_STATUS_CODE_WITH_DEVICE_PATH (\r
-  //      EFI_PROGRESS_CODE,\r
-  //      EFI_IO_BUS_PCI | EFI_P_PC_ENABLE,\r
-  //      PciIoDevice->DevicePath\r
-  //      );\r
-  //  }\r
+  if ((Attributes & EFI_PCI_DEVICE_ENABLE) == EFI_PCI_DEVICE_ENABLE) {\r
+    Attributes &= (PciIoDevice->Supports);\r
+\r
+    //\r
+    // Raise the EFI_P_PC_ENABLE Status code\r
+    //\r
+    REPORT_STATUS_CODE_WITH_DEVICE_PATH (\r
+      EFI_PROGRESS_CODE,\r
+      EFI_IO_BUS_PCI | EFI_P_PC_ENABLE,\r
+      PciIoDevice->DevicePath\r
+      );\r
+  }\r
 \r
   //\r
   // If no attributes can be supported, then return.\r