]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Bus/Usb/UsbBusDxe/UsbHub.c
MdeModulePkg: Apply uncrustify changes
[mirror_edk2.git] / MdeModulePkg / Bus / Usb / UsbBusDxe / UsbHub.c
index 022bab5fc1f12df3a2f668488e6eeee00cf997c9..0497a73f443c9fb59f22dd6bf0ad7d1a0279116b 100644 (file)
@@ -17,19 +17,19 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
 // bit maps.\r
 //\r
 USB_CHANGE_FEATURE_MAP  mHubFeatureMap[] = {\r
-  {USB_PORT_STAT_C_CONNECTION,  EfiUsbPortConnectChange},\r
-  {USB_PORT_STAT_C_ENABLE,      EfiUsbPortEnableChange},\r
-  {USB_PORT_STAT_C_SUSPEND,     EfiUsbPortSuspendChange},\r
-  {USB_PORT_STAT_C_OVERCURRENT, EfiUsbPortOverCurrentChange},\r
-  {USB_PORT_STAT_C_RESET,       EfiUsbPortResetChange}\r
+  { USB_PORT_STAT_C_CONNECTION,  EfiUsbPortConnectChange     },\r
+  { USB_PORT_STAT_C_ENABLE,      EfiUsbPortEnableChange      },\r
+  { USB_PORT_STAT_C_SUSPEND,     EfiUsbPortSuspendChange     },\r
+  { USB_PORT_STAT_C_OVERCURRENT, EfiUsbPortOverCurrentChange },\r
+  { USB_PORT_STAT_C_RESET,       EfiUsbPortResetChange       }\r
 };\r
 \r
 USB_CHANGE_FEATURE_MAP  mRootHubFeatureMap[] = {\r
-  {USB_PORT_STAT_C_CONNECTION,  EfiUsbPortConnectChange},\r
-  {USB_PORT_STAT_C_ENABLE,      EfiUsbPortEnableChange},\r
-  {USB_PORT_STAT_C_SUSPEND,     EfiUsbPortSuspendChange},\r
-  {USB_PORT_STAT_C_OVERCURRENT, EfiUsbPortOverCurrentChange},\r
-  {USB_PORT_STAT_C_RESET,       EfiUsbPortResetChange},\r
+  { USB_PORT_STAT_C_CONNECTION,  EfiUsbPortConnectChange     },\r
+  { USB_PORT_STAT_C_ENABLE,      EfiUsbPortEnableChange      },\r
+  { USB_PORT_STAT_C_SUSPEND,     EfiUsbPortSuspendChange     },\r
+  { USB_PORT_STAT_C_OVERCURRENT, EfiUsbPortOverCurrentChange },\r
+  { USB_PORT_STAT_C_RESET,       EfiUsbPortResetChange       },\r
 };\r
 \r
 //\r
