1. Added EFI_MEDIA_CHANGED and EFI_INVALID_PARAMETER returns in UsbMassReadBlocks().
authorvanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Wed, 13 Feb 2008 09:08:24 +0000 (09:08 +0000)
committervanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Wed, 13 Feb 2008 09:08:24 +0000 (09:08 +0000)
2. Use DEBUG () to replace some native debug function in USB stack modules.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@4689 6f19259b-4bc3-4df7-8a09-765794883524

17 files changed:
MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c
MdeModulePkg/Bus/Pci/EhciDxe/EhciDebug.c
MdeModulePkg/Bus/Pci/EhciDxe/EhciDebug.h
MdeModulePkg/Bus/Pci/EhciDxe/EhciReg.c
MdeModulePkg/Bus/Pci/EhciDxe/EhciSched.c
MdeModulePkg/Bus/Pci/EhciDxe/UsbHcMem.c
MdeModulePkg/Bus/Pci/UhciDxe/Uhci.c
MdeModulePkg/Bus/Pci/UhciDxe/UhciDebug.c
MdeModulePkg/Bus/Pci/UhciDxe/UhciDebug.h
MdeModulePkg/Bus/Pci/UhciDxe/UhciReg.c
MdeModulePkg/Bus/Pci/UhciDxe/UhciSched.c
MdeModulePkg/Bus/Pci/UhciDxe/UsbHcMem.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMass.h
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBoot.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBot.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassCbi.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassImpl.c

index e37cf71..e8fb112 100644 (file)
@@ -81,7 +81,7 @@ EhcGetCapability (
   *PortNumber     = (UINT8) (Ehc->HcStructParams & HCSP_NPORTS);\r
   *Is64BitCapable = (UINT8) (Ehc->HcCapParams & HCCP_64BIT);\r
 \r
-  EHC_DEBUG (("EhcGetCapability: %d ports, 64 bit %d\n", *PortNumber, *Is64BitCapable));\r
+  DEBUG ((EFI_D_INFO, "EhcGetCapability: %d ports, 64 bit %d\n", *PortNumber, *Is64BitCapable));\r
 \r
   gBS->RestoreTPL (OldTpl);\r
   return EFI_SUCCESS;\r
@@ -161,7 +161,7 @@ EhcReset (
   }\r
 \r
 ON_EXIT:\r
-  EHC_DEBUG (("EhcReset: exit status %r\n", Status));\r
+  DEBUG ((EFI_D_INFO, "EhcReset: exit status %r\n", Status));\r
   gBS->RestoreTPL (OldTpl);\r
   return Status;\r
 }\r
@@ -206,7 +206,7 @@ EhcGetState (
 \r
   gBS->RestoreTPL (OldTpl);\r
 \r
-  EHC_DEBUG (("EhcGetState: current state %d\n", *State));\r
+  DEBUG ((EFI_D_INFO, "EhcGetState: current state %d\n", *State));\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -281,7 +281,7 @@ EhcSetState (
     Status = EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  EHC_DEBUG (("EhcSetState: exit status %r\n", Status));\r
+  DEBUG ((EFI_D_INFO, "EhcSetState: exit status %r\n", Status));\r
   gBS->RestoreTPL (OldTpl);\r
   return Status;\r
 }\r
@@ -450,7 +450,7 @@ EhcSetRootHubPortFeature (
       Status = EhcRunHC (Ehc, EHC_GENERIC_TIMEOUT);\r
 \r
       if (EFI_ERROR (Status)) {\r
-        EHC_DEBUG (("EhcSetRootHubPortFeature :failed to start HC - %r\n", Status));\r
+        DEBUG ((EFI_D_INFO, "EhcSetRootHubPortFeature :failed to start HC - %r\n", Status));\r
         break;\r
       }\r
     }\r
@@ -480,7 +480,7 @@ EhcSetRootHubPortFeature (
   }\r
 \r
 ON_EXIT:\r
-  EHC_DEBUG (("EhcSetRootHubPortFeature: exit status %r\n", Status));\r
+  DEBUG ((EFI_D_INFO, "EhcSetRootHubPortFeature: exit status %r\n", Status));\r
 \r
   gBS->RestoreTPL (OldTpl);\r
   return Status;\r
@@ -608,7 +608,7 @@ EhcClearRootHubPortFeature (
   }\r
 \r
 ON_EXIT:\r
-  EHC_DEBUG (("EhcClearRootHubPortFeature: exit status %r\n", Status));\r
+  DEBUG ((EFI_D_INFO, "EhcClearRootHubPortFeature: exit status %r\n", Status));\r
   gBS->RestoreTPL (OldTpl);\r
   return Status;\r
 }\r
