]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Bus/Usb/UsbBusDxe/UsbUtility.c
modify coding style to pass ecc tool and provide comments that complied with Doxgen.
[mirror_edk2.git] / MdeModulePkg / Bus / Usb / UsbBusDxe / UsbUtility.c
index e7a0c28412324e51d35ff83a9e1c2026b419132d..c9d4f7030caa34a17d53f35b012575d362e587ea 100644 (file)
@@ -1,5 +1,7 @@
 /** @file\r
 \r
+    Wrapper function for usb host controller interface.\r
+\r
 Copyright (c) 2007, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
@@ -9,17 +11,6 @@ http://opensource.org/licenses/bsd-license.php
 THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
 WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
-  Module Name:\r
-\r
-    UsbUtility.c\r
-\r
-  Abstract:\r
-\r
-    Wrapper function for usb host controller interface\r
-\r
-  Revision History\r
-\r
-\r
 **/\r
 \r
 \r
@@ -59,14 +50,14 @@ STATIC USB_CLASS_FORMAT_DEVICE_PATH mAllUsbClassDevicePath = {
 \r
 \r
 /**\r
-  Get the capability of the host controller\r
+  Get the capability of the host controller.\r
 \r
-  @param  UsbBus           The usb driver\r
-  @param  MaxSpeed         The maximum speed this host controller supports\r
-  @param  NumOfPort        The number of the root hub port\r
-  @param  Is64BitCapable   Whether this controller support 64 bit addressing\r
+  @param  UsbBus           The usb driver.\r
+  @param  MaxSpeed         The maximum speed this host controller supports.\r
+  @param  NumOfPort        The number of the root hub port.\r
+  @param  Is64BitCapable   Whether this controller support 64 bit addressing.\r
 \r
-  @retval EFI_SUCCESS      The host controller capability is returned\r
+  @retval EFI_SUCCESS      The host controller capability is returned.\r
   @retval Others           Failed to retrieve the host controller capability.\r
 \r
 **/\r
@@ -100,13 +91,16 @@ UsbHcGetCapability (
 \r
 \r
 /**\r
-  Reset the host controller\r
-\r
-  @param  UsbBus           The usb bus driver\r
-  @param  Attributes       The reset type, only global reset is used by this driver\r
+  Reset the host controller.\r
 \r
-  @return GC_TODO: add return values\r
+  @param  UsbBus                The usb bus driver.\r
+  @param  Attributes            The reset type, only global reset is used by this driver.\r
 \r
+  @retval EFI_SUCCESS           The reset operation succeeded.\r
+  @retval EFI_INVALID_PARAMETER Attributes is not valid.\r
+  @retval EFI_UNSUPPOURTED      The type of reset specified by Attributes is\r
+                                not currently supported by the host controller.\r
+  @retval EFI_DEVICE_ERROR      Host controller isn't halted to reset.\r
 **/\r
 EFI_STATUS\r
 UsbHcReset (\r
@@ -127,13 +121,13 @@ UsbHcReset (
 \r
 \r
 /**\r
-  Get the current operation state of the host controller\r
+  Get the current operation state of the host controller.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  State            The host controller operation state\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  State            The host controller operation state.\r
 \r
-  @retval EFI_SUCCESS      The operation state is returned in State\r
-  @retval Others           Failed to get the host controller state\r
+  @retval EFI_SUCCESS      The operation state is returned in State.\r
+  @retval Others           Failed to get the host controller state.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -155,13 +149,13 @@ UsbHcGetState (
 \r
 \r
 /**\r
-  Set the host controller operation state\r
+  Set the host controller operation state.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  State            The state to set\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  State            The state to set.\r
 \r
-  @retval EFI_SUCCESS      The host controller is now working at State\r
-  @retval Others           Failed to set operation state\r
+  @retval EFI_SUCCESS      The host controller is now working at State.\r
+  @retval Others           Failed to set operation state.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -183,14 +177,14 @@ UsbHcSetState (
 \r
 \r
 /**\r
-  Get the root hub port state\r
+  Get the root hub port state.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  PortIndex        The index of port\r
-  @param  PortStatus       The variable to save port state\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  PortIndex        The index of port.\r
+  @param  PortStatus       The variable to save port state.\r
 \r
-  @retval EFI_SUCCESS      The root port state is returned in\r
-  @retval Others           Failed to get the root hub port state\r
+  @retval EFI_SUCCESS      The root port state is returned in.\r
+  @retval Others           Failed to get the root hub port state.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -213,14 +207,14 @@ UsbHcGetRootHubPortStatus (
 \r
 \r
 /**\r
-  Set the root hub port feature\r
+  Set the root hub port feature.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  PortIndex        The port index\r
-  @param  Feature          The port feature to set\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  PortIndex        The port index.\r
+  @param  Feature          The port feature to set.\r
 \r
-  @retval EFI_SUCCESS      The port feature is set\r
-  @retval Others           Failed to set port feature\r
+  @retval EFI_SUCCESS      The port feature is set.\r
+  @retval Others           Failed to set port feature.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -244,14 +238,14 @@ UsbHcSetRootHubPortFeature (
 \r
 \r
 /**\r
-  Clear the root hub port feature\r
+  Clear the root hub port feature.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  PortIndex        The port index\r
-  @param  Feature          The port feature to clear\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  PortIndex        The port index.\r
+  @param  Feature          The port feature to clear.\r
 \r
-  @retval EFI_SUCCESS      The port feature is clear\r
-  @retval Others           Failed to clear port feature\r
+  @retval EFI_SUCCESS      The port feature is clear.\r
+  @retval Others           Failed to clear port feature.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -274,22 +268,22 @@ UsbHcClearRootHubPortFeature (
 \r
 \r
 /**\r
-  Execute a control transfer to the device\r
-\r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The device address\r
-  @param  DevSpeed         The device speed\r
-  @param  MaxPacket        Maximum packet size of endpoint 0\r
-  @param  Request          The control transfer request\r
-  @param  Direction        The direction of data stage\r
-  @param  Data             The buffer holding data\r
-  @param  DataLength       The length of the data\r
-  @param  TimeOut          Timeout (in ms) to wait until timeout\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  UsbResult        The result of transfer\r
-\r
-  @retval EFI_SUCCESS      The control transfer finished without error\r
-  @retval Others           The control transfer failed, reason returned in UsbReslt\r
+  Execute a control transfer to the device.\r
+\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The device address.\r
+  @param  DevSpeed         The device speed.\r
+  @param  MaxPacket        Maximum packet size of endpoint 0.\r
+  @param  Request          The control transfer request.\r
+  @param  Direction        The direction of data stage.\r
+  @param  Data             The buffer holding data.\r
+  @param  DataLength       The length of the data.\r
+  @param  TimeOut          Timeout (in ms) to wait until timeout.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  UsbResult        The result of transfer.\r
+\r
+  @retval EFI_SUCCESS      The control transfer finished without error.\r
+  @retval Others           The control transfer failed, reason returned in UsbReslt.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -346,25 +340,25 @@ UsbHcControlTransfer (
 \r
 \r
 /**\r
-  Execute a bulk transfer to the device's endpoint\r
+  Execute a bulk transfer to the device's endpoint.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The target device address\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The target device address.\r
   @param  EpAddr           The target endpoint address, with direction encoded in\r
-                           bit 7\r
-  @param  DevSpeed         The device's speed\r
-  @param  MaxPacket        The endpoint's max packet size\r
-  @param  BufferNum        The number of data buffer\r
-  @param  Data             Array of pointers to data buffer\r
-  @param  DataLength       The length of data buffer\r
+                           bit 7.\r
+  @param  DevSpeed         The device's speed.\r
+  @param  MaxPacket        The endpoint's max packet size.\r
+  @param  BufferNum        The number of data buffer.\r
+  @param  Data             Array of pointers to data buffer.\r
+  @param  DataLength       The length of data buffer.\r
   @param  DataToggle       On input, the initial data toggle to use, also  return\r
                            the next toggle on output.\r
-  @param  TimeOut          The time to wait until timeout\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  UsbResult        The result of USB execution\r
+  @param  TimeOut          The time to wait until timeout.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  UsbResult        The result of USB execution.\r
 \r
-  @retval EFI_SUCCESS      The bulk transfer is finished without error\r
-  @retval Others           Failed to execute bulk transfer, result in UsbResult\r
+  @retval EFI_SUCCESS      The bulk transfer is finished without error.\r
+  @retval Others           Failed to execute bulk transfer, result in UsbResult.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -419,25 +413,25 @@ UsbHcBulkTransfer (
 \r
 \r
 /**\r
-  Queue or cancel an asynchronous interrupt transfer\r
+  Queue or cancel an asynchronous interrupt transfer.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The target device address\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The target device address.\r
   @param  EpAddr           The target endpoint address, with direction encoded in\r
-                           bit 7\r
-  @param  DevSpeed         The device's speed\r
-  @param  MaxPacket        The endpoint's max packet size\r
+                           bit 7.\r
+  @param  DevSpeed         The device's speed.\r
+  @param  MaxPacket        The endpoint's max packet size.\r
   @param  IsNewTransfer    Whether this is a new request. If not, cancel the old\r
-                           request\r
-  @param  DataToggle       Data toggle to use on input, next toggle on output\r
-  @param  PollingInterval  The interval to poll the interrupt transfer (in ms)\r
-  @param  DataLength       The length of periodical data receive\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  Callback         Function to call when data is received\r
-  @param  Context          The context to the callback\r
+                           request.\r
+  @param  DataToggle       Data toggle to use on input, next toggle on output.\r
+  @param  PollingInterval  The interval to poll the interrupt transfer (in ms).\r
+  @param  DataLength       The length of periodical data receive.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  Callback         Function to call when data is received.\r
+  @param  Context          The context to the callback.\r
 \r
-  @retval EFI_SUCCESS      The asynchronous transfer is queued\r
-  @retval Others           Failed to queue the transfer\r
+  @retval EFI_SUCCESS      The asynchronous transfer is queued.\r
+  @retval Others           Failed to queue the transfer.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -497,24 +491,24 @@ UsbHcAsyncInterruptTransfer (
 \r
 \r
 /**\r
-  Execute a synchronous interrupt transfer to the target endpoint\r
+  Execute a synchronous interrupt transfer to the target endpoint.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The target device address\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The target device address.\r
   @param  EpAddr           The target endpoint address, with direction encoded in\r
-                           bit 7\r
-  @param  DevSpeed         The device's speed\r
-  @param  MaxPacket        The endpoint's max packet size\r
-  @param  Data             Pointer to data buffer\r
-  @param  DataLength       The length of data buffer\r
+                           bit 7.\r
+  @param  DevSpeed         The device's speed.\r
+  @param  MaxPacket        The endpoint's max packet size.\r
+  @param  Data             Pointer to data buffer.\r
+  @param  DataLength       The length of data buffer.\r
   @param  DataToggle       On input, the initial data toggle to use, also  return\r
                            the next toggle on output.\r
-  @param  TimeOut          The time to wait until timeout\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  UsbResult        The result of USB execution\r
+  @param  TimeOut          The time to wait until timeout.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  UsbResult        The result of USB execution.\r
 \r
-  @retval EFI_SUCCESS      The synchronous interrupt transfer is OK\r
-  @retval Others           Failed to execute the synchronous interrupt transfer\r
+  @retval EFI_SUCCESS      The synchronous interrupt transfer is OK.\r
+  @retval Others           Failed to execute the synchronous interrupt transfer.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -570,21 +564,21 @@ UsbHcSyncInterruptTransfer (
 \r
 \r
 /**\r
-  Execute a synchronous Isochronous USB transfer\r
+  Execute a synchronous Isochronous USB transfer.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The target device address\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The target device address.\r
   @param  EpAddr           The target endpoint address, with direction encoded in\r
-                           bit 7\r
-  @param  DevSpeed         The device's speed\r
-  @param  MaxPacket        The endpoint's max packet size\r
-  @param  BufferNum        The number of data buffer\r
-  @param  Data             Array of pointers to data buffer\r
-  @param  DataLength       The length of data buffer\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  UsbResult        The result of USB execution\r
+                           bit 7.\r
+  @param  DevSpeed         The device's speed.\r
+  @param  MaxPacket        The endpoint's max packet size.\r
+  @param  BufferNum        The number of data buffer.\r
+  @param  Data             Array of pointers to data buffer.\r
+  @param  DataLength       The length of data buffer.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  UsbResult        The result of USB execution.\r
 \r
-  @retval EFI_UNSUPPORTED  The isochronous transfer isn't supported now\r
+  @retval EFI_UNSUPPORTED  The isochronous transfer isn't supported now.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -606,22 +600,22 @@ UsbHcIsochronousTransfer (
 \r
 \r
 /**\r
-  Queue an asynchronous isochronous transfer\r
+  Queue an asynchronous isochronous transfer.\r
 \r
-  @param  UsbBus           The USB bus driver\r
-  @param  DevAddr          The target device address\r
+  @param  UsbBus           The USB bus driver.\r
+  @param  DevAddr          The target device address.\r
   @param  EpAddr           The target endpoint address, with direction encoded in\r
-                           bit 7\r
-  @param  DevSpeed         The device's speed\r
-  @param  MaxPacket        The endpoint's max packet size\r
-  @param  BufferNum        The number of data buffer\r
-  @param  Data             Array of pointers to data buffer\r
-  @param  DataLength       The length of data buffer\r
-  @param  Translator       The transaction translator for low/full speed device\r
-  @param  Callback         The function to call when data is transferred\r
-  @param  Context          The context to the callback function\r
-\r
-  @retval EFI_UNSUPPORTED  The asynchronous isochronous transfer isn't supported\r
+                           bit 7.\r
+  @param  DevSpeed         The device's speed.\r
+  @param  MaxPacket        The endpoint's max packet size.\r
+  @param  BufferNum        The number of data buffer.\r
+  @param  Data             Array of pointers to data buffer.\r
+  @param  DataLength       The length of data buffer.\r
+  @param  Translator       The transaction translator for low/full speed device.\r
+  @param  Callback         The function to call when data is transferred.\r
+  @param  Context          The context to the callback function.\r
+\r
+  @retval EFI_UNSUPPORTED  The asynchronous isochronous transfer isn't supported.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -644,12 +638,12 @@ UsbHcAsyncIsochronousTransfer (
 \r
 \r
 /**\r
-  Open the USB host controller protocol BY_CHILD\r
+  Open the USB host controller protocol BY_CHILD.\r
 \r
-  @param  Bus              The USB bus driver\r
-  @param  Child            The child handle\r
+  @param  Bus              The USB bus driver.\r
+  @param  Child            The child handle.\r
 \r
-  @return The open protocol return\r
+  @return The open protocol return.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -688,12 +682,12 @@ UsbOpenHostProtoByChild (
 \r
 \r
 /**\r
-  Close the USB host controller protocol BY_CHILD\r
+  Close the USB host controller protocol BY_CHILD.\r
 \r
-  @param  Bus              The USB bus driver\r
-  @param  Child            The child handle\r
+  @param  Bus              The USB bus driver.\r
+  @param  Child            The child handle.\r
 \r
-  @return None\r
+  @return None.\r
 \r
 **/\r
 VOID\r
@@ -721,13 +715,12 @@ UsbCloseHostProtoByChild (
 }\r
 \r
 \r
-\r
 /**\r
   return the current TPL, copied from the EDKII glue lib.\r
 \r
-  VOID\r
+  @param  VOID.\r
 \r
-  @return Current TPL\r
+  @return Current TPL.\r
 \r
 **/\r
 EFI_TPL\r
@@ -744,11 +737,11 @@ UsbGetCurrentTpl (
 }\r
 \r
 /**\r
-  Create a new device path which only contain the first Usb part of the DevicePath\r
+  Create a new device path which only contain the first Usb part of the DevicePath.\r
 \r
-  @param DevicePath  A full device path which contain the usb nodes\r
+  @param DevicePath  A full device path which contain the usb nodes.\r
 \r
-  @return            A new device path which only contain the Usb part of the DevicePath\r
+  @return            A new device path which only contain the Usb part of the DevicePath.\r
 \r
 **/\r
 EFI_DEVICE_PATH_PROTOCOL *\r
@@ -816,11 +809,11 @@ GetUsbDPFromFullDP (
 /**\r
   Check whether a usb device path is in a DEVICE_PATH_LIST_ITEM list.\r
 \r
-  @param UsbDP       a usb device path of DEVICE_PATH_LIST_ITEM\r
-  @parem UsbIoDPList a DEVICE_PATH_LIST_ITEM list\r
+  @param UsbDP       a usb device path of DEVICE_PATH_LIST_ITEM.\r
+  @param UsbIoDPList a DEVICE_PATH_LIST_ITEM list.\r
 \r
-  @retval TRUE       there is a DEVICE_PATH_LIST_ITEM in UsbIoDPList which contains the passed in UsbDP\r
-  @retval FALSE      there is no DEVICE_PATH_LIST_ITEM in UsbIoDPList which contains the passed in UsbDP\r
+  @retval TRUE       there is a DEVICE_PATH_LIST_ITEM in UsbIoDPList which contains the passed in UsbDP.\r
+  @retval FALSE      there is no DEVICE_PATH_LIST_ITEM in UsbIoDPList which contains the passed in UsbDP.\r
 \r
 **/\r
 BOOLEAN\r
@@ -867,11 +860,11 @@ SearchUsbDPInList (
 /**\r
   Add a usb device path into the DEVICE_PATH_LIST_ITEM list.\r
 \r
-  @param UsbDP        a usb device path of DEVICE_PATH_LIST_ITEM\r
-  @param UsbIoDPList  a DEVICE_PATH_LIST_ITEM list\r
+  @param UsbDP                   a usb device path of DEVICE_PATH_LIST_ITEM.\r
+  @param UsbIoDPList             a DEVICE_PATH_LIST_ITEM list.\r
 \r
-  @retval EFI_INVALID_PARAMETER\r
-  @retval EFI_SUCCESS\r
+  @retval EFI_INVALID_PARAMETER  If parameters are invalid, return this value.\r
+  @retval EFI_SUCCESS            If Add operation is successful, return this value.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -909,13 +902,13 @@ AddUsbDPToList (
 \r
 /**\r
   Check whether usb device, whose interface is UsbIf, matches the usb class which indicated by\r
-  UsbClassDevicePathPtr whose is a short form usb class device path\r
+  UsbClassDevicePathPtr whose is a short form usb class device path.\r
 \r
-  @param UsbClassDevicePathPtr    a short form usb class device path\r
-  @param UsbIf                    a usb device interface\r
+  @param UsbClassDevicePathPtr    a short form usb class device path.\r
+  @param UsbIf                    a usb device interface.\r
 \r
-  @retval TRUE                    the usb device match the usb class\r
-  @retval FALSE                   the usb device does not match the usb class\r
+  @retval TRUE                    the usb device match the usb class.\r
+  @retval FALSE                   the usb device does not match the usb class.\r
 \r
 **/\r
 BOOLEAN\r
@@ -989,16 +982,15 @@ MatchUsbClass (
 \r
 /**\r
   Check whether usb device, whose interface is UsbIf, matches the usb WWID requirement which indicated by\r
-  UsbWWIDDevicePathPtr whose is a short form usb WWID device path\r
+  UsbWWIDDevicePathPtr whose is a short form usb WWID device path.\r
 \r
-  @param UsbClassDevicePathPtr   a short form usb WWID device path\r
-  @param UsbIf                   a usb device interface\r
+  @param UsbWWIDDevicePathPtr    a short form usb WWID device path.\r
+  @param UsbIf                   a usb device interface.\r
 \r
-  @retval TRUE                   the usb device match the usb WWID requirement\r
-  @retval FALSE                  the usb device does not match the usb WWID requirement\r
+  @retval TRUE                   the usb device match the usb WWID requirement.\r
+  @retval FALSE                  the usb device does not match the usb WWID requirement.\r
 \r
 **/\r
-STATIC\r
 BOOLEAN\r
 MatchUsbWwid (\r
   IN USB_WWID_DEVICE_PATH       *UsbWWIDDevicePathPtr,\r
@@ -1047,12 +1039,12 @@ MatchUsbWwid (
 }\r
 \r
 /**\r
-  Free a DEVICE_PATH_LIST_ITEM list\r
+  Free a DEVICE_PATH_LIST_ITEM list.\r
 \r
-  @param UsbIoDPList  a DEVICE_PATH_LIST_ITEM list pointer\r
+  @param  UsbIoDPList            a DEVICE_PATH_LIST_ITEM list pointer.\r
 \r
-  @retval EFI_INVALID_PARAMETER\r
-  @retval EFI_SUCCESS\r
+  @retval EFI_INVALID_PARAMETER  If parameters are invalid, return this value.\r
+  @retval EFI_SUCCESS            If free operation is successful, return this value.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1094,14 +1086,13 @@ UsbBusFreeUsbDPList (
 \r
 /**\r
   Store a wanted usb child device info (its Usb part of device path) which is indicated by\r
-  RemainingDevicePath in a Usb bus which  is indicated by UsbBusId\r
+  RemainingDevicePath in a Usb bus which  is indicated by UsbBusId.\r
 \r
-  @param UsbBusId              Point to EFI_USB_BUS_PROTOCOL interface\r
-  @param RemainingDevicePath   The remaining device patch\r
+  @param  UsbBusId               Point to EFI_USB_BUS_PROTOCOL interface.\r
+  @param  RemainingDevicePath    The remaining device patch.\r
 \r
-  @retval EFI_SUCCESS\r
-  @retval EFI_INVALID_PARAMETER\r
-  @retval EFI_OUT_OF_RESOURCES\r
+  @retval EFI_SUCCESS            Add operation is successful.\r
+  @retval EFI_INVALID_PARAMETER  The parameters are invalid.\r
 \r
 **/\r
 EFI_STATUS\r
@@ -1158,14 +1149,13 @@ UsbBusAddWantedUsbIoDP (
 }\r
 \r
 /**\r
-  Check whether a usb child  device is the wanted device in a bus\r
+  Check whether a usb child device is the wanted device in a bus.\r
 \r
-  @param Bus      The Usb bus's private data pointer\r
-  @param UsbIf    The usb child  device inferface\r
+  @param  Bus     The Usb bus's private data pointer.\r
+  @param  UsbIf   The usb child device inferface.\r
 \r
-  @retval EFI_SUCCESS\r
-  @retval EFI_INVALID_PARAMETER\r
-  @retval EFI_OUT_OF_RESOURCES\r
+  @retval True    If a usb child device is the wanted device in a bus.\r
+  @retval False   If a usb child device is *NOT* the wanted device in a bus.\r
 \r
 **/\r
 BOOLEAN\r
@@ -1265,13 +1255,12 @@ UsbBusIsWantedUsbIO (
 \r
 /**\r
   Recursively connnect every wanted usb child device to ensure they all fully connected.\r
-  Check all the child Usb IO handles in this bus, recursively connecte if it is wanted usb child device\r
+  Check all the child Usb IO handles in this bus, recursively connecte if it is wanted usb child device.\r
 \r
-  @param UsbBusId   point to EFI_USB_BUS_PROTOCOL interface\r
+  @param  UsbBusId                  Point to EFI_USB_BUS_PROTOCOL interface.\r
 \r
-  @retval EFI_SUCCESS\r
-  @retval EFI_INVALID_PARAMETER\r
-  @retval EFI_OUT_OF_RESOURCES\r
+  @retval EFI_SUCCESS               Connect is done successfully.\r
+  @retval EFI_INVALID_PARAMETER     The parameter is invalid.\r
 \r
 **/\r
 EFI_STATUS\r