@@ -37,6 +37,7 @@ USB_CHANGE_FEATURE_MAP  mRootHubFeatureMap[] = {
 // is related to an interface, these requests are sent\r
 // to the control endpoint of the device.\r
 //\r
+\r
 /**\r
   USB hub control transfer to set the hub depth.\r
 \r
@@ -49,11 +50,11 @@ USB_CHANGE_FEATURE_MAP  mRootHubFeatureMap[] = {
 **/\r
 EFI_STATUS\r
 UsbHubCtrlSetHubDepth (\r
-  IN  USB_DEVICE          *HubDev,\r
-  IN  UINT16              Depth\r
+  IN  USB_DEVICE  *HubDev,\r
+  IN  UINT16      Depth\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   Status = UsbCtrlRequest (\r
              HubDev,\r
@@ -82,11 +83,11 @@ UsbHubCtrlSetHubDepth (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlClearHubFeature (\r
-  IN USB_DEVICE           *HubDev,\r
-  IN UINT16               Feature\r
+  IN USB_DEVICE  *HubDev,\r
+  IN UINT16      Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   Status = UsbCtrlRequest (\r
              HubDev,\r
@@ -103,7 +104,6 @@ UsbHubCtrlClearHubFeature (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Clear the feature of the device's port.\r
 \r
@@ -117,12 +117,12 @@ UsbHubCtrlClearHubFeature (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlClearPortFeature (\r
-  IN USB_DEVICE           *HubDev,\r
-  IN UINT8                Port,\r
-  IN UINT16               Feature\r
+  IN USB_DEVICE  *HubDev,\r
+  IN UINT8       Port,\r
+  IN UINT16      Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // In USB bus, all the port index starts from 0. But HUB\r
@@ -135,7 +135,7 @@ UsbHubCtrlClearPortFeature (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_CLEAR_FEATURE,\r
              Feature,\r
-             (UINT16) (Port + 1),\r
+             (UINT16)(Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -143,7 +143,6 @@ UsbHubCtrlClearPortFeature (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Clear the transaction translate buffer if full/low\r
   speed control/bulk transfer failed and the transfer\r
@@ -163,21 +162,21 @@ UsbHubCtrlClearPortFeature (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlClearTTBuffer (\r
-  IN USB_DEVICE           *HubDev,\r
-  IN UINT8                Port,\r
-  IN UINT16               DevAddr,\r
-  IN UINT16               EpNum,\r
-  IN UINT16               EpType\r
+  IN USB_DEVICE  *HubDev,\r
+  IN UINT8       Port,\r
+  IN UINT16      DevAddr,\r
+  IN UINT16      EpNum,\r
+  IN UINT16      EpType\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  UINT16                  Value;\r
+  EFI_STATUS  Status;\r
+  UINT16      Value;\r
 \r
   //\r
   // Check USB2.0 spec page 424 for wValue's encoding\r
   //\r
-  Value = (UINT16) ((EpNum & 0x0F) | (DevAddr << 4) |\r
-          ((EpType & 0x03) << 11) | ((EpNum & 0x80) << 15));\r
+  Value = (UINT16)((EpNum & 0x0F) | (DevAddr << 4) |\r
+                   ((EpType & 0x03) << 11) | ((EpNum & 0x80) << 15));\r
 \r
   Status = UsbCtrlRequest (\r
              HubDev,\r
@@ -186,7 +185,7 @@ UsbHubCtrlClearTTBuffer (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_CLEAR_TT,\r
              Value,\r
-             (UINT16) (Port + 1),\r
+             (UINT16)(Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -207,13 +206,13 @@ UsbHubCtrlClearTTBuffer (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlGetHubDesc (\r
-  IN  USB_DEVICE          *HubDev,\r
-  OUT VOID                *Buf,\r
-  IN  UINTN               Len\r
+  IN  USB_DEVICE  *HubDev,\r
+  OUT VOID        *Buf,\r
+  IN  UINTN       Len\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  UINT8                   DescType;\r
+  EFI_STATUS  Status;\r
+  UINT8       DescType;\r
 \r
   DescType = (HubDev->Speed == EFI_USB_SPEED_SUPER) ?\r
              USB_DESC_TYPE_HUB_SUPER_SPEED :\r
@@ -225,7 +224,7 @@ UsbHubCtrlGetHubDesc (
              USB_REQ_TYPE_CLASS,\r
              USB_HUB_TARGET_HUB,\r
              USB_HUB_REQ_GET_DESC,\r
-             (UINT16) (DescType << 8),\r
+             (UINT16)(DescType << 8),\r
              0,\r
              Buf,\r
              Len\r
@@ -234,7 +233,6 @@ UsbHubCtrlGetHubDesc (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Usb hub control transfer to get the hub status.\r
 \r
@@ -247,11 +245,11 @@ UsbHubCtrlGetHubDesc (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlGetHubStatus (\r
-  IN  USB_DEVICE          *HubDev,\r
-  OUT UINT32              *State\r
+  IN  USB_DEVICE  *HubDev,\r
+  OUT UINT32      *State\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   Status = UsbCtrlRequest (\r
              HubDev,\r
@@ -268,7 +266,6 @@ UsbHubCtrlGetHubStatus (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Usb hub control transfer to get the port status.\r
 \r
@@ -282,12 +279,12 @@ UsbHubCtrlGetHubStatus (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlGetPortStatus (\r
-  IN  USB_DEVICE          *HubDev,\r
-  IN  UINT8               Port,\r
-  OUT VOID                *State\r
+  IN  USB_DEVICE  *HubDev,\r
+  IN  UINT8       Port,\r
+  OUT VOID        *State\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // In USB bus, all the port index starts from 0. But HUB\r
@@ -302,7 +299,7 @@ UsbHubCtrlGetPortStatus (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_GET_STATUS,\r
              0,\r
-             (UINT16) (Port + 1),\r
+             (UINT16)(Port + 1),\r
              State,\r
              4\r
              );\r
@@ -310,7 +307,6 @@ UsbHubCtrlGetPortStatus (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Usb hub control transfer to set the port feature.\r
 \r
@@ -324,12 +320,12 @@ UsbHubCtrlGetPortStatus (
 **/\r
 EFI_STATUS\r
 UsbHubCtrlSetPortFeature (\r
-  IN USB_DEVICE           *HubDev,\r
-  IN UINT8                Port,\r
-  IN UINT8                Feature\r
+  IN USB_DEVICE  *HubDev,\r
+  IN UINT8       Port,\r
+  IN UINT8       Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // In USB bus, all the port index starts from 0. But HUB\r
@@ -342,7 +338,7 @@ UsbHubCtrlSetPortFeature (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_SET_FEATURE,\r
              Feature,\r
-             (UINT16) (Port + 1),\r
+             (UINT16)(Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -350,7 +346,6 @@ UsbHubCtrlSetPortFeature (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Read the whole usb hub descriptor. It is necessary\r
   to do it in two steps because hub descriptor is of\r
@@ -369,7 +364,7 @@ UsbHubReadDesc (
   OUT EFI_USB_HUB_DESCRIPTOR  *HubDesc\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
   //\r
   // First get the hub descriptor length\r
@@ -386,8 +381,6 @@ UsbHubReadDesc (
   return UsbHubCtrlGetHubDesc (HubDev, HubDesc, HubDesc->Length);\r
 }\r
 \r
-\r
-\r
 /**\r
   Ack the hub change bits. If these bits are not ACKed, Hub will\r
   always return changed bit map from its interrupt endpoint.\r
@@ -400,13 +393,13 @@ UsbHubReadDesc (
 **/\r
 EFI_STATUS\r
 UsbHubAckHubStatus (\r
-  IN  USB_DEVICE         *HubDev\r
+  IN  USB_DEVICE  *HubDev\r
   )\r
 {\r
-  EFI_USB_PORT_STATUS     HubState;\r
-  EFI_STATUS              Status;\r
+  EFI_USB_PORT_STATUS  HubState;\r
+  EFI_STATUS           Status;\r
 \r
-  Status = UsbHubCtrlGetHubStatus (HubDev, (UINT32 *) &HubState);\r
+  Status = UsbHubCtrlGetHubStatus (HubDev, (UINT32 *)&HubState);\r
 \r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
@@ -423,7 +416,6 @@ UsbHubAckHubStatus (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Test whether the interface is a hub interface.\r
 \r
@@ -435,7 +427,7 @@ UsbHubAckHubStatus (
 **/\r
 BOOLEAN\r
 UsbIsHubInterface (\r
-  IN USB_INTERFACE        *UsbIf\r
+  IN USB_INTERFACE  *UsbIf\r
   )\r
 {\r
   EFI_USB_INTERFACE_DESCRIPTOR  *Setting;\r
@@ -447,15 +439,14 @@ UsbIsHubInterface (
   Setting = &UsbIf->IfSetting->Desc;\r
 \r
   if ((Setting->InterfaceClass == USB_HUB_CLASS_CODE) &&\r
-      (Setting->InterfaceSubClass == USB_HUB_SUBCLASS_CODE)) {\r
-\r
+      (Setting->InterfaceSubClass == USB_HUB_SUBCLASS_CODE))\r
+  {\r
     return TRUE;\r
   }\r
 \r
   return FALSE;\r
 }\r
 \r
-\r
 /**\r
   The callback function to the USB hub status change\r
   interrupt endpoint. It is called periodically by\r
@@ -473,20 +464,20 @@ UsbIsHubInterface (
 EFI_STATUS\r
 EFIAPI\r
 UsbOnHubInterrupt (\r
-  IN  VOID                *Data,\r
-  IN  UINTN               DataLength,\r
-  IN  VOID                *Context,\r
-  IN  UINT32              Result\r
+  IN  VOID    *Data,\r
+  IN  UINTN   DataLength,\r
+  IN  VOID    *Context,\r
+  IN  UINT32  Result\r
   )\r
 {\r
-  USB_INTERFACE               *HubIf;\r
-  EFI_USB_IO_PROTOCOL         *UsbIo;\r
-  EFI_USB_ENDPOINT_DESCRIPTOR *EpDesc;\r
-  EFI_STATUS                  Status;\r
+  USB_INTERFACE                *HubIf;\r
+  EFI_USB_IO_PROTOCOL          *UsbIo;\r
+  EFI_USB_ENDPOINT_DESCRIPTOR  *EpDesc;\r
+  EFI_STATUS                   Status;\r
 \r
-  HubIf   = (USB_INTERFACE *) Context;\r
-  UsbIo   = &(HubIf->UsbIo);\r
-  EpDesc  = &(HubIf->HubEp->Desc);\r
+  HubIf  = (USB_INTERFACE *)Context;\r
+  UsbIo  = &(HubIf->UsbIo);\r
+  EpDesc = &(HubIf->HubEp->Desc);\r
 \r
   if (Result != EFI_USB_NOERROR) {\r
     //\r
@@ -516,7 +507,7 @@ UsbOnHubInterrupt (
                       );\r
 \r
     if (EFI_ERROR (Status)) {\r
-      DEBUG (( DEBUG_ERROR, "UsbOnHubInterrupt: failed to remove async transfer - %r\n", Status));\r
+      DEBUG ((DEBUG_ERROR, "UsbOnHubInterrupt: failed to remove async transfer - %r\n", Status));\r
       return Status;\r
     }\r
 \r
@@ -531,7 +522,7 @@ UsbOnHubInterrupt (
                       );\r
 \r
     if (EFI_ERROR (Status)) {\r
-      DEBUG (( DEBUG_ERROR, "UsbOnHubInterrupt: failed to submit new async transfer - %r\n", Status));\r
+      DEBUG ((DEBUG_ERROR, "UsbOnHubInterrupt: failed to submit new async transfer - %r\n", Status));\r
     }\r
 \r
     return Status;\r
@@ -559,9 +550,6 @@ UsbOnHubInterrupt (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
-\r
-\r
 /**\r
   Initialize the device for a non-root hub.\r
 \r
@@ -573,7 +561,7 @@ UsbOnHubInterrupt (
 **/\r
 EFI_STATUS\r
 UsbHubInit (\r
-  IN USB_INTERFACE        *HubIf\r
+  IN USB_INTERFACE  *HubIf\r
   )\r
 {\r
   UINT8                   HubDescBuffer[256];\r
@@ -590,11 +578,11 @@ UsbHubInit (
   //\r
   // Locate the interrupt endpoint for port change map\r
   //\r
-  HubIf->IsHub  = FALSE;\r
-  Setting       = HubIf->IfSetting;\r
-  HubDev        = HubIf->Device;\r
-  EpDesc        = NULL;\r
-  NumEndpoints  = Setting->Desc.NumEndpoints;\r
+  HubIf->IsHub = FALSE;\r
+  Setting      = HubIf->IfSetting;\r
+  HubDev       = HubIf->Device;\r
+  EpDesc       = NULL;\r
+  NumEndpoints = Setting->Desc.NumEndpoints;\r
 \r
   for (Index = 0; Index < NumEndpoints; Index++) {\r
     ASSERT ((Setting->Endpoints != NULL) && (Setting->Endpoints[Index] != NULL));\r
@@ -602,13 +590,14 @@ UsbHubInit (
     EpDesc = Setting->Endpoints[Index];\r
 \r
     if (USB_BIT_IS_SET (EpDesc->Desc.EndpointAddress, USB_ENDPOINT_DIR_IN) &&\r
-       (USB_ENDPOINT_TYPE (&EpDesc->Desc) == USB_ENDPOINT_INTERRUPT)) {\r
+        (USB_ENDPOINT_TYPE (&EpDesc->Desc) == USB_ENDPOINT_INTERRUPT))\r
+    {\r
       break;\r
     }\r
   }\r
 \r
   if (Index == NumEndpoints) {\r
-    DEBUG (( DEBUG_ERROR, "UsbHubInit: no interrupt endpoint found for hub %d\n", HubDev->Address));\r
+    DEBUG ((DEBUG_ERROR, "UsbHubInit: no interrupt endpoint found for hub %d\n", HubDev->Address));\r
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
@@ -616,17 +605,17 @@ UsbHubInit (
   // The length field of descriptor is UINT8 type, so the buffer\r
   // with 256 bytes is enough to hold the descriptor data.\r
   //\r
-  HubDesc = (EFI_USB_HUB_DESCRIPTOR *) HubDescBuffer;\r
-  Status = UsbHubReadDesc (HubDev, HubDesc);\r
+  HubDesc = (EFI_USB_HUB_DESCRIPTOR *)HubDescBuffer;\r
+  Status  = UsbHubReadDesc (HubDev, HubDesc);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG (( DEBUG_ERROR, "UsbHubInit: failed to read HUB descriptor %r\n", Status));\r
+    DEBUG ((DEBUG_ERROR, "UsbHubInit: failed to read HUB descriptor %r\n", Status));\r
     return Status;\r
   }\r
 \r
   HubIf->NumOfPort = HubDesc->NumPorts;\r
 \r
-  DEBUG (( DEBUG_INFO, "UsbHubInit: hub %d has %d ports\n", HubDev->Address,HubIf->NumOfPort));\r
+  DEBUG ((DEBUG_INFO, "UsbHubInit: hub %d has %d ports\n", HubDev->Address, HubIf->NumOfPort));\r
 \r
   //\r
   // OK, set IsHub to TRUE. Now usb bus can handle this device\r
@@ -652,7 +641,7 @@ UsbHubInit (
     // for both gang/individual powered hubs.\r
     //\r
     for (Index = 0; Index < HubDesc->NumPorts; Index++) {\r
-      UsbHubCtrlSetPortFeature (HubIf->Device, Index, (EFI_USB_PORT_FEATURE) USB_HUB_PORT_POWER);\r
+      UsbHubCtrlSetPortFeature (HubIf->Device, Index, (EFI_USB_PORT_FEATURE)USB_HUB_PORT_POWER);\r
     }\r
 \r
     //\r
@@ -661,6 +650,7 @@ UsbHubInit (
     if (HubDesc->PwrOn2PwrGood > 0) {\r
       gBS->Stall (HubDesc->PwrOn2PwrGood * USB_SET_PORT_POWER_STALL);\r
     }\r
+\r
     UsbHubAckHubStatus (HubIf->Device);\r
   }\r
 \r
@@ -676,8 +666,12 @@ UsbHubInit (
                   );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG (( DEBUG_ERROR, "UsbHubInit: failed to create signal for hub %d - %r\n",\r
-                HubDev->Address, Status));\r
+    DEBUG ((\r
+      DEBUG_ERROR,\r
+      "UsbHubInit: failed to create signal for hub %d - %r\n",\r
+      HubDev->Address,\r
+      Status\r
+      ));\r
 \r
     return Status;\r
   }\r
@@ -701,8 +695,12 @@ UsbHubInit (
                     );\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG (( DEBUG_ERROR, "UsbHubInit: failed to queue interrupt transfer for hub %d - %r\n",\r
-                HubDev->Address, Status));\r
+    DEBUG ((\r
+      DEBUG_ERROR,\r
+      "UsbHubInit: failed to queue interrupt transfer for hub %d - %r\n",\r
+      HubDev->Address,\r
+      Status\r
+      ));\r
 \r
     gBS->CloseEvent (HubIf->HubNotify);\r
     HubIf->HubNotify = NULL;\r
@@ -710,12 +708,10 @@ UsbHubInit (
     return Status;\r
   }\r
 \r
-  DEBUG (( DEBUG_INFO, "UsbHubInit: hub %d initialized\n", HubDev->Address));\r
+  DEBUG ((DEBUG_INFO, "UsbHubInit: hub %d initialized\n", HubDev->Address));\r
   return Status;\r
 }\r
 \r
-\r
-\r
 /**\r
   Get the port status. This function is required to\r
   ACK the port change bits although it will return\r
@@ -732,20 +728,18 @@ UsbHubInit (
 **/\r
 EFI_STATUS\r
 UsbHubGetPortStatus (\r
-  IN  USB_INTERFACE       *HubIf,\r
-  IN  UINT8               Port,\r
-  OUT EFI_USB_PORT_STATUS *PortState\r
+  IN  USB_INTERFACE        *HubIf,\r
+  IN  UINT8                Port,\r
+  OUT EFI_USB_PORT_STATUS  *PortState\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status  = UsbHubCtrlGetPortStatus (HubIf->Device, Port, PortState);\r
+  Status = UsbHubCtrlGetPortStatus (HubIf->Device, Port, PortState);\r
 \r
   return Status;\r
 }\r
 \r
-\r
-\r
 /**\r
   Clear the port change status.\r
 \r
@@ -755,8 +749,8 @@ UsbHubGetPortStatus (
 **/\r
 VOID\r
 UsbHubClearPortChange (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port\r
+  IN USB_INTERFACE  *HubIf,\r
+  IN UINT8          Port\r
   )\r
 {\r
   EFI_USB_PORT_STATUS     PortState;\r
@@ -780,13 +774,11 @@ UsbHubClearPortChange (
     Map = &mHubFeatureMap[Index];\r
 \r
     if (USB_BIT_IS_SET (PortState.PortChangeStatus, Map->ChangedBit)) {\r
-      UsbHubCtrlClearPortFeature (HubIf->Device, Port, (UINT16) Map->Feature);\r
+      UsbHubCtrlClearPortFeature (HubIf->Device, Port, (UINT16)Map->Feature);\r
     }\r
   }\r
 }\r
 \r
-\r
-\r
 /**\r
   Function to set the port feature for non-root hub.\r
 \r
@@ -800,19 +792,18 @@ UsbHubClearPortChange (
 **/\r
 EFI_STATUS\r
 UsbHubSetPortFeature (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port,\r
-  IN EFI_USB_PORT_FEATURE Feature\r
+  IN USB_INTERFACE         *HubIf,\r
+  IN UINT8                 Port,\r
+  IN EFI_USB_PORT_FEATURE  Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status = UsbHubCtrlSetPortFeature (HubIf->Device, Port, (UINT8) Feature);\r
+  Status = UsbHubCtrlSetPortFeature (HubIf->Device, Port, (UINT8)Feature);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Interface function to clear the port feature for non-root hub.\r
 \r
@@ -826,19 +817,18 @@ UsbHubSetPortFeature (
 **/\r
 EFI_STATUS\r
 UsbHubClearPortFeature (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port,\r
-  IN EFI_USB_PORT_FEATURE Feature\r
+  IN USB_INTERFACE         *HubIf,\r
+  IN UINT8                 Port,\r
+  IN EFI_USB_PORT_FEATURE  Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status = UsbHubCtrlClearPortFeature (HubIf->Device, Port, (UINT8) Feature);\r
+  Status = UsbHubCtrlClearPortFeature (HubIf->Device, Port, (UINT8)Feature);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Interface function to reset the port.\r
 \r
@@ -852,15 +842,15 @@ UsbHubClearPortFeature (
 **/\r
 EFI_STATUS\r
 UsbHubResetPort (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port\r
+  IN USB_INTERFACE  *HubIf,\r
+  IN UINT8          Port\r
   )\r
 {\r
-  EFI_USB_PORT_STATUS     PortState;\r
-  UINTN                   Index;\r
-  EFI_STATUS              Status;\r
+  EFI_USB_PORT_STATUS  PortState;\r
+  UINTN                Index;\r
+  EFI_STATUS           Status;\r
 \r
-  Status  = UsbHubSetPortFeature (HubIf, Port, (EFI_USB_PORT_FEATURE) USB_HUB_PORT_RESET);\r
+  Status = UsbHubSetPortFeature (HubIf, Port, (EFI_USB_PORT_FEATURE)USB_HUB_PORT_RESET);\r
 \r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
@@ -885,7 +875,8 @@ UsbHubResetPort (
     }\r
 \r
     if (!EFI_ERROR (Status) &&\r
-        USB_BIT_IS_SET (PortState.PortChangeStatus, USB_PORT_STAT_C_RESET)) {\r
+        USB_BIT_IS_SET (PortState.PortChangeStatus, USB_PORT_STAT_C_RESET))\r
+    {\r
       gBS->Stall (USB_SET_PORT_RECOVERY_STALL);\r
       return EFI_SUCCESS;\r
     }\r
@@ -896,7 +887,6 @@ UsbHubResetPort (
   return EFI_TIMEOUT;\r
 }\r
 \r
-\r
 /**\r
   Release the hub's control of the interface.\r
 \r
@@ -907,11 +897,11 @@ UsbHubResetPort (
 **/\r
 EFI_STATUS\r
 UsbHubRelease (\r
-  IN USB_INTERFACE        *HubIf\r
+  IN USB_INTERFACE  *HubIf\r
   )\r
 {\r
-  EFI_USB_IO_PROTOCOL     *UsbIo;\r
-  EFI_STATUS              Status;\r
+  EFI_USB_IO_PROTOCOL  *UsbIo;\r
+  EFI_STATUS           Status;\r
 \r
   UsbIo  = &HubIf->UsbIo;\r
   Status = UsbIo->UsbAsyncInterruptTransfer (\r
@@ -930,17 +920,15 @@ UsbHubRelease (
 \r
   gBS->CloseEvent (HubIf->HubNotify);\r
 \r
-  HubIf->IsHub      = FALSE;\r
-  HubIf->HubApi     = NULL;\r
-  HubIf->HubEp      = NULL;\r
-  HubIf->HubNotify  = NULL;\r
+  HubIf->IsHub     = FALSE;\r
+  HubIf->HubApi    = NULL;\r
+  HubIf->HubEp     = NULL;\r
+  HubIf->HubNotify = NULL;\r
 \r
-  DEBUG (( DEBUG_INFO, "UsbHubRelease: hub device %d released\n", HubIf->Device->Address));\r
+  DEBUG ((DEBUG_INFO, "UsbHubRelease: hub device %d released\n", HubIf->Device->Address));\r
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
-\r
 /**\r
   Initialize the interface for root hub.\r
 \r
@@ -952,13 +940,13 @@ UsbHubRelease (
 **/\r
 EFI_STATUS\r
 UsbRootHubInit (\r
-  IN USB_INTERFACE        *HubIf\r
+  IN USB_INTERFACE  *HubIf\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
-  UINT8                   MaxSpeed;\r
-  UINT8                   NumOfPort;\r
-  UINT8                   Support64;\r
+  EFI_STATUS  Status;\r
+  UINT8       MaxSpeed;\r
+  UINT8       NumOfPort;\r
+  UINT8       Support64;\r
 \r
   Status = UsbHcGetCapability (HubIf->Device->Bus, &MaxSpeed, &NumOfPort, &Support64);\r
 \r
@@ -966,15 +954,20 @@ UsbRootHubInit (
     return Status;\r
   }\r
 \r
-  DEBUG (( DEBUG_INFO, "UsbRootHubInit: root hub %p - max speed %d, %d ports\n",\r
-              HubIf, MaxSpeed, NumOfPort));\r
-\r
-  HubIf->IsHub      = TRUE;\r
-  HubIf->HubApi     = &mUsbRootHubApi;\r
-  HubIf->HubEp      = NULL;\r
-  HubIf->MaxSpeed   = MaxSpeed;\r
-  HubIf->NumOfPort  = NumOfPort;\r
-  HubIf->HubNotify  = NULL;\r
+  DEBUG ((\r
+    DEBUG_INFO,\r
+    "UsbRootHubInit: root hub %p - max speed %d, %d ports\n",\r
+    HubIf,\r
+    MaxSpeed,\r
+    NumOfPort\r
+    ));\r
+\r
+  HubIf->IsHub     = TRUE;\r
+  HubIf->HubApi    = &mUsbRootHubApi;\r
+  HubIf->HubEp     = NULL;\r
+  HubIf->MaxSpeed  = MaxSpeed;\r
+  HubIf->NumOfPort = NumOfPort;\r
+  HubIf->HubNotify = NULL;\r
 \r
   //\r
   // Create a timer to poll root hub ports periodically\r
@@ -1010,7 +1003,6 @@ UsbRootHubInit (
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Get the port status. This function is required to\r
   ACK the port change bits although it will return\r
@@ -1027,21 +1019,20 @@ UsbRootHubInit (
 **/\r
 EFI_STATUS\r
 UsbRootHubGetPortStatus (\r
-  IN  USB_INTERFACE       *HubIf,\r
-  IN  UINT8               Port,\r
-  OUT EFI_USB_PORT_STATUS *PortState\r
+  IN  USB_INTERFACE        *HubIf,\r
+  IN  UINT8                Port,\r
+  OUT EFI_USB_PORT_STATUS  *PortState\r
   )\r
 {\r
-  USB_BUS                 *Bus;\r
-  EFI_STATUS              Status;\r
+  USB_BUS     *Bus;\r
+  EFI_STATUS  Status;\r
 \r
-  Bus     = HubIf->Device->Bus;\r
-  Status  = UsbHcGetRootHubPortStatus (Bus, Port, PortState);\r
+  Bus    = HubIf->Device->Bus;\r
+  Status = UsbHcGetRootHubPortStatus (Bus, Port, PortState);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Clear the port change status.\r
 \r
@@ -1051,8 +1042,8 @@ UsbRootHubGetPortStatus (
 **/\r
 VOID\r
 UsbRootHubClearPortChange (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port\r
+  IN USB_INTERFACE  *HubIf,\r
+  IN UINT8          Port\r
   )\r
 {\r
   EFI_USB_PORT_STATUS     PortState;\r
@@ -1076,12 +1067,11 @@ UsbRootHubClearPortChange (
     Map = &mRootHubFeatureMap[Index];\r
 \r
     if (USB_BIT_IS_SET (PortState.PortChangeStatus, Map->ChangedBit)) {\r
-      UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, (EFI_USB_PORT_FEATURE) Map->Feature);\r
+      UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, (EFI_USB_PORT_FEATURE)Map->Feature);\r
     }\r
   }\r
 }\r
 \r
-\r
 /**\r
   Set the root hub port feature.\r
 \r
@@ -1095,19 +1085,18 @@ UsbRootHubClearPortChange (
 **/\r
 EFI_STATUS\r
 UsbRootHubSetPortFeature (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port,\r
-  IN EFI_USB_PORT_FEATURE Feature\r
+  IN USB_INTERFACE         *HubIf,\r
+  IN UINT8                 Port,\r
+  IN EFI_USB_PORT_FEATURE  Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status  = UsbHcSetRootHubPortFeature (HubIf->Device->Bus, Port, Feature);\r
+  Status = UsbHcSetRootHubPortFeature (HubIf->Device->Bus, Port, Feature);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Clear the root hub port feature.\r
 \r
@@ -1121,19 +1110,18 @@ UsbRootHubSetPortFeature (
 **/\r
 EFI_STATUS\r
 UsbRootHubClearPortFeature (\r
-  IN USB_INTERFACE        *HubIf,\r
-  IN UINT8                Port,\r
-  IN EFI_USB_PORT_FEATURE Feature\r
+  IN USB_INTERFACE         *HubIf,\r
+  IN UINT8                 Port,\r
+  IN EFI_USB_PORT_FEATURE  Feature\r
   )\r
 {\r
-  EFI_STATUS              Status;\r
+  EFI_STATUS  Status;\r
 \r
-  Status  = UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, Feature);\r
+  Status = UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, Feature);\r
 \r
   return Status;\r
 }\r
 \r
-\r
 /**\r
   Interface function to reset the root hub port.\r
 \r
@@ -1149,26 +1137,26 @@ UsbRootHubClearPortFeature (
 **/\r
 EFI_STATUS\r
 UsbRootHubResetPort (\r
-  IN USB_INTERFACE        *RootIf,\r
-  IN UINT8                Port\r
+  IN USB_INTERFACE  *RootIf,\r
+  IN UINT8          Port\r
   )\r
 {\r
-  USB_BUS                 *Bus;\r
-  EFI_STATUS              Status;\r
-  EFI_USB_PORT_STATUS     PortState;\r
-  UINTN                   Index;\r
+  USB_BUS              *Bus;\r
+  EFI_STATUS           Status;\r
+  EFI_USB_PORT_STATUS  PortState;\r
+  UINTN                Index;\r
 \r
   //\r
   // Notice: although EHCI requires that ENABLED bit be cleared\r
   // when reset the port, we don't need to care that here. It\r
   // should be handled in the EHCI driver.\r
   //\r
-  Bus     = RootIf->Device->Bus;\r
+  Bus = RootIf->Device->Bus;\r
 \r
-  Status  = UsbHcSetRootHubPortFeature (Bus, Port, EfiUsbPortReset);\r
+  Status = UsbHcSetRootHubPortFeature (Bus, Port, EfiUsbPortReset);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG (( DEBUG_ERROR, "UsbRootHubResetPort: failed to start reset on port %d\n", Port));\r
+    DEBUG ((DEBUG_ERROR, "UsbRootHubResetPort: failed to start reset on port %d\n", Port));\r
     return Status;\r
   }\r
 \r
@@ -1181,7 +1169,7 @@ UsbRootHubResetPort (
   Status = UsbHcClearRootHubPortFeature (Bus, Port, EfiUsbPortReset);\r
 \r
   if (EFI_ERROR (Status)) {\r
-    DEBUG (( DEBUG_ERROR, "UsbRootHubResetPort: failed to clear reset on port %d\n", Port));\r
+    DEBUG ((DEBUG_ERROR, "UsbRootHubResetPort: failed to clear reset on port %d\n", Port));\r
     return Status;\r
   }\r
 \r
@@ -1220,17 +1208,15 @@ UsbRootHubResetPort (
     // automatically enable the port, we need to enable it manually.\r
     //\r
     if (RootIf->MaxSpeed == EFI_USB_SPEED_HIGH) {\r
-      DEBUG (( DEBUG_ERROR, "UsbRootHubResetPort: release low/full speed device (%d) to UHCI\n", Port));\r
+      DEBUG ((DEBUG_ERROR, "UsbRootHubResetPort: release low/full speed device (%d) to UHCI\n", Port));\r
 \r
       UsbRootHubSetPortFeature (RootIf, Port, EfiUsbPortOwner);\r
       return EFI_NOT_FOUND;\r
-\r
     } else {\r
-\r
       Status = UsbRootHubSetPortFeature (RootIf, Port, EfiUsbPortEnable);\r
 \r
       if (EFI_ERROR (Status)) {\r
-        DEBUG (( DEBUG_ERROR, "UsbRootHubResetPort: failed to enable port %d for UHCI\n", Port));\r
+        DEBUG ((DEBUG_ERROR, "UsbRootHubResetPort: failed to enable port %d for UHCI\n", Port));\r
         return Status;\r
       }\r
 \r
@@ -1241,7 +1227,6 @@ UsbRootHubResetPort (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
 /**\r
   Release the root hub's control of the interface.\r
 \r
@@ -1253,10 +1238,10 @@ UsbRootHubResetPort (
 **/\r
 EFI_STATUS\r
 UsbRootHubRelease (\r
-  IN USB_INTERFACE        *HubIf\r
+  IN USB_INTERFACE  *HubIf\r
   )\r
 {\r
-  DEBUG (( DEBUG_INFO, "UsbRootHubRelease: root hub released for hub %p\n", HubIf));\r
+  DEBUG ((DEBUG_INFO, "UsbRootHubRelease: root hub released for hub %p\n", HubIf));\r
 \r
   gBS->SetTimer (HubIf->HubNotify, TimerCancel, USB_ROOTHUB_POLL_INTERVAL);\r
   gBS->CloseEvent (HubIf->HubNotify);\r
@@ -1264,7 +1249,7 @@ UsbRootHubRelease (
   return EFI_SUCCESS;\r
 }\r
 \r
-USB_HUB_API mUsbHubApi = {\r
+USB_HUB_API  mUsbHubApi = {\r
   UsbHubInit,\r
   UsbHubGetPortStatus,\r
   UsbHubClearPortChange,\r
@@ -1274,7 +1259,7 @@ USB_HUB_API mUsbHubApi = {
   UsbHubRelease\r
 };\r
 \r
-USB_HUB_API mUsbRootHubApi = {\r
+USB_HUB_API  mUsbRootHubApi = {\r
   UsbRootHubInit,\r
   UsbRootHubGetPortStatus,\r
   UsbRootHubClearPortChange,\r