@@ -700,7 +700,7 @@ EhcControlTransfer (
   *TransferResult = EFI_USB_ERR_SYSTEM;\r
 \r
   if (EhcIsHalt (Ehc) || EhcIsSysError (Ehc)) {\r
-    EHC_ERROR (("EhcControlTransfer: HC halted at entrance\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcControlTransfer: HC halted at entrance\n"));\r
 \r
     EhcAckAllInterrupt (Ehc);\r
     goto ON_EXIT;\r
@@ -736,7 +736,7 @@ EhcControlTransfer (
           );\r
 \r
   if (Urb == NULL) {\r
-    EHC_ERROR (("EhcControlTransfer: failed to create URB"));\r
+    DEBUG ((EFI_D_ERROR, "EhcControlTransfer: failed to create URB"));\r
 \r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
@@ -765,7 +765,7 @@ ON_EXIT:
   gBS->RestoreTPL (OldTpl);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcControlTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
+    DEBUG ((EFI_D_ERROR, "EhcControlTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
   }\r
 \r
   return Status;\r
@@ -851,7 +851,7 @@ EhcBulkTransfer (
   Status          = EFI_DEVICE_ERROR;\r
 \r
   if (EhcIsHalt (Ehc) || EhcIsSysError (Ehc)) {\r
-    EHC_ERROR (("EhcBulkTransfer: HC is halted\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcBulkTransfer: HC is halted\n"));\r
 \r
     EhcAckAllInterrupt (Ehc);\r
     goto ON_EXIT;\r
@@ -881,7 +881,7 @@ EhcBulkTransfer (
           );\r
 \r
   if (Urb == NULL) {\r
-    EHC_ERROR (("EhcBulkTransfer: failed to create URB\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcBulkTransfer: failed to create URB\n"));\r
 \r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
@@ -907,7 +907,7 @@ ON_EXIT:
   gBS->RestoreTPL (OldTpl);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcBulkTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
+    DEBUG ((EFI_D_ERROR, "EhcBulkTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
   }\r
 \r
   return Status;\r
@@ -998,14 +998,14 @@ EhcAsyncInterruptTransfer (
   if (!IsNewTransfer) {\r
     Status = EhciDelAsyncIntTransfer (Ehc, DeviceAddress, EndPointAddress, DataToggle);\r
 \r
-    EHC_DEBUG (("EhcAsyncInterruptTransfer: remove old transfer - %r\n", Status));\r
+    DEBUG ((EFI_D_INFO, "EhcAsyncInterruptTransfer: remove old transfer - %r\n", Status));\r
     goto ON_EXIT;\r
   }\r
 \r
   Status = EFI_SUCCESS;\r
 \r
   if (EhcIsHalt (Ehc) || EhcIsSysError (Ehc)) {\r
-    EHC_ERROR (("EhcAsyncInterruptTransfer: HC is halt\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcAsyncInterruptTransfer: HC is halt\n"));\r
     EhcAckAllInterrupt (Ehc);\r
 \r
     Status = EFI_DEVICE_ERROR;\r
@@ -1017,7 +1017,7 @@ EhcAsyncInterruptTransfer (
   Data = AllocatePool (DataLength);\r
 \r
   if (Data == NULL) {\r
-    EHC_ERROR (("EhcAsyncInterruptTransfer: failed to allocate buffer\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcAsyncInterruptTransfer: failed to allocate buffer\n"));\r
 \r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
@@ -1041,7 +1041,7 @@ EhcAsyncInterruptTransfer (
           );\r
 \r
   if (Urb == NULL) {\r
-    EHC_ERROR (("EhcAsyncInterruptTransfer: failed to create URB\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcAsyncInterruptTransfer: failed to create URB\n"));\r
 \r
     gBS->FreePool (Data);\r
     Status = EFI_OUT_OF_RESOURCES;\r
@@ -1141,7 +1141,7 @@ EhcSyncInterruptTransfer (
   Status          = EFI_DEVICE_ERROR;\r
 \r
   if (EhcIsHalt (Ehc) || EhcIsSysError (Ehc)) {\r
-    EHC_ERROR (("EhcSyncInterruptTransfer: HC is halt\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcSyncInterruptTransfer: HC is halt\n"));\r
 \r
     EhcAckAllInterrupt (Ehc);\r
     goto ON_EXIT;\r
@@ -1167,7 +1167,7 @@ EhcSyncInterruptTransfer (
           );\r
 \r
   if (Urb == NULL) {\r
-    EHC_ERROR (("EhcSyncInterruptTransfer: failed to create URB\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcSyncInterruptTransfer: failed to create URB\n"));\r
 \r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto ON_EXIT;\r
@@ -1190,7 +1190,7 @@ ON_EXIT:
   gBS->RestoreTPL (OldTpl);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcSyncInterruptTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
+    DEBUG ((EFI_D_ERROR, "EhcSyncInterruptTransfer: error - %r, transfer - %x\n", Status, *TransferResult));\r
   }\r
 \r
   return Status;\r
@@ -1448,7 +1448,7 @@ EhcCreateUsb2Hc (
   Ehc->HcCapParams    = EhcReadCapRegister (Ehc, EHC_HCCPARAMS_OFFSET);\r
   Ehc->CapLen         = EhcReadCapRegister (Ehc, EHC_CAPLENGTH_OFFSET) & 0x0FF;\r
 \r
-  EHC_DEBUG (("EhcCreateUsb2Hc: capability length %d\n", Ehc->CapLen));\r
+  DEBUG ((EFI_D_INFO, "EhcCreateUsb2Hc: capability length %d\n", Ehc->CapLen));\r
 \r
   //\r
   // Create AsyncRequest Polling Timer\r
@@ -1511,7 +1511,7 @@ EhcDriverBindingStart (
                   );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to open PCI_IO\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to open PCI_IO\n"));\r
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
@@ -1548,7 +1548,7 @@ EhcDriverBindingStart (
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to enable controller\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to enable controller\n"));\r
     goto CLOSE_PCIIO;\r
   }\r
 \r
@@ -1558,7 +1558,7 @@ EhcDriverBindingStart (
   Ehc = EhcCreateUsb2Hc (PciIo, OriginalPciAttributes);\r
 \r
   if (Ehc == NULL) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to create USB2_HC\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to create USB2_HC\n"));\r
 \r
     Status = EFI_OUT_OF_RESOURCES;\r
     goto CLOSE_PCIIO;\r
@@ -1572,7 +1572,7 @@ EhcDriverBindingStart (
                   );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to install USB2_HC Protocol\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to install USB2_HC Protocol\n"));\r
     goto FREE_POOL;\r
   }\r
 \r
@@ -1585,7 +1585,7 @@ EhcDriverBindingStart (
   Status = EhcInitHC (Ehc);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to init host controller\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to init host controller\n"));\r
     goto UNINSTALL_USBHC;\r
   }\r
 \r
@@ -1595,7 +1595,7 @@ EhcDriverBindingStart (
   Status = gBS->SetTimer (Ehc->PollTimer, TimerPeriodic, EHC_ASYNC_POLL_INTERVAL);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcDriverBindingStart: failed to start async interrupt monitor\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcDriverBindingStart: failed to start async interrupt monitor\n"));\r
 \r
     EhcHaltHC (Ehc, EHC_GENERIC_TIMEOUT);\r
     goto UNINSTALL_USBHC;\r
@@ -1621,7 +1621,7 @@ EhcDriverBindingStart (
     );\r
 \r
 \r
-  EHC_DEBUG (("EhcDriverBindingStart: EHCI started for controller @ %x\n", Controller));\r
+  DEBUG ((EFI_D_INFO, "EhcDriverBindingStart: EHCI started for controller @ %x\n", Controller));\r
   return EFI_SUCCESS;\r
 \r
 UNINSTALL_USBHC:\r
index 086cdbc..25a4dbf 100644 (file)
@@ -15,8 +15,6 @@ Module Name:
 \r
 Abstract:\r
   This file provides the information dump support for EHCI when in debug mode.\r
-You can dynamically adjust the debug level by changing variable mEhcDebugLevel\r
-and mEhcErrorLevel.\r
 \r
 Revision History\r
 \r
@@ -25,100 +23,11 @@ Revision History
 \r
 #include "Ehci.h"\r
 \r
-#ifdef EFI_DEBUG\r
-UINTN mEhcDebugMask   = USB_DEBUG_FORCE_OUTPUT;\r
-\r
-\r
-/**\r
-  EHCI's debug output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Level    The output level\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhciDebugPrint (\r
-  IN  UINTN               Level,\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-\r
-  if (Level & mEhcDebugMask) {\r
-    if (mEhcDebugMask & USB_DEBUG_FORCE_OUTPUT) {\r
-      DebugVPrint (DEBUG_ERROR, Format, Marker);\r
-    } else {\r
-      DebugVPrint (DEBUG_INFO, Format, Marker);\r
-    }\r
-  }\r
-\r
-  VA_END (Marker);\r
-}\r
-\r
-\r
-/**\r
-  EHCI's debug output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhcDebug (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-  DebugVPrint (DEBUG_INFO, Format, Marker);\r
-  VA_END (Marker);\r
-}\r
-\r
-\r
-/**\r
-  EHCI's error output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhcError (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-  DebugVPrint (DEBUG_ERROR, Format, Marker);\r
-  VA_END (Marker);\r
-}\r
-\r
-\r
 /**\r
   Dump the status byte in QTD/QH to a more friendly\r
   format\r
 \r
   @param  State    The state in the QTD/QH\r
-  @param  Level    The output level\r
 \r
   @return None\r
 \r
@@ -126,43 +35,42 @@ EhcError (
 STATIC\r
 VOID\r
 EhcDumpStatus (\r
-  IN UINT32               State,\r
-  IN UINTN                Level\r
+  IN UINT32               State\r
   )\r
 {\r
   if (EHC_BIT_IS_SET (State, QTD_STAT_DO_PING)) {\r
-    EhciDebugPrint (Level, "  Do_Ping");\r
+    DEBUG ((EFI_D_INFO, "  Do_Ping"));\r
   } else {\r
-    EhciDebugPrint (Level, "  Do_Out");\r
+    DEBUG ((EFI_D_INFO, "  Do_Out"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_DO_CS)) {\r
-    EhciDebugPrint (Level, "  Do_CS");\r
+    DEBUG ((EFI_D_INFO, "  Do_CS"));\r
   } else {\r
-    EhciDebugPrint (Level, "  Do_SS");\r
+    DEBUG ((EFI_D_INFO, "  Do_SS"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_TRANS_ERR)) {\r
-    EhciDebugPrint (Level, "  Transfer_Error");\r
+    DEBUG ((EFI_D_INFO, "  Transfer_Error"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_BABBLE_ERR)) {\r
-    EhciDebugPrint (Level, "  Babble_Error");\r
+    DEBUG ((EFI_D_INFO, "  Babble_Error"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_BUFF_ERR)) {\r
-    EhciDebugPrint (Level, "  Buffer_Error");\r
+    DEBUG ((EFI_D_INFO, "  Buffer_Error"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_HALTED)) {\r
-    EhciDebugPrint (Level, "  Halted");\r
+    DEBUG ((EFI_D_INFO, "  Halted"));\r
   }\r
 \r
   if (EHC_BIT_IS_SET (State, QTD_STAT_ACTIVE)) {\r
-    EhciDebugPrint (Level, "  Active");\r
+    DEBUG ((EFI_D_INFO, "  Active"));\r
   }\r
 \r
-  EhciDebugPrint (Level, "\n");\r
+  DEBUG ((EFI_D_INFO, "\n"));\r
 }\r
 \r
 \r
@@ -178,47 +86,44 @@ EhcDumpStatus (
 VOID\r
 EhcDumpQtd (\r
   IN EHC_QTD              *Qtd,\r
-  IN UINT8                *Msg\r
+  IN CHAR8                *Msg\r
   )\r
 {\r
   QTD_HW                  *QtdHw;\r
   UINTN                   Index;\r
-  UINTN                   Level;\r
-\r
-  Level = EHC_DEBUG_QTD;\r
 \r
   if (Msg != NULL) {\r
-    EhciDebugPrint (Level, Msg);\r
+    DEBUG ((EFI_D_INFO, Msg));\r
   }\r
 \r
-  EhciDebugPrint (Level, "Queue TD @ 0x%x, data length %d\n", Qtd, Qtd->DataLen);\r
+  DEBUG ((EFI_D_INFO, "Queue TD @ 0x%x, data length %d\n", Qtd, Qtd->DataLen));\r
 \r
   QtdHw = &Qtd->QtdHw;\r
 \r
-  EhciDebugPrint (Level, "Next QTD     : %x\n", QtdHw->NextQtd);\r
-  EhciDebugPrint (Level, "AltNext QTD  : %x\n", QtdHw->AltNext);\r
-  EhciDebugPrint (Level, "Status       : %x\n", QtdHw->Status);\r
-  EhcDumpStatus (QtdHw->Status, Level);\r
+  DEBUG ((EFI_D_INFO, "Next QTD     : %x\n", QtdHw->NextQtd));\r
+  DEBUG ((EFI_D_INFO, "AltNext QTD  : %x\n", QtdHw->AltNext));\r
+  DEBUG ((EFI_D_INFO, "Status       : %x\n", QtdHw->Status));\r
+  EhcDumpStatus (QtdHw->Status);\r
 \r
   if (QtdHw->Pid == QTD_PID_SETUP) {\r
-    EhciDebugPrint (Level, "PID          : Setup\n");\r
+    DEBUG ((EFI_D_INFO, "PID          : Setup\n"));\r
 \r
   } else if (QtdHw->Pid == QTD_PID_INPUT) {\r
-    EhciDebugPrint (Level, "PID          : IN\n");\r
+    DEBUG ((EFI_D_INFO, "PID          : IN\n"));\r
 \r
   } else if (QtdHw->Pid == QTD_PID_OUTPUT) {\r
-    EhciDebugPrint (Level, "PID          : OUT\n");\r
+    DEBUG ((EFI_D_INFO, "PID          : OUT\n"));\r
 \r
   }\r
 \r
-  EhciDebugPrint (Level, "Error Count  : %d\n", QtdHw->ErrCnt);\r
-  EhciDebugPrint (Level, "Current Page : %d\n", QtdHw->CurPage);\r
-  EhciDebugPrint (Level, "IOC          : %d\n", QtdHw->IOC);\r
-  EhciDebugPrint (Level, "Total Bytes  : %d\n", QtdHw->TotalBytes);\r
-  EhciDebugPrint (Level, "Data Toggle  : %d\n", QtdHw->DataToggle);\r
+  DEBUG ((EFI_D_INFO, "Error Count  : %d\n", QtdHw->ErrCnt));\r
+  DEBUG ((EFI_D_INFO, "Current Page : %d\n", QtdHw->CurPage));\r
+  DEBUG ((EFI_D_INFO, "IOC          : %d\n", QtdHw->IOC));\r
+  DEBUG ((EFI_D_INFO, "Total Bytes  : %d\n", QtdHw->TotalBytes));\r
+  DEBUG ((EFI_D_INFO, "Data Toggle  : %d\n", QtdHw->DataToggle));\r
 \r
   for (Index = 0; Index < 5; Index++) {\r
-    EhciDebugPrint (Level, "Page[%d]      : 0x%x\n", Index, QtdHw->Page[Index]);\r
+    DEBUG ((EFI_D_INFO, "Page[%d]      : 0x%x\n", Index, QtdHw->Page[Index]));\r
   }\r
 }\r
 \r
@@ -236,7 +141,7 @@ EhcDumpQtd (
 VOID\r
 EhcDumpQh (\r
   IN EHC_QH               *Qh,\r
-  IN UINT8                *Msg,\r
+  IN CHAR8                *Msg,\r
   IN BOOLEAN              DumpBuf\r
   )\r
 {\r
@@ -244,64 +149,62 @@ EhcDumpQh (
   QH_HW                   *QhHw;\r
   LIST_ENTRY              *Entry;\r
   UINTN                   Index;\r
-  UINTN                   Level;\r
-\r
-  Level = EHC_DEBUG_QH;\r
 \r
   if (Msg != NULL) {\r
-    EhciDebugPrint (Level, Msg);\r
+    DEBUG ((EFI_D_INFO, Msg));\r
   }\r
 \r
-  EhciDebugPrint (Level, "Queue head @ 0x%x, interval %d, next qh %x\n",\r
-                                Qh, Qh->Interval, Qh->NextQh);\r
+  DEBUG ((EFI_D_INFO, "Queue head @ 0x%x, interval %d, next qh %x\n",\r
+                                Qh, Qh->Interval, Qh->NextQh));\r
 \r
   QhHw = &Qh->QhHw;\r
 \r
-  EhciDebugPrint (Level, "Hoziontal link: %x\n", QhHw->HorizonLink);\r
-  EhciDebugPrint (Level, "Device address: %d\n", QhHw->DeviceAddr);\r
-  EhciDebugPrint (Level, "Inactive      : %d\n", QhHw->Inactive);\r
-  EhciDebugPrint (Level, "EP number     : %d\n", QhHw->EpNum);\r
-  EhciDebugPrint (Level, "EP speed      : %d\n", QhHw->EpSpeed);\r
-  EhciDebugPrint (Level, "DT control    : %d\n", QhHw->DtCtrl);\r
-  EhciDebugPrint (Level, "Reclaim head  : %d\n", QhHw->ReclaimHead);\r
-  EhciDebugPrint (Level, "Max packet len: %d\n", QhHw->MaxPacketLen);\r
-  EhciDebugPrint (Level, "Ctrl EP       : %d\n", QhHw->CtrlEp);\r
-  EhciDebugPrint (Level, "Nak reload    : %d\n", QhHw->NakReload);\r
-\r
-  EhciDebugPrint (Level, "SMask         : %x\n", QhHw->SMask);\r
-  EhciDebugPrint (Level, "CMask         : %x\n", QhHw->CMask);\r
-  EhciDebugPrint (Level, "Hub address   : %d\n", QhHw->HubAddr);\r
-  EhciDebugPrint (Level, "Hub port      : %d\n", QhHw->PortNum);\r
-  EhciDebugPrint (Level, "Multiplier    : %d\n", QhHw->Multiplier);\r
-\r
-  EhciDebugPrint (Level, "Cur QTD       : %x\n", QhHw->CurQtd);\r
-\r
-  EhciDebugPrint (Level, "Next QTD      : %x\n", QhHw->NextQtd);\r
-  EhciDebugPrint (Level, "AltNext QTD   : %x\n", QhHw->AltQtd);\r
-  EhciDebugPrint (Level, "Status        : %x\n", QhHw->Status);\r
-  EhcDumpStatus (QhHw->Status, Level);\r
+  DEBUG ((EFI_D_INFO, "Hoziontal link: %x\n", QhHw->HorizonLink));\r
+  DEBUG ((EFI_D_INFO, "Device address: %d\n", QhHw->DeviceAddr));\r
+  DEBUG ((EFI_D_INFO, "Inactive      : %d\n", QhHw->Inactive));\r
+  DEBUG ((EFI_D_INFO, "EP number     : %d\n", QhHw->EpNum));\r
+  DEBUG ((EFI_D_INFO, "EP speed      : %d\n", QhHw->EpSpeed));\r
+  DEBUG ((EFI_D_INFO, "DT control    : %d\n", QhHw->DtCtrl));\r
+  DEBUG ((EFI_D_INFO, "Reclaim head  : %d\n", QhHw->ReclaimHead));\r
+  DEBUG ((EFI_D_INFO, "Max packet len: %d\n", QhHw->MaxPacketLen));\r
+  DEBUG ((EFI_D_INFO, "Ctrl EP       : %d\n", QhHw->CtrlEp));\r
+  DEBUG ((EFI_D_INFO, "Nak reload    : %d\n", QhHw->NakReload));\r
+\r
+  DEBUG ((EFI_D_INFO, "SMask         : %x\n", QhHw->SMask));\r
+  DEBUG ((EFI_D_INFO, "CMask         : %x\n", QhHw->CMask));\r
+  DEBUG ((EFI_D_INFO, "Hub address   : %d\n", QhHw->HubAddr));\r
+  DEBUG ((EFI_D_INFO, "Hub port      : %d\n", QhHw->PortNum));\r
+  DEBUG ((EFI_D_INFO, "Multiplier    : %d\n", QhHw->Multiplier));\r
+\r
+  DEBUG ((EFI_D_INFO, "Cur QTD       : %x\n", QhHw->CurQtd));\r
+\r
+  DEBUG ((EFI_D_INFO, "Next QTD      : %x\n", QhHw->NextQtd));\r
+  DEBUG ((EFI_D_INFO, "AltNext QTD   : %x\n", QhHw->AltQtd));\r
+  DEBUG ((EFI_D_INFO, "Status        : %x\n", QhHw->Status));\r
+\r
+  EhcDumpStatus (QhHw->Status);\r
 \r
   if (QhHw->Pid == QTD_PID_SETUP) {\r
-    EhciDebugPrint (Level, "PID           : Setup\n");\r
+    DEBUG ((EFI_D_INFO, "PID           : Setup\n"));\r
 \r
   } else if (QhHw->Pid == QTD_PID_INPUT) {\r
-    EhciDebugPrint (Level, "PID           : IN\n");\r
+    DEBUG ((EFI_D_INFO, "PID           : IN\n"));\r
 \r
   } else if (QhHw->Pid == QTD_PID_OUTPUT) {\r
-    EhciDebugPrint (Level, "PID           : OUT\n");\r
+    DEBUG ((EFI_D_INFO, "PID           : OUT\n"));\r
   }\r
 \r
-  EhciDebugPrint (Level, "Error Count   : %d\n", QhHw->ErrCnt);\r
-  EhciDebugPrint (Level, "Current Page  : %d\n", QhHw->CurPage);\r
-  EhciDebugPrint (Level, "IOC           : %d\n", QhHw->IOC);\r
-  EhciDebugPrint (Level, "Total Bytes   : %d\n", QhHw->TotalBytes);\r
-  EhciDebugPrint (Level, "Data Toggle   : %d\n", QhHw->DataToggle);\r
+  DEBUG ((EFI_D_INFO, "Error Count   : %d\n", QhHw->ErrCnt));\r
+  DEBUG ((EFI_D_INFO, "Current Page  : %d\n", QhHw->CurPage));\r
+  DEBUG ((EFI_D_INFO, "IOC           : %d\n", QhHw->IOC));\r
+  DEBUG ((EFI_D_INFO, "Total Bytes   : %d\n", QhHw->TotalBytes));\r
+  DEBUG ((EFI_D_INFO, "Data Toggle   : %d\n", QhHw->DataToggle));\r
 \r
   for (Index = 0; Index < 5; Index++) {\r
-    EhciDebugPrint (Level, "Page[%d]       : 0x%x\n", Index, QhHw->Page[Index]);\r
+    DEBUG ((EFI_D_INFO, "Page[%d]       : 0x%x\n", Index, QhHw->Page[Index]));\r
   }\r
 \r
-  EhciDebugPrint (Level, "\n");\r
+  DEBUG ((EFI_D_INFO, "\n"));\r
 \r
   EFI_LIST_FOR_EACH (Entry, &Qh->Qtds) {\r
     Qtd = EFI_LIST_CONTAINER (Entry, EHC_QTD, QtdList);\r
@@ -333,13 +236,11 @@ EhcDumpBuf (
 \r
   for (Index = 0; Index < Len; Index++) {\r
     if (Index % 16 == 0) {\r
-      EhciDebugPrint (EHC_DEBUG_BUF, "\n");\r
+      DEBUG ((EFI_D_INFO,"\n"));\r
     }\r
 \r
-    EhciDebugPrint (EHC_DEBUG_BUF, "%02x ", Buf[Index]);\r
+    DEBUG ((EFI_D_INFO, "%02x ", Buf[Index]));\r
   }\r
 \r
-  EhciDebugPrint (EHC_DEBUG_BUF, "\n");\r
+  DEBUG ((EFI_D_INFO, "\n"));\r
 }\r
-\r
-#endif\r
index 4df8e9e..3dd4074 100644 (file)
@@ -25,73 +25,6 @@ Revision History
 #define _EFI_EHCI_DEBUG_H_\r
 \r
 \r
-enum {\r
-  USB_DEBUG_FORCE_OUTPUT  = (UINTN)(1 << 0),\r
-\r
-  EHC_DEBUG_QH            = (UINTN)(1 << 8),\r
-  EHC_DEBUG_QTD           = (UINTN)(1 << 9),\r
-  EHC_DEBUG_BUF           = (UINTN)(1 << 10)\r
-};\r
-\r
-\r
-/**\r
-  EHCI's debug output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Level    The output level\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhciDebugPrint (\r
-  IN  UINTN               Level,\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-;\r
-\r
-\r
-/**\r
-  EHCI's debug output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhcDebug (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-;\r
-\r
-\r
-\r
-/**\r
-  EHCI's error output function. It determines whether\r
-  to output by the mask and level\r
-\r
-  @param  Format   The format parameters to the print\r
-  @param  ...      The variable length parameters after format\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-EhcError (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-;\r
-\r
-\r
-\r
 /**\r
   Dump the fields of a QTD\r
 \r
@@ -104,7 +37,7 @@ EhcError (
 VOID\r
 EhcDumpQtd (\r
   IN EHC_QTD              *Qtd,\r
-  IN UINT8                *Msg\r
+  IN CHAR8                *Msg\r
   )\r
 ;\r
 \r
@@ -123,7 +56,7 @@ EhcDumpQtd (
 VOID\r
 EhcDumpQh (\r
   IN EHC_QH               *Qh,\r
-  IN UINT8                *Msg,\r
+  IN CHAR8                *Msg,\r
   IN BOOLEAN              DumpBuf\r
   )\r
 ;\r
@@ -146,18 +79,4 @@ EhcDumpBuf (
   )\r
 ;\r
 \r
-#ifdef EFI_DEBUG\r
-  #define EHC_DEBUG(arg)                  EhcDebug    arg\r
-  #define EHC_ERROR(arg)                  EhcError    arg\r
-  #define EHC_DUMP_QH(arg)                EhcDumpQh   arg\r
-  #define EHC_DUMP_QTD(arg)               EhcDumpQtd  arg\r
-  #define EHC_DUMP_BUF(arg)               EhcDumpBuf  arg\r
-#else\r
-  #define EHC_DEBUG(arg)\r
-  #define EHC_ERROR(arg)\r
-  #define EHC_DUMP_QH(arg)\r
-  #define EHC_DUMP_QTD(arg)\r
-  #define EHC_DUMP_BUF(arg)\r
-#endif\r
-\r
 #endif\r
index 45b6df5..b40b011 100644 (file)
@@ -54,7 +54,7 @@ EhcReadCapRegister (
                              );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcReadCapRegister: Pci Io read error - %r at %d\n", Status, Offset));\r
+    DEBUG ((EFI_D_ERROR, "EhcReadCapRegister: Pci Io read error - %r at %d\n", Status, Offset));\r
     Data = 0xFFFF;\r
   }\r
 \r
@@ -92,7 +92,7 @@ EhcReadOpReg (
                              );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcReadOpReg: Pci Io Read error - %r at %d\n", Status, Offset));\r
+    DEBUG ((EFI_D_ERROR, "EhcReadOpReg: Pci Io Read error - %r at %d\n", Status, Offset));\r
     Data = 0xFFFF;\r
   }\r
 \r
@@ -131,7 +131,7 @@ EhcWriteOpReg (
                              );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcWriteOpReg: Pci Io Write error: %r at %d\n", Status, Offset));\r
+    DEBUG ((EFI_D_ERROR, "EhcWriteOpReg: Pci Io Write error: %r at %d\n", Status, Offset));\r
   }\r
 }\r
 \r
@@ -245,7 +245,7 @@ EhcClearLegacySupport (
   UINT32                    Value;\r
   UINT32                    TimeOut;\r
 \r
-  EHC_DEBUG (("EhcClearLegacySupport: called to clear legacy support\n"));\r
+  DEBUG ((EFI_D_INFO, "EhcClearLegacySupport: called to clear legacy support\n"));\r
 \r
   PciIo     = Ehc->PciIo;\r
   ExtendCap = (Ehc->HcCapParams >> 8) & 0xFF;\r
@@ -622,14 +622,14 @@ EhcInitHC (
   Status = EhcEnablePeriodSchd (Ehc, EHC_GENERIC_TIMEOUT);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcInitHC: failed to enable period schedule\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcInitHC: failed to enable period schedule\n"));\r
     return Status;\r
   }\r
 \r
   Status = EhcEnableAsyncSchd (Ehc, EHC_GENERIC_TIMEOUT);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcInitHC: failed to enable async schedule\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcInitHC: failed to enable async schedule\n"));\r
     return Status;\r
   }\r
 \r
index 349b5c2..60b095f 100644 (file)
@@ -345,7 +345,7 @@ EhcUnlinkQhFromAsync (
   Status = EhcSetAndWaitDoorBell (Ehc, EHC_GENERIC_TIMEOUT);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    EHC_ERROR (("EhcUnlinkQhFromAsync: Failed to synchronize with doorbell\n"));\r
+    DEBUG ((EFI_D_ERROR, "EhcUnlinkQhFromAsync: Failed to synchronize with doorbell\n"));\r
   }\r
 }\r
 \r
@@ -598,7 +598,7 @@ EhcCheckUrbResult (
       }\r
 \r
       if ((QtdHw->TotalBytes != 0) && (QtdHw->Pid == QTD_PID_INPUT)) {\r
-        EHC_DUMP_QH ((Urb->Qh, "Short packet read", FALSE));\r
+        EhcDumpQh (Urb->Qh, "Short packet read", FALSE);\r
 \r
         //\r
         // Short packet read condition. If it isn't a setup transfer,\r
@@ -607,13 +607,13 @@ EhcCheckUrbResult (
         // Status Stage of the setup transfer to get the finial result\r
         //\r
         if (QtdHw->AltNext == QTD_LINK (Ehc->ShortReadStop, FALSE)) {\r
-          EHC_DEBUG (("EhcCheckUrbResult: Short packet read, break\n"));\r
+          DEBUG ((EFI_D_INFO, "EhcCheckUrbResult: Short packet read, break\n"));\r
 \r
           Finished = TRUE;\r
           goto ON_EXIT;\r
         }\r
 \r
-        EHC_DEBUG (("EhcCheckUrbResult: Short packet read, continue\n"));\r
+        DEBUG ((EFI_D_INFO, "EhcCheckUrbResult: Short packet read, continue\n"));\r
       }\r
     }\r
   }\r
@@ -673,14 +673,14 @@ EhcExecTransfer (
   }\r
 \r
   if (!Finished) {\r
-    EHC_ERROR (("EhcExecTransfer: transfer not finished in %dms\n", TimeOut));\r
-    EHC_DUMP_QH ((Urb->Qh, NULL, FALSE));\r
+    DEBUG ((EFI_D_ERROR, "EhcExecTransfer: transfer not finished in %dms\n", TimeOut));\r
+    EhcDumpQh (Urb->Qh, NULL, FALSE);\r
 \r
     Status = EFI_TIMEOUT;\r
 \r
   } else if (Urb->Result != EFI_USB_NOERROR) {\r
-    EHC_ERROR (("EhcExecTransfer: transfer failed with %x\n", Urb->Result));\r
-    EHC_DUMP_QH ((Urb->Qh, NULL, FALSE));\r
+    DEBUG ((EFI_D_ERROR, "EhcExecTransfer: transfer failed with %x\n", Urb->Result));\r
+    EhcDumpQh (Urb->Qh, NULL, FALSE);\r
 \r
     Status = EFI_DEVICE_ERROR;\r
   }\r
@@ -781,14 +781,14 @@ EhcFlushAsyncIntMap (
 \r
 Routine Description:\r
 \r
-  Flush data from PCI controller specific address to mapped system \r
+  Flush data from PCI controller specific address to mapped system\r
   memory address.\r
 \r
 Arguments:\r
 \r
   Ehc         - The EHCI device\r
   Urb         - The URB to unmap\r
-  \r
+\r
 Returns:\r
 \r
   EFI_SUCCESS      - Success to flush data to mapped system memory\r
@@ -811,7 +811,7 @@ Returns:
   } else {\r
     MapOp = EfiPciIoOperationBusMasterRead;\r
   }\r
-  \r
+\r
   Status = PciIo->Unmap (PciIo, Urb->DataMap);\r
   if (EFI_ERROR (Status)) {\r
     goto ON_ERROR;\r
@@ -953,14 +953,14 @@ EhcMoniteAsyncRequests (
     }\r
 \r
     //\r
-    // Flush any PCI posted write transactions from a PCI host \r
+    // Flush any PCI posted write transactions from a PCI host\r
     // bridge to system memory.\r
     //\r
     Status = EhcFlushAsyncIntMap (Ehc, Urb);\r
     if (EFI_ERROR (Status)) {\r
-      EHC_ERROR (("EhcMoniteAsyncRequests: Fail to Flush AsyncInt Mapped Memeory\n"));\r
+      DEBUG ((EFI_D_ERROR, "EhcMoniteAsyncRequests: Fail to Flush AsyncInt Mapped Memeory\n"));\r
     }\r
-    \r
+\r
     //\r
     // Allocate a buffer then copy the transferred data for user.\r
     // If failed to allocate the buffer, update the URB for next\r
index c1b0f49..d038ebc 100644 (file)
@@ -24,9 +24,6 @@ Revision History
 #include "Ehci.h"\r
 \r
 \r
-UINTN mUsbHcDebugLevel = DEBUG_INFO;\r
-\r
-\r
 /**\r
   Allocate a block of memory to be used by the buffer pool\r
 \r
@@ -117,9 +114,6 @@ UsbHcAllocMemBlock (
   Block->Buf      = (UINT8 *) ((UINTN) MappedAddr);\r
   Block->Mapping  = Mapping;\r
 \r
-  DEBUG ((mUsbHcDebugLevel, "UsbHcAllocMemBlock: block %x created with buffer %x\n",\r
-                          Block, Block->Buf));\r
-\r
   return Block;\r
 \r
 FREE_BUFFER:\r
@@ -455,7 +449,7 @@ UsbHcAllocateMem (
   NewBlock = UsbHcAllocMemBlock (Pool, Pages);\r
 \r
   if (NewBlock == NULL) {\r
-    DEBUG ((mUsbHcDebugLevel, "UsbHcAllocateMem: failed to allocate block\n"));\r
+    DEBUG ((EFI_D_INFO, "UsbHcAllocateMem: failed to allocate block\n"));\r
     return NULL;\r
   }\r
 \r
@@ -539,8 +533,6 @@ UsbHcFreeMem (
   // Release the current memory block if it is empty and not the head\r
   //\r
   if ((Block != Head) && UsbHcIsMemBlockEmpty (Block)) {\r
-    DEBUG ((mUsbHcDebugLevel, "UsbHcFreeMem: block %x is empty, recycle\n", Block));\r
-\r
     UsbHcUnlinkMemBlock (Head, Block);\r
     UsbHcFreeMemBlock (Pool, Block);\r
   }\r
index 7f43517..d129af8 100644 (file)
@@ -320,7 +320,7 @@ Uhci2GetCapability (
 \r
   Uhc->RootPorts = *PortNumber;\r
 \r
-  UHCI_DEBUG (("Uhci2GetCapability: %d ports\n", Uhc->RootPorts));\r
+  DEBUG ((EFI_D_INFO, "Uhci2GetCapability: %d ports\n", Uhc->RootPorts));\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -376,7 +376,7 @@ Uhci2GetRootHubPortStatus (
   }\r
 \r
   if (PortSC & USBPORTSC_SUSP) {\r
-    UHCI_DEBUG (("Uhci2GetRootHubPortStatus: port %d is suspended\n", PortNumber));\r
+    DEBUG ((EFI_D_INFO, "Uhci2GetRootHubPortStatus: port %d is suspended\n", PortNumber));\r
     PortStatus->PortStatus |= USB_PORT_STAT_SUSPEND;\r
   }\r
 \r
index d47314f..117bef2 100644 (file)
@@ -16,8 +16,6 @@ Module Name:
 Abstract:\r
 \r
   This file provides the information dump support for Uhci when in debug mode.\r
-  You can dynamically adjust the debug level by changing variable gEHCDebugLevel\r
-  and gEHCErrorLevel.\r
 \r
 Revision History\r
 \r
@@ -27,87 +25,6 @@ Revision History
 #include "Uhci.h"\r
 #include "UhciDebug.h"\r
 \r
-#ifdef EFI_DEBUG\r
-\r
-UINTN mUhciDebugMask = USB_DEBUG_FORCE_OUTPUT;\r
-\r
-\r
-/**\r
-  Debug debug print interface for UHCI\r
-\r
-  @param  Format  String to use for the print, followed by print arguments\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-UhciDebug (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-  DebugVPrint (DEBUG_INFO, Format, Marker);\r
-  VA_END (Marker);\r
-}\r
-\r
-\r
-/**\r
-  Debug error print interface for UHCI\r
-\r
-  @param  Format  String to use for the print, followed by print arguments\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-UhciError (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-  DebugVPrint (DEBUG_ERROR, Format, Marker);\r
-  VA_END (Marker);\r
-}\r
-\r
-\r
-\r
-/**\r
-  Debug print interface for UHCI\r
-\r
-  @param  Level   Level to control debug print\r
-  @param  Format  String to use for the print, followed by print arguments\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-UhciDebugPrint (\r
-  IN  UINTN               Level,\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-{\r
-  VA_LIST                 Marker;\r
-\r
-  VA_START (Marker, Format);\r
-\r
-  if (Level & mUhciDebugMask) {\r
-    if (mUhciDebugMask & USB_DEBUG_FORCE_OUTPUT) {\r
-      DebugVPrint (DEBUG_ERROR, Format, Marker);\r
-    } else {\r
-      DebugVPrint (DEBUG_INFO, Format, Marker);\r
-    }\r
-  }\r
-\r
-  VA_END (Marker);\r
-}\r
-\r
 \r
 /**\r
   Dump the content of QH structure\r
@@ -122,16 +39,12 @@ UhciDumpQh (
   IN UHCI_QH_SW    *QhSw\r
   )\r
 {\r
-  UINTN                   Level;\r
-\r
-  Level = UHCI_DEBUG_QH;\r
-\r
-  UhciDebugPrint (Level, "&QhSw @ 0x%x\n", QhSw);\r
-  UhciDebugPrint (Level, "QhSw.NextQh    - 0x%x\n", QhSw->NextQh);\r
-  UhciDebugPrint (Level, "QhSw.TDs       - 0x%x\n", QhSw->TDs);\r
-  UhciDebugPrint (Level, "QhSw.QhHw:\n");\r
-  UhciDebugPrint (Level, " Horizon  Link - %x\n", QhSw->QhHw.HorizonLink);\r
-  UhciDebugPrint (Level, " Vertical Link - %x\n\n", QhSw->QhHw.VerticalLink);\r
+  DEBUG ((EFI_D_INFO, "&QhSw @ 0x%x\n", QhSw));\r
+  DEBUG ((EFI_D_INFO, "QhSw.NextQh    - 0x%x\n", QhSw->NextQh));\r
+  DEBUG ((EFI_D_INFO, "QhSw.TDs       - 0x%x\n", QhSw->TDs));\r
+  DEBUG ((EFI_D_INFO, "QhSw.QhHw:\n"));\r
+  DEBUG ((EFI_D_INFO, " Horizon  Link - %x\n", QhSw->QhHw.HorizonLink));\r
+  DEBUG ((EFI_D_INFO, " Vertical Link - %x\n\n", QhSw->QhHw.VerticalLink));\r
 }\r
 \r
 \r
@@ -150,34 +63,31 @@ UhciDumpTds (
   )\r
 {\r
   UHCI_TD_SW              *CurTdSw;\r
-  UINTN                   Level;\r
 \r
-  Level   = UHCI_DEBUG_TD;\r
   CurTdSw = TdSw;\r
 \r
   while (CurTdSw != NULL) {\r
-    UhciDebugPrint (Level, "TdSw @ 0x%x\n",           CurTdSw);\r
-    UhciDebugPrint (Level, "TdSw.NextTd   - 0x%x\n",  CurTdSw->NextTd);\r
-    UhciDebugPrint (Level, "TdSw.DataLen  - %d\n",    CurTdSw->DataLen);\r
-    UhciDebugPrint (Level, "TdSw.Data     - 0x%x\n",  CurTdSw->Data);\r
-    UhciDebugPrint (Level, "TdHw:\n");\r
-    UhciDebugPrint (Level, " NextLink     - 0x%x\n",  CurTdSw->TdHw.NextLink);\r
-    UhciDebugPrint (Level, " ActualLen    - %d\n",    CurTdSw->TdHw.ActualLen);\r
-    UhciDebugPrint (Level, " Status       - 0x%x\n",  CurTdSw->TdHw.Status);\r
-    UhciDebugPrint (Level, " IOC          - %d\n",    CurTdSw->TdHw.IntOnCpl);\r
-    UhciDebugPrint (Level, " IsIsoCh      - %d\n",    CurTdSw->TdHw.IsIsoch);\r
-    UhciDebugPrint (Level, " LowSpeed     - %d\n",    CurTdSw->TdHw.LowSpeed);\r
-    UhciDebugPrint (Level, " ErrorCount   - %d\n",    CurTdSw->TdHw.ErrorCount);\r
-    UhciDebugPrint (Level, " ShortPacket  - %d\n",    CurTdSw->TdHw.ShortPacket);\r
-    UhciDebugPrint (Level, " PidCode      - 0x%x\n",  CurTdSw->TdHw.PidCode);\r
-    UhciDebugPrint (Level, " DevAddr      - %d\n",    CurTdSw->TdHw.DeviceAddr);\r
-    UhciDebugPrint (Level, " EndPoint     - %d\n",    CurTdSw->TdHw.EndPoint);\r
-    UhciDebugPrint (Level, " DataToggle   - %d\n",    CurTdSw->TdHw.DataToggle);\r
-    UhciDebugPrint (Level, " MaxPacketLen - %d\n",    CurTdSw->TdHw.MaxPacketLen);\r
-    UhciDebugPrint (Level, " DataBuffer   - 0x%x\n\n",CurTdSw->TdHw.DataBuffer);\r
+    DEBUG ((EFI_D_INFO, "TdSw @ 0x%x\n",           CurTdSw));\r
+    DEBUG ((EFI_D_INFO, "TdSw.NextTd   - 0x%x\n",  CurTdSw->NextTd));\r
+    DEBUG ((EFI_D_INFO, "TdSw.DataLen  - %d\n",    CurTdSw->DataLen));\r
+    DEBUG ((EFI_D_INFO, "TdSw.Data     - 0x%x\n",  CurTdSw->Data));\r
+    DEBUG ((EFI_D_INFO, "TdHw:\n"));\r
+    DEBUG ((EFI_D_INFO, " NextLink     - 0x%x\n",  CurTdSw->TdHw.NextLink));\r
+    DEBUG ((EFI_D_INFO, " ActualLen    - %d\n",    CurTdSw->TdHw.ActualLen));\r
+    DEBUG ((EFI_D_INFO, " Status       - 0x%x\n",  CurTdSw->TdHw.Status));\r
+    DEBUG ((EFI_D_INFO, " IOC          - %d\n",    CurTdSw->TdHw.IntOnCpl));\r
+    DEBUG ((EFI_D_INFO, " IsIsoCh      - %d\n",    CurTdSw->TdHw.IsIsoch));\r
+    DEBUG ((EFI_D_INFO, " LowSpeed     - %d\n",    CurTdSw->TdHw.LowSpeed));\r
+    DEBUG ((EFI_D_INFO, " ErrorCount   - %d\n",    CurTdSw->TdHw.ErrorCount));\r
+    DEBUG ((EFI_D_INFO, " ShortPacket  - %d\n",    CurTdSw->TdHw.ShortPacket));\r
+    DEBUG ((EFI_D_INFO, " PidCode      - 0x%x\n",  CurTdSw->TdHw.PidCode));\r
+    DEBUG ((EFI_D_INFO, " DevAddr      - %d\n",    CurTdSw->TdHw.DeviceAddr));\r
+    DEBUG ((EFI_D_INFO, " EndPoint     - %d\n",    CurTdSw->TdHw.EndPoint));\r
+    DEBUG ((EFI_D_INFO, " DataToggle   - %d\n",    CurTdSw->TdHw.DataToggle));\r
+    DEBUG ((EFI_D_INFO, " MaxPacketLen - %d\n",    CurTdSw->TdHw.MaxPacketLen));\r
+    DEBUG ((EFI_D_INFO, " DataBuffer   - 0x%x\n\n",CurTdSw->TdHw.DataBuffer));\r
 \r
     CurTdSw = CurTdSw->NextTd;\r
   }\r
 }\r
 \r
-#endif\r
index d71791a..5628e39 100644 (file)
@@ -25,69 +25,6 @@ Revision History
 #ifndef _EFI_UHCI_DEBUG_H_\r
 #define _EFI_UHCI_DEBUG_H_\r
 \r
-//\r
-// DEBUG support\r
-//\r
-#define USB_DEBUG_FORCE_OUTPUT  (UINTN) (1 << 0)\r
-#define UHCI_DEBUG_QH           (UINTN) (1 << 2)\r
-#define UHCI_DEBUG_TD           (UINTN) (1 << 3)\r
-\r
-VOID\r
-UhciDebugPrint (\r
-  IN  UINTN               Level,\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-\r
-  Debug print interface for UHCI\r
-\r
-Arguments:\r
-\r
-  Level   - Level to control debug print\r
-  Format  - String to use for the print, followed by print arguments\r
-\r
-Returns:\r
-\r
-  None\r
-\r
---*/\r
-;\r
-\r
-\r
-/**\r
-  Debug print interface for UHCI\r
-\r
-  @param  Format  String to use for the print, followed by print arguments\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-UhciDebug (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-;\r
-\r
-\r
-/**\r
-  Debug error print interface for UHCI\r
-\r
-  @param  Format  String to use for the print, followed by print arguments\r
-\r
-  @return None\r
-\r
-**/\r
-VOID\r
-UhciError (\r
-  IN  CHAR8               *Format,\r
-  ...\r
-  )\r
-;\r
-\r
 \r
 /**\r
   Dump the content of QH structure\r
@@ -118,17 +55,4 @@ UhciDumpTds (
   )\r
 ;\r
 \r
-\r
-#ifdef EFI_DEBUG\r
-  #define UHCI_DEBUG(arg)             UhciDebug arg\r
-  #define UHCI_ERROR(arg)             UhciError arg\r
-  #define UHCI_DUMP_TDS(arg)          UhciDumpTds arg\r
-  #define UHCI_DUMP_QH(arg)           UhciDumpQh arg\r
-#else\r
-  #define UHCI_DEBUG(arg)\r
-  #define UHCI_ERROR(arg)\r
-  #define UHCI_DUMP_TDS(arg)\r
-  #define UHCI_DUMP_QH(arg)\r
-#endif\r
-\r
 #endif\r
index 9c632a6..0742f20 100644 (file)
@@ -53,7 +53,7 @@ UhciReadReg (
                       );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    UHCI_ERROR (("UhciReadReg: PciIo Io.Read error: %r at offset %d\n", Status, Offset));\r
+    DEBUG ((EFI_D_ERROR, "UhciReadReg: PciIo Io.Read error: %r at offset %d\n", Status, Offset));\r
 \r
     Data = 0xFFFF;\r
   }\r
@@ -91,7 +91,7 @@ UhciWriteReg (
                       );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    UHCI_ERROR (("UhciWriteReg: PciIo Io.Write error: %r at offset %d\n", Status, Offset));\r
+    DEBUG ((EFI_D_ERROR, "UhciWriteReg: PciIo Io.Write error: %r at offset %d\n", Status, Offset));\r
   }\r
 }\r
 \r
@@ -167,7 +167,7 @@ UhciAckAllInterrupt (
   // is a temporary error status.\r
   //\r
   if (!UhciIsHcWorking (Uhc->PciIo)) {\r
-    UHCI_ERROR (("UhciAckAllInterrupt: re-enable the UHCI from system error\n"));\r
+    DEBUG ((EFI_D_ERROR, "UhciAckAllInterrupt: re-enable the UHCI from system error\n"));\r
     Uhc->Usb2Hc.SetState (&Uhc->Usb2Hc, EfiUsbHcStateOperational);\r
   }\r
 }\r
@@ -232,7 +232,7 @@ UhciIsHcWorking (
   UsbSts = UhciReadReg (PciIo, USBSTS_OFFSET);\r
 \r
   if (UsbSts & (USBSTS_HCPE | USBSTS_HSE | USBSTS_HCH)) {\r
-    UHCI_ERROR (("UhciIsHcWorking: current USB state is %x\n", UsbSts));\r
+    DEBUG ((EFI_D_ERROR, "UhciIsHcWorking: current USB state is %x\n", UsbSts));\r
     return FALSE;\r
   }\r
 \r
@@ -271,7 +271,7 @@ UhciSetFrameListBaseAddr (
                        );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    UHCI_ERROR (("UhciSetFrameListBaseAddr: PciIo Io.Write error: %r\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UhciSetFrameListBaseAddr: PciIo Io.Write error: %r\n", Status));\r
   }\r
 }\r
 \r
index 401d32e..3c2b7d4 100644 (file)
@@ -121,11 +121,7 @@ UhciInitFrameList (
   // in supporting the full speed bandwidth reclamation in the previous\r
   // mentioned form. Most new platforms don't suffer it.\r
   //\r
-#ifdef UHCI_NO_BW_RECLAMATION\r
-  Uhc->BulkQh->QhHw.HorizonLink     = QH_HLINK (NULL, TRUE);\r
-#else\r
   Uhc->BulkQh->QhHw.HorizonLink     = QH_HLINK (Uhc->BulkQh, FALSE);\r
-#endif\r
 \r
   Uhc->BulkQh->NextQh               = NULL;\r
 \r
@@ -515,7 +511,7 @@ UhciCheckTdStatus (
       // terminate the transfer\r
       //\r
       if (!IsLow && (TdHw->ShortPacket == 1) && (Len < Td->DataLen)) {\r
-        UHCI_DEBUG (("UhciCheckTdStatus: short packet read occured\n"));\r
+        DEBUG ((EFI_D_INFO, "UhciCheckTdStatus: short packet read occured\n"));\r
 \r
         Finished = TRUE;\r
         goto ON_EXIT;\r
@@ -576,7 +572,7 @@ UhciExecuteTransfer (
   Finished = FALSE;\r
   Status   = EFI_SUCCESS;\r
   Delay    = (TimeOut * UHC_1_MILLISECOND / UHC_SYNC_POLL_INTERVAL) + 1;\r
-  \r
+\r
   for (Index = 0; Index < Delay; Index++) {\r
     Finished = UhciCheckTdStatus (Uhc, Td, IsLow, QhResult);\r
 \r
@@ -591,16 +587,16 @@ UhciExecuteTransfer (
   }\r
 \r
   if (!Finished) {\r
-    UHCI_ERROR (("UhciExecuteTransfer: execution not finished for %dms\n", TimeOut));\r
-    UHCI_DUMP_QH  ((Qh));\r
-    UHCI_DUMP_TDS ((Td));\r
+    DEBUG ((EFI_D_ERROR, "UhciExecuteTransfer: execution not finished for %dms\n", TimeOut));\r
+    UhciDumpQh (Qh);\r
+    UhciDumpTds (Td);\r
 \r
     Status = EFI_TIMEOUT;\r
 \r
   } else if (QhResult->Result != EFI_USB_NOERROR) {\r
-    UHCI_ERROR (("UhciExecuteTransfer: execution failed with result %x\n", QhResult->Result));\r
-    UHCI_DUMP_QH  ((Qh));\r
-    UHCI_DUMP_TDS ((Td));\r
+    DEBUG ((EFI_D_ERROR, "UhciExecuteTransfer: execution failed with result %x\n", QhResult->Result));\r
+    UhciDumpQh (Qh);\r
+    UhciDumpTds (Td);\r
 \r
     Status = EFI_DEVICE_ERROR;\r
   }\r
index 5e02a0d..c7881ec 100644 (file)
@@ -23,9 +23,6 @@ Revision History
 #include "Uhci.h"\r
 \r
 \r
-UINTN mUsbHcDebugLevel = DEBUG_INFO;\r
-\r
-\r
 /**\r
   Allocate a block of memory to be used by the buffer pool\r
 \r
@@ -116,9 +113,6 @@ UsbHcAllocMemBlock (
   Block->Buf      = (UINT8 *) ((UINTN) MappedAddr);\r
   Block->Mapping  = Mapping;\r
 \r
-  DEBUG ((mUsbHcDebugLevel, "UsbHcAllocMemBlock: block %x created with buffer %x\n",\r
-                          Block, Block->Buf));\r
-\r
   return Block;\r
 \r
 FREE_BUFFER:\r
@@ -454,7 +448,7 @@ UsbHcAllocateMem (
   NewBlock = UsbHcAllocMemBlock (Pool, Pages);\r
 \r
   if (NewBlock == NULL) {\r
-    DEBUG ((mUsbHcDebugLevel, "UsbHcAllocateMem: failed to allocate block\n"));\r
+    DEBUG ((EFI_D_INFO, "UsbHcAllocateMem: failed to allocate block\n"));\r
     return NULL;\r
   }\r
 \r
@@ -538,8 +532,6 @@ UsbHcFreeMem (
   // Release the current memory block if it is empty and not the head\r
   //\r
   if ((Block != Head) && UsbHcIsMemBlockEmpty (Block)) {\r
-    DEBUG ((mUsbHcDebugLevel, "UsbHcFreeMem: block %x is empty, recycle\n", Block));\r
-\r
     UsbHcUnlinkMemBlock (Head, Block);\r
     UsbHcFreeMemBlock (Pool, Block);\r
   }\r
index 3e7c700..fe3f8b9 100644 (file)
@@ -62,7 +62,7 @@ enum {
   USB_MASS_STORE_RBC      = 0x01, // Reduced Block Commands\r
   USB_MASS_STORE_8020I    = 0x02, // SFF-8020i, typically a CD/DVD device\r
   USB_MASS_STORE_QIC      = 0x03, // Typically a tape device\r
-  USB_MASS_STORE_UFI      = 0x04, // Typically a floopy disk driver device\r
+  USB_MASS_STORE_UFI      = 0x04, // Typically a floppy disk driver device\r
   USB_MASS_STORE_8070I    = 0x05, // SFF-8070i, typically a floppy disk driver device.\r
   USB_MASS_STORE_SCSI     = 0x06, // SCSI transparent command set\r
 \r
@@ -138,6 +138,4 @@ UsbClearEndpointStall (
   IN UINT8                    EndpointAddress\r
   );\r
 \r
-extern UINTN mUsbMscInfo;\r
-extern UINTN mUsbMscError;\r
 #endif\r
index 1299059..8dd7ada 100644 (file)
@@ -167,7 +167,7 @@ UsbBootRequestSense (
                         &CmdResult\r
                         );\r
   if (EFI_ERROR (Status) || CmdResult != USB_MASS_CMD_SUCCESS) {\r
-    DEBUG ((mUsbMscError, "UsbBootRequestSense: (%r) CmdResult=0x%x\n", Status, CmdResult));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootRequestSense: (%r) CmdResult=0x%x\n", Status, CmdResult));\r
     return Status;\r
   }\r
 \r
@@ -225,7 +225,7 @@ UsbBootRequestSense (
     break;\r
   }\r
 \r
-  DEBUG ((mUsbMscInfo, "UsbBootRequestSense: (%r) with sense key %x/%x/%x\n",\r
+  DEBUG ((EFI_D_INFO, "UsbBootRequestSense: (%r) with sense key %x/%x/%x\n",\r
           Status,\r
           USB_BOOT_SENSE_KEY (SenseData.SenseKey),\r
           SenseData.ASC,\r
@@ -291,7 +291,7 @@ UsbBootExecCmd (
     return EFI_SUCCESS;\r
   }\r
 \r
-  DEBUG ((mUsbMscInfo, "UsbBootExecCmd: Fail to Exec 0x%x Cmd /w %r\n",\r
+  DEBUG ((EFI_D_INFO, "UsbBootExecCmd: Fail to Exec 0x%x Cmd /w %r\n",\r
           *(UINT8 *)Cmd ,Status));\r
 \r
   return UsbBootRequestSense (UsbMass);\r
@@ -349,7 +349,7 @@ UsbBootExecCmdWithRetry (
                DataDir,\r
                Data,\r
                DataLen,\r
-               Timeout * (Index + 1)\r
+               Timeout\r
                );\r
     if (Status == EFI_SUCCESS ||\r
         Status == EFI_MEDIA_CHANGED) {\r
@@ -513,7 +513,7 @@ UsbBootReadCapacity (
     return EFI_NOT_READY;\r
   }\r
 \r
-  DEBUG ((mUsbMscInfo, "UsbBootReadCapacity Success LBA=%ld BlockSize=%d\n",\r
+  DEBUG ((EFI_D_INFO, "UsbBootReadCapacity Success LBA=%ld BlockSize=%d\n",\r
           Media->LastBlock, Media->BlockSize));\r
 \r
   return EFI_SUCCESS;\r
@@ -603,7 +603,7 @@ UsbBootGetParams (
 \r
   Status = UsbBootInquiry (UsbMass);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "UsbBootGetParams: UsbBootInquiry (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootGetParams: UsbBootInquiry (%r)\n", Status));\r
     return Status;\r
   }\r
 \r
@@ -670,7 +670,7 @@ UsbBootDetectMedia (
 \r
   Status = UsbBootIsUnitReady (UsbMass);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "UsbBootDetectMedia: UsbBootIsUnitReady (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootDetectMedia: UsbBootIsUnitReady (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -687,7 +687,7 @@ UsbBootDetectMedia (
 \r
   Status = UsbBootReadCapacity (UsbMass);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "UsbBootDetectMedia: UsbBootReadCapacity (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootDetectMedia: UsbBootReadCapacity (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -709,7 +709,7 @@ ON_ERROR:
       (Media->LastBlock != OldMedia.LastBlock)) {\r
 \r
     OldTpl = UsbGetCurrentTpl ();\r
-    DEBUG ((mUsbMscError, "UsbBootDetectMedia: TPL before reinstall BlockIoProtocol is %d\n", OldTpl));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootDetectMedia: TPL before reinstall BlockIoProtocol is %d\n", OldTpl));\r
 \r
     gBS->RestoreTPL (TPL_CALLBACK);\r
 \r
@@ -720,7 +720,7 @@ ON_ERROR:
            &UsbMass->BlockIo\r
            );\r
 \r
-    DEBUG ((mUsbMscError, "UsbBootDetectMedia: TPL after reinstall is %d\n", UsbGetCurrentTpl()));\r
+    DEBUG ((EFI_D_ERROR, "UsbBootDetectMedia: TPL after reinstall is %d\n", UsbGetCurrentTpl()));\r
     ASSERT (UsbGetCurrentTpl () == TPL_CALLBACK);\r
 \r
     gBS->RaiseTPL (OldTpl);\r
index 3f4d0fc..3277fa9 100644 (file)
@@ -25,9 +25,6 @@ Revision History
 #include "UsbMass.h"\r
 #include "UsbMassBot.h"\r
 \r
-UINTN mUsbBotInfo  = DEBUG_INFO;\r
-UINTN mUsbBotError = DEBUG_ERROR;\r
-\r
 STATIC\r
 EFI_STATUS\r
 UsbBotResetDevice (\r
@@ -83,7 +80,7 @@ UsbBotInit (
   Status = UsbIo->UsbGetInterfaceDescriptor (UsbIo, &UsbBot->Interface);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbBotError, "UsbBotInit: Get invalid BOT interface (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotInit: Get invalid BOT interface (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -120,7 +117,7 @@ UsbBotInit (
   }\r
 \r
   if ((UsbBot->BulkInEndpoint == NULL) || (UsbBot->BulkOutEndpoint == NULL)) {\r
-    DEBUG ((mUsbBotError, "UsbBotInit: In/Out Endpoint invalid\n"));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotInit: In/Out Endpoint invalid\n"));\r
     Status = EFI_UNSUPPORTED;\r
     goto ON_ERROR;\r
   }\r
@@ -277,9 +274,9 @@ UsbBotDataTransfer (
                             &Result\r
                             );\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbBotError, "UsbBotDataTransfer: (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotDataTransfer: (%r)\n", Status));\r
     if (USB_IS_ERROR (Result, EFI_USB_ERR_STALL)) {\r
-      DEBUG ((mUsbBotError, "UsbBotDataTransfer: DataIn Stall\n"));\r
+      DEBUG ((EFI_D_ERROR, "UsbBotDataTransfer: DataIn Stall\n"));\r
       UsbClearEndpointStall (UsbBot->UsbIo, Endpoint->EndpointAddress);\r
     } else if (USB_IS_ERROR (Result, EFI_USB_ERR_NAK)) {\r
       Status = EFI_NOT_READY;\r
@@ -323,7 +320,7 @@ UsbBotGetStatus (
   EFI_USB_IO_PROTOCOL       *UsbIo;\r
   UINT32                    Index;\r
   UINTN                     Timeout;\r
-  \r
+\r
   *CmdStatus = USB_BOT_COMMAND_ERROR;\r
   Status     = EFI_DEVICE_ERROR;\r
   Endpoint   = UsbBot->BulkInEndpoint->EndpointAddress;\r
@@ -346,9 +343,9 @@ UsbBotGetStatus (
                       &Result\r
                       );\r
     if (EFI_ERROR(Status)) {\r
-      DEBUG ((mUsbBotError, "UsbBotGetStatus (%r)\n", Status));\r
+      DEBUG ((EFI_D_ERROR, "UsbBotGetStatus (%r)\n", Status));\r
       if (USB_IS_ERROR (Result, EFI_USB_ERR_STALL)) {\r
-        DEBUG ((mUsbBotError, "UsbBotGetStatus: DataIn Stall\n"));\r
+        DEBUG ((EFI_D_ERROR, "UsbBotGetStatus: DataIn Stall\n"));\r
         UsbClearEndpointStall (UsbIo, Endpoint);\r
       }\r
       continue;\r
@@ -358,13 +355,13 @@ UsbBotGetStatus (
       //\r
       // Invalid Csw need perform reset recovery\r
       //\r
-      DEBUG ((mUsbBotError, "UsbBotGetStatus: Device return a invalid signature\n"));\r
+      DEBUG ((EFI_D_ERROR, "UsbBotGetStatus: Device return a invalid signature\n"));\r
       Status = UsbBotResetDevice (UsbBot, FALSE);\r
     } else if (Csw.CmdStatus == USB_BOT_COMMAND_ERROR) {\r
       //\r
       // Respond phase error need perform reset recovery\r
       //\r
-      DEBUG ((mUsbBotError, "UsbBotGetStatus: Device return a phase error\n"));\r
+      DEBUG ((EFI_D_ERROR, "UsbBotGetStatus: Device return a phase error\n"));\r
       Status = UsbBotResetDevice (UsbBot, FALSE);\r
     } else {\r
 \r
@@ -426,7 +423,7 @@ UsbBotExecCommand (
   //\r
   Status = UsbBotSendCommand (UsbBot, Cmd, CmdLen, DataDir, DataLen);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbBotError, "UsbBotExecCommand: UsbBotSendCommand (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotExecCommand: UsbBotSendCommand (%r)\n", Status));\r
     return Status;\r
   }\r
 \r
@@ -443,7 +440,7 @@ UsbBotExecCommand (
   //\r
   Status = UsbBotGetStatus (UsbBot, DataLen, &Result);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbBotError, "UsbBotExecCommand: UsbBotGetStatus (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotExecCommand: UsbBotGetStatus (%r)\n", Status));\r
     return Status;\r
   }\r
 \r
@@ -512,7 +509,7 @@ UsbBotResetDevice (
                             );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbBotError, "UsbBotResetDevice: (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbBotResetDevice: (%r)\n", Status));\r
     return Status;\r
   }\r
 \r
index 0162350..a46a9a3 100644 (file)
@@ -27,9 +27,6 @@ Revision History
 #include "UsbMass.h"\r
 #include "UsbMassCbi.h"\r
 \r
-UINTN mUsbCbiInfo  = DEBUG_INFO;\r
-UINTN mUsbCbiError = DEBUG_ERROR;\r
-\r
 STATIC\r
 EFI_STATUS\r
 UsbCbiResetDevice (\r
@@ -450,7 +447,7 @@ UsbCbiExecCommand (
   //\r
   Status = UsbCbiSendCommand (UsbCbi, Cmd, CmdLen, Timeout);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbCbiError, "UsbCbiExecCommand: UsbCbiSendCommand (%r)\n",Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbCbiExecCommand: UsbCbiSendCommand (%r)\n",Status));\r
     return Status;\r
   }\r
 \r
@@ -462,7 +459,7 @@ UsbCbiExecCommand (
 \r
   Status   = UsbCbiDataTransfer (UsbCbi, DataDir, Data, &TransLen, Timeout);\r
   if (UsbCbi->InterruptEndpoint == NULL) {\r
-    DEBUG ((mUsbCbiError, "UsbCbiExecCommand: UsbCbiDataTransfer (%r)\n",Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbCbiExecCommand: UsbCbiDataTransfer (%r)\n",Status));\r
     return Status;\r
   }\r
 \r
@@ -471,7 +468,7 @@ UsbCbiExecCommand (
   //\r
   Status = UsbCbiGetStatus (UsbCbi, Timeout, &Result);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbCbiError, "UsbCbiExecCommand: UsbCbiGetStatus (%r)\n",Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbCbiExecCommand: UsbCbiGetStatus (%r)\n",Status));\r
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
index f077e2a..71ccfa9 100644 (file)
@@ -40,10 +40,6 @@ USB_MASS_TRANSPORT *mUsbMassTransport[] = {
   NULL\r
 };\r
 \r
-UINTN mUsbMscInfo  = DEBUG_INFO;\r
-UINTN mUsbMscError = DEBUG_ERROR;\r
-\r
-\r
 /**\r
   Retrieve the media parameters such as disk gemotric for the\r
   device's BLOCK IO protocol.\r
@@ -84,6 +80,7 @@ UsbMassInitMedia (
   Media->ReadOnly         = FALSE;\r
   Media->WriteCaching     = FALSE;\r
   Media->IoAlign          = 0;\r
+  Media->MediaId          = 1;\r
 \r
   //\r
   // Some device may spend several seconds before it is ready.\r
@@ -102,7 +99,7 @@ UsbMassInitMedia (
 \r
     Status = UsbBootIsUnitReady (UsbMass);\r
     if (EFI_ERROR (Status)) {\r
-      gBS->Stall (USB_BOOT_RETRY_UNIT_READY_STALL * (Index + 1)); \r
+      gBS->Stall (USB_BOOT_RETRY_UNIT_READY_STALL * (Index + 1));\r
     }\r
 \r
   }\r
@@ -122,6 +119,7 @@ UsbMassInitMedia (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 UsbMassReset (\r
   IN EFI_BLOCK_IO_PROTOCOL    *This,\r
   IN BOOLEAN                  ExtendedVerification\r
@@ -162,6 +160,7 @@ UsbMassReset (
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 UsbMassReadBlocks (\r
   IN EFI_BLOCK_IO_PROTOCOL    *This,\r
   IN UINT32                   MediaId,\r
@@ -175,7 +174,7 @@ UsbMassReadBlocks (
   EFI_STATUS          Status;\r
   EFI_TPL             OldTpl;\r
   UINTN               TotalBlock;\r
-  \r
+\r
   OldTpl  = gBS->RaiseTPL (USB_MASS_TPL);\r
   UsbMass = USB_MASS_DEVICE_FROM_BLOCKIO (This);\r
   Media   = &UsbMass->BlockIoMedia;\r
@@ -187,21 +186,21 @@ UsbMassReadBlocks (
     Status = EFI_INVALID_PARAMETER;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   //\r
   // If it is a removable media, such as CD-Rom or Usb-Floppy,\r
-  // need to detect the media before each rw. While some of \r
+  // need to detect the media before each rw. While some of\r
   // Usb-Flash is marked as removable media.\r
-  // \r
-  // \r
+  //\r
+  //\r
   if (Media->RemovableMedia == TRUE) {\r
     Status = UsbBootDetectMedia (UsbMass);\r
     if (EFI_ERROR (Status)) {\r
-      DEBUG ((mUsbMscError, "UsbMassReadBlocks: UsbBootDetectMedia (%r)\n", Status));\r
+      DEBUG ((EFI_D_ERROR, "UsbMassReadBlocks: UsbBootDetectMedia (%r)\n", Status));\r
       goto ON_EXIT;\r
-    } \r
+    }\r
   }\r
-  \r
+\r
   //\r
   // Make sure BlockSize and LBA is consistent with BufferSize\r
   //\r
@@ -213,13 +212,23 @@ UsbMassReadBlocks (
   TotalBlock = BufferSize / Media->BlockSize;\r
 \r
   if (Lba + TotalBlock - 1 > Media->LastBlock) {\r
-    Status = EFI_BAD_BUFFER_SIZE;\r
+    Status = EFI_INVALID_PARAMETER;\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  if (!(Media->MediaPresent)) {\r
+    Status = EFI_NO_MEDIA;\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  if (MediaId != Media->MediaId) {\r
+    Status = EFI_MEDIA_CHANGED;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   Status = UsbBootReadBlocks (UsbMass, (UINT32) Lba, TotalBlock, Buffer);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "UsbMassReadBlocks: UsbBootReadBlocks (%r) -> Reset\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbMassReadBlocks: UsbBootReadBlocks (%r) -> Reset\n", Status));\r
     UsbMassReset (This, TRUE);\r
   }\r
 \r
@@ -249,6 +258,7 @@ ON_EXIT:
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 UsbMassWriteBlocks (\r
   IN EFI_BLOCK_IO_PROTOCOL    *This,\r
   IN UINT32                   MediaId,\r
@@ -274,21 +284,21 @@ UsbMassWriteBlocks (
     Status = EFI_INVALID_PARAMETER;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
   //\r
   // If it is a removable media, such as CD-Rom or Usb-Floppy,\r
-  // need to detect the media before each rw. While some of \r
+  // need to detect the media before each rw. While some of\r
   // Usb-Flash is marked as removable media.\r
-  // \r
-  // \r
+  //\r
+  //\r
   if (Media->RemovableMedia == TRUE) {\r
     Status = UsbBootDetectMedia (UsbMass);\r
     if (EFI_ERROR (Status)) {\r
-      DEBUG ((mUsbMscError, "UsbMassWriteBlocks: UsbBootDetectMedia (%r)\n", Status));\r
+      DEBUG ((EFI_D_ERROR, "UsbMassWriteBlocks: UsbBootDetectMedia (%r)\n", Status));\r
       goto ON_EXIT;\r
-    } \r
+    }\r
   }\r
-  \r
+\r
   //\r
   // Make sure BlockSize and LBA is consistent with BufferSize\r
   //\r
@@ -300,20 +310,30 @@ UsbMassWriteBlocks (
   TotalBlock = BufferSize / Media->BlockSize;\r
 \r
   if (Lba + TotalBlock - 1 > Media->LastBlock) {\r
-    Status = EFI_BAD_BUFFER_SIZE;\r
+    Status = EFI_INVALID_PARAMETER;\r
     goto ON_EXIT;\r
   }\r
-  \r
+\r
+  if (!(Media->MediaPresent)) {\r
+    Status = EFI_NO_MEDIA;\r
+    goto ON_EXIT;\r
+  }\r
+\r
+  if (MediaId != Media->MediaId) {\r
+    Status = EFI_MEDIA_CHANGED;\r
+    goto ON_EXIT;\r
+  }\r
+\r
   //\r
   // Try to write the data even the device is marked as ReadOnly,\r
   // and clear the status should the write succeed.\r
   //\r
   Status = UsbBootWriteBlocks (UsbMass, (UINT32) Lba, TotalBlock, Buffer);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "UsbMassWriteBlocks: UsbBootWriteBlocks (%r) -> Reset\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "UsbMassWriteBlocks: UsbBootWriteBlocks (%r) -> Reset\n", Status));\r
     UsbMassReset (This, TRUE);\r
   }\r
-  \r
+\r
 ON_EXIT:\r
   gBS->RestoreTPL (OldTpl);\r
   return Status;\r
@@ -330,6 +350,7 @@ ON_EXIT:
 \r
 **/\r
 EFI_STATUS\r
+EFIAPI\r
 UsbMassFlushBlocks (\r
   IN EFI_BLOCK_IO_PROTOCOL  *This\r
   )\r
@@ -402,7 +423,7 @@ USBMassDriverBindingSupported (
     }\r
   }\r
 \r
-  DEBUG ((mUsbMscInfo, "Found a USB mass store device %r\n", Status));\r
+  DEBUG ((EFI_D_INFO, "Found a USB mass store device %r\n", Status));\r
 \r
 ON_EXIT:\r
   gBS->CloseProtocol (\r
@@ -467,7 +488,7 @@ USBMassDriverBindingStart (
   //\r
   Status = UsbIo->UsbGetInterfaceDescriptor (UsbIo, &Interface);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "USBMassDriverBindingStart: UsbIo->UsbGetInterfaceDescriptor (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "USBMassDriverBindingStart: UsbIo->UsbGetInterfaceDescriptor (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -484,7 +505,7 @@ USBMassDriverBindingStart (
   }\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((mUsbMscError, "USBMassDriverBindingStart: Transport->Init (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "USBMassDriverBindingStart: Transport->Init (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r
@@ -508,11 +529,11 @@ USBMassDriverBindingStart (
          (UsbMass->Pdt != USB_PDT_CDROM) &&\r
          (UsbMass->Pdt != USB_PDT_OPTICAL) &&\r
          (UsbMass->Pdt != USB_PDT_SIMPLE_DIRECT)) {\r
-      DEBUG ((mUsbMscError, "USBMassDriverBindingStart: Found an unsupported peripheral type[%d]\n", UsbMass->Pdt));\r
+      DEBUG ((EFI_D_ERROR, "USBMassDriverBindingStart: Found an unsupported peripheral type[%d]\n", UsbMass->Pdt));\r
       goto ON_ERROR;\r
     }\r
   } else if (Status != EFI_NO_MEDIA){\r
-    DEBUG ((mUsbMscError, "USBMassDriverBindingStart: UsbMassInitMedia (%r)\n", Status));\r
+    DEBUG ((EFI_D_ERROR, "USBMassDriverBindingStart: UsbMassInitMedia (%r)\n", Status));\r
     goto ON_ERROR;\r
   }\r
 \r