USB: fix codingstyle issues in drivers/usb/core/message.c

Fixes a number of coding style issues in the message.c file.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

1 files changed, 204 insertions, 194 deletions

diff --git a/drivers/usb/core/message.c b/drivers/usb/core/message.c
index 58b8e1c6b9a6..fefb92296e8f 100644
--- a/drivers/usb/core/message.c
+++ b/drivers/usb/core/message.c
@@ -39,7 +39,7 @@ static void usb_api_blocking_completion(struct urb *urb)
  * own interruptible routines.
  */
 static int usb_start_wait_urb(struct urb *urb, int timeout, int *actual_length)
-{ 
+{
         struct api_context ctx;
         unsigned long expire;
         int retval;
@@ -74,9 +74,9 @@ out:
 }
 
 /*-------------------------------------------------------------------*/
-// returns status (negative) or length (positive)
+/* returns status (negative) or length (positive) */
 static int usb_internal_control_msg(struct usb_device *usb_dev,
-                                    unsigned int pipe, 
+                                    unsigned int pipe,
                                     struct usb_ctrlrequest *cmd,
                                     void *data, int len, int timeout)
 {
@@ -87,7 +87,7 @@ static int usb_internal_control_msg(struct usb_device *usb_dev,
         urb = usb_alloc_urb(0, GFP_NOIO);
         if (!urb)
                 return -ENOMEM;
-  
+
         usb_fill_control_urb(urb, usb_dev, pipe, (unsigned char *)cmd, data,
                              len, usb_api_blocking_completion, NULL);
 
@@ -99,47 +99,51 @@ static int usb_internal_control_msg(struct usb_device *usb_dev,
 }
 
 /**
- *      usb_control_msg - Builds a control urb, sends it off and waits for completion
- *      @dev: pointer to the usb device to send the message to
- *      @pipe: endpoint "pipe" to send the message to
- *      @request: USB message request value
- *      @requesttype: USB message request type value
- *      @value: USB message value
- *      @index: USB message index value
- *      @data: pointer to the data to send
- *      @size: length in bytes of the data to send
- *      @timeout: time in msecs to wait for the message to complete before
- *              timing out (if 0 the wait is forever)
- *      Context: !in_interrupt ()
- *
- *      This function sends a simple control message to a specified endpoint
- *      and waits for the message to complete, or timeout.
- *      
- *      If successful, it returns the number of bytes transferred, otherwise a negative error number.
- *
- *      Don't use this function from within an interrupt context, like a
- *      bottom half handler.  If you need an asynchronous message, or need to send
- *      a message from within interrupt context, use usb_submit_urb()
- *      If a thread in your driver uses this call, make sure your disconnect()
- *      method can wait for it to complete.  Since you don't have a handle on
- *      the URB used, you can't cancel the request.
+ * usb_control_msg - Builds a control urb, sends it off and waits for completion
+ * @dev: pointer to the usb device to send the message to
+ * @pipe: endpoint "pipe" to send the message to
+ * @request: USB message request value
+ * @requesttype: USB message request type value
+ * @value: USB message value
+ * @index: USB message index value
+ * @data: pointer to the data to send
+ * @size: length in bytes of the data to send
+ * @timeout: time in msecs to wait for the message to complete before timing
+ *      out (if 0 the wait is forever)
+ *
+ * Context: !in_interrupt ()
+ *
+ * This function sends a simple control message to a specified endpoint and
+ * waits for the message to complete, or timeout.
+ *
+ * If successful, it returns the number of bytes transferred, otherwise a
+ * negative error number.
+ *
+ * Don't use this function from within an interrupt context, like a bottom half
+ * handler.  If you need an asynchronous message, or need to send a message
+ * from within interrupt context, use usb_submit_urb().
+ * If a thread in your driver uses this call, make sure your disconnect()
+ * method can wait for it to complete.  Since you don't have a handle on the
+ * URB used, you can't cancel the request.
  */
-int usb_control_msg(struct usb_device *dev, unsigned int pipe, __u8 request, __u8 requesttype,
-                         __u16 value, __u16 index, void *data, __u16 size, int timeout)
+int usb_control_msg(struct usb_device *dev, unsigned int pipe, __u8 request,
+                    __u8 requesttype, __u16 value, __u16 index, void *data,
+                    __u16 size, int timeout)
 {
-        struct usb_ctrlrequest *dr = kmalloc(sizeof(struct usb_ctrlrequest), GFP_NOIO);
+        struct usb_ctrlrequest *dr;
         int ret;
-        
+
+        dr = kmalloc(sizeof(struct usb_ctrlrequest), GFP_NOIO);
         if (!dr)
                 return -ENOMEM;
 
-        dr->bRequestType= requesttype;
+        dr->bRequestType = requesttype;
         dr->bRequest = request;
         dr->wValue = cpu_to_le16p(&value);
         dr->wIndex = cpu_to_le16p(&index);
         dr->wLength = cpu_to_le16p(&size);
 
-        //dbg("usb_control_msg");       
+        /* dbg("usb_control_msg"); */
 
         ret = usb_internal_control_msg(dev, pipe, dr, data, size, timeout);
 
@@ -155,9 +159,11 @@ EXPORT_SYMBOL_GPL(usb_control_msg);
  * @pipe: endpoint "pipe" to send the message to
  * @data: pointer to the data to send
  * @len: length in bytes of the data to send
- * @actual_length: pointer to a location to put the actual length transferred in bytes
+ * @actual_length: pointer to a location to put the actual length transferred
+ *      in bytes
  * @timeout: time in msecs to wait for the message to complete before
  *      timing out (if 0 the wait is forever)
+ *
  * Context: !in_interrupt ()
  *
  * This function sends a simple interrupt message to a specified endpoint and
@@ -181,38 +187,38 @@ int usb_interrupt_msg(struct usb_device *usb_dev, unsigned int pipe,
 EXPORT_SYMBOL_GPL(usb_interrupt_msg);
 
 /**
- *      usb_bulk_msg - Builds a bulk urb, sends it off and waits for completion
- *      @usb_dev: pointer to the usb device to send the message to
- *      @pipe: endpoint "pipe" to send the message to
- *      @data: pointer to the data to send
- *      @len: length in bytes of the data to send
- *      @actual_length: pointer to a location to put the actual length transferred in bytes
- *      @timeout: time in msecs to wait for the message to complete before
- *              timing out (if 0 the wait is forever)
- *      Context: !in_interrupt ()
- *
- *      This function sends a simple bulk message to a specified endpoint
- *      and waits for the message to complete, or timeout.
- *      
- *      If successful, it returns 0, otherwise a negative error number.
- *      The number of actual bytes transferred will be stored in the 
- *      actual_length paramater.
- *
- *      Don't use this function from within an interrupt context, like a
- *      bottom half handler.  If you need an asynchronous message, or need to
- *      send a message from within interrupt context, use usb_submit_urb()
- *      If a thread in your driver uses this call, make sure your disconnect()
- *      method can wait for it to complete.  Since you don't have a handle on
- *      the URB used, you can't cancel the request.
- *
- *      Because there is no usb_interrupt_msg() and no USBDEVFS_INTERRUPT
- *      ioctl, users are forced to abuse this routine by using it to submit
- *      URBs for interrupt endpoints.  We will take the liberty of creating
- *      an interrupt URB (with the default interval) if the target is an
- *      interrupt endpoint.
+ * usb_bulk_msg - Builds a bulk urb, sends it off and waits for completion
+ * @usb_dev: pointer to the usb device to send the message to
+ * @pipe: endpoint "pipe" to send the message to
+ * @data: pointer to the data to send
+ * @len: length in bytes of the data to send
+ * @actual_length: pointer to a location to put the actual length transferred
+ *      in bytes
+ * @timeout: time in msecs to wait for the message to complete before
+ *      timing out (if 0 the wait is forever)
+ *
+ * Context: !in_interrupt ()
+ *
+ * This function sends a simple bulk message to a specified endpoint
+ * and waits for the message to complete, or timeout.
+ *
+ * If successful, it returns 0, otherwise a negative error number.  The number
+ * of actual bytes transferred will be stored in the actual_length paramater.
+ *
+ * Don't use this function from within an interrupt context, like a bottom half
+ * handler.  If you need an asynchronous message, or need to send a message
+ * from within interrupt context, use usb_submit_urb() If a thread in your
+ * driver uses this call, make sure your disconnect() method can wait for it to
+ * complete.  Since you don't have a handle on the URB used, you can't cancel
+ * the request.
+ *
+ * Because there is no usb_interrupt_msg() and no USBDEVFS_INTERRUPT ioctl,
+ * users are forced to abuse this routine by using it to submit URBs for
+ * interrupt endpoints.  We will take the liberty of creating an interrupt URB
+ * (with the default interval) if the target is an interrupt endpoint.
  */
-int usb_bulk_msg(struct usb_device *usb_dev, unsigned int pipe, 
-                        void *data, int len, int *actual_length, int timeout)
+int usb_bulk_msg(struct usb_device *usb_dev, unsigned int pipe,
+                 void *data, int len, int *actual_length, int timeout)
 {
         struct urb *urb;
         struct usb_host_endpoint *ep;
@@ -242,26 +248,26 @@ EXPORT_SYMBOL_GPL(usb_bulk_msg);
 
 /*-------------------------------------------------------------------*/
 
-static void sg_clean (struct usb_sg_request *io)
+static void sg_clean(struct usb_sg_request *io)
 {
         if (io->urbs) {
                 while (io->entries--)
-                        usb_free_urb (io->urbs [io->entries]);
-                kfree (io->urbs);
+                        usb_free_urb(io->urbs [io->entries]);
+                kfree(io->urbs);
                 io->urbs = NULL;
         }
         if (io->dev->dev.dma_mask != NULL)
-                usb_buffer_unmap_sg (io->dev, usb_pipein(io->pipe),
-                                io->sg, io->nents);
+                usb_buffer_unmap_sg(io->dev, usb_pipein(io->pipe),
+                                    io->sg, io->nents);
         io->dev = NULL;
 }
 
-static void sg_complete (struct urb *urb)
+static void sg_complete(struct urb *urb)
 {
-        struct usb_sg_request   *io = urb->context;
+        struct usb_sg_request *io = urb->context;
         int status = urb->status;
 
-        spin_lock (&io->lock);
+        spin_lock(&io->lock);
 
         /* In 2.5 we require hcds' endpoint queues not to progress after fault
          * reports, until the completion callback (this!) returns.  That lets
@@ -277,13 +283,13 @@ static void sg_complete (struct urb *urb)
                         && (io->status != -ECONNRESET
                                 || status != -ECONNRESET)
                         && urb->actual_length) {
-                dev_err (io->dev->bus->controller,
+                dev_err(io->dev->bus->controller,
                         "dev %s ep%d%s scatterlist error %d/%d\n",
                         io->dev->devpath,
                         usb_endpoint_num(&urb->ep->desc),
                         usb_urb_dir_in(urb) ? "in" : "out",
                         status, io->status);
-                // BUG ();
+                /* BUG (); */
         }
 
         if (io->status == 0 && status && status != -ECONNRESET) {
@@ -295,22 +301,22 @@ static void sg_complete (struct urb *urb)
                  * unlink pending urbs so they won't rx/tx bad data.
                  * careful: unlink can sometimes be synchronous...
                  */
-                spin_unlock (&io->lock);
+                spin_unlock(&io->lock);
                 for (i = 0, found = 0; i < io->entries; i++) {
                         if (!io->urbs [i] || !io->urbs [i]->dev)
                                 continue;
                         if (found) {
-                                retval = usb_unlink_urb (io->urbs [i]);
+                                retval = usb_unlink_urb(io->urbs [i]);
                                 if (retval != -EINPROGRESS &&
                                     retval != -ENODEV &&
                                     retval != -EBUSY)
-                                        dev_err (&io->dev->dev,
+                                        dev_err(&io->dev->dev,
                                                 "%s, unlink --> %d\n",
                                                 __FUNCTION__, retval);
                         } else if (urb == io->urbs [i])
                                 found = 1;
                 }
-                spin_lock (&io->lock);
+                spin_lock(&io->lock);
         }
         urb->dev = NULL;
 
@@ -318,9 +324,9 @@ static void sg_complete (struct urb *urb)
         io->bytes += urb->actual_length;
         io->count--;
         if (!io->count)
-                complete (&io->complete);
+                complete(&io->complete);
 
-        spin_unlock (&io->lock);
+        spin_unlock(&io->lock);
 }
 
 
@@ -349,28 +355,21 @@ static void sg_complete (struct urb *urb)
  * The request may be canceled with usb_sg_cancel(), either before or after
  * usb_sg_wait() is called.
  */
-int usb_sg_init (
-        struct usb_sg_request   *io,
-        struct usb_device       *dev,
-        unsigned                pipe, 
-        unsigned                period,
-        struct scatterlist      *sg,
-        int                     nents,
-        size_t                  length,
-        gfp_t                   mem_flags
-)
+int usb_sg_init(struct usb_sg_request *io, struct usb_device *dev,
+                unsigned pipe, unsigned period, struct scatterlist *sg,
+                int nents, size_t length, gfp_t mem_flags)
 {
-        int                     i;
-        int                     urb_flags;
-        int                     dma;
+        int i;
+        int urb_flags;
+        int dma;
 
         if (!io || !dev || !sg
-                        || usb_pipecontrol (pipe)
-                        || usb_pipeisoc (pipe)
+                        || usb_pipecontrol(pipe)
+                        || usb_pipeisoc(pipe)
                         || nents <= 0)
                 return -EINVAL;
 
-        spin_lock_init (&io->lock);
+        spin_lock_init(&io->lock);
         io->dev = dev;
         io->pipe = pipe;
         io->sg = sg;
@@ -382,7 +381,7 @@ int usb_sg_init (
         dma = (dev->dev.dma_mask != NULL);
         if (dma)
                 io->entries = usb_buffer_map_sg(dev, usb_pipein(pipe),
-                                sg, nents);
+                                                sg, nents);
         else
                 io->entries = nents;
 
@@ -391,30 +390,30 @@ int usb_sg_init (
                 return io->entries;
 
         io->count = io->entries;
-        io->urbs = kmalloc (io->entries * sizeof *io->urbs, mem_flags);
+        io->urbs = kmalloc(io->entries * sizeof *io->urbs, mem_flags);
         if (!io->urbs)
                 goto nomem;
 
         urb_flags = URB_NO_TRANSFER_DMA_MAP | URB_NO_INTERRUPT;
-        if (usb_pipein (pipe))
+        if (usb_pipein(pipe))
                 urb_flags |= URB_SHORT_NOT_OK;
 
         for (i = 0; i < io->entries; i++) {
-                unsigned                len;
+                unsigned len;
 
-                io->urbs [i] = usb_alloc_urb (0, mem_flags);
-                if (!io->urbs [i]) {
+                io->urbs[i] = usb_alloc_urb(0, mem_flags);
+                if (!io->urbs[i]) {
                         io->entries = i;
                         goto nomem;
                 }
 
-                io->urbs [i]->dev = NULL;
-                io->urbs [i]->pipe = pipe;
-                io->urbs [i]->interval = period;
-                io->urbs [i]->transfer_flags = urb_flags;
+                io->urbs[i]->dev = NULL;
+                io->urbs[i]->pipe = pipe;
+                io->urbs[i]->interval = period;
+                io->urbs[i]->transfer_flags = urb_flags;
 
-                io->urbs [i]->complete = sg_complete;
-                io->urbs [i]->context = io;
+                io->urbs[i]->complete = sg_complete;
+                io->urbs[i]->context = io;
 
                 /*
                  * Some systems need to revert to PIO when DMA is temporarily
@@ -433,8 +432,8 @@ int usb_sg_init (
                  * to prevent stale pointers and to help spot bugs.
                  */
                 if (dma) {
-                        io->urbs [i]->transfer_dma = sg_dma_address (sg + i);
-                        len = sg_dma_len (sg + i);
+                        io->urbs[i]->transfer_dma = sg_dma_address(sg + i);
+                        len = sg_dma_len(sg + i);
 #if defined(CONFIG_HIGHMEM) || defined(CONFIG_GART_IOMMU)
                         io->urbs[i]->transfer_buffer = NULL;
 #else
@@ -442,28 +441,28 @@ int usb_sg_init (
 #endif
                 } else {
                         /* hc may use _only_ transfer_buffer */
-                        io->urbs [i]->transfer_buffer = sg_virt(&sg[i]);
-                        len = sg [i].length;
+                        io->urbs[i]->transfer_buffer = sg_virt(&sg[i]);
+                        len = sg[i].length;
                 }
 
                 if (length) {
-                        len = min_t (unsigned, len, length);
+                        len = min_t(unsigned, len, length);
                         length -= len;
                         if (length == 0)
                                 io->entries = i + 1;
                 }
-                io->urbs [i]->transfer_buffer_length = len;
+                io->urbs[i]->transfer_buffer_length = len;
         }
-        io->urbs [--i]->transfer_flags &= ~URB_NO_INTERRUPT;
+        io->urbs[--i]->transfer_flags &= ~URB_NO_INTERRUPT;
 
         /* transaction state */
         io->status = 0;
         io->bytes = 0;
-        init_completion (&io->complete);
+        init_completion(&io->complete);
         return 0;
 
 nomem:
-        sg_clean (io);
+        sg_clean(io);
         return -ENOMEM;
 }
 EXPORT_SYMBOL_GPL(usb_sg_init);
@@ -507,31 +506,32 @@ EXPORT_SYMBOL_GPL(usb_sg_init);
  * speed interrupt endpoints, which allow at most one packet per millisecond,
  * of at most 8 or 64 bytes (respectively).
  */
-void usb_sg_wait (struct usb_sg_request *io)
+void usb_sg_wait(struct usb_sg_request *io)
 {
-        int             i, entries = io->entries;
+        int i;
+        int entries = io->entries;
 
         /* queue the urbs.  */
-        spin_lock_irq (&io->lock);
+        spin_lock_irq(&io->lock);
         i = 0;
         while (i < entries && !io->status) {
-                int     retval;
+                int retval;
 
-                io->urbs [i]->dev = io->dev;
-                retval = usb_submit_urb (io->urbs [i], GFP_ATOMIC);
+                io->urbs[i]->dev = io->dev;
+                retval = usb_submit_urb(io->urbs [i], GFP_ATOMIC);
 
                 /* after we submit, let completions or cancelations fire;
                  * we handshake using io->status.
                  */
-                spin_unlock_irq (&io->lock);
+                spin_unlock_irq(&io->lock);
                 switch (retval) {
                         /* maybe we retrying will recover */
-                case -ENXIO:    // hc didn't queue this one
+                case -ENXIO:    /* hc didn't queue this one */
                 case -EAGAIN:
                 case -ENOMEM:
                         io->urbs[i]->dev = NULL;
                         retval = 0;
-                        yield ();
+                        yield();
                         break;
 
                         /* no error? continue immediately.
@@ -542,33 +542,33 @@ void usb_sg_wait (struct usb_sg_request *io)
                          */
                 case 0:
                         ++i;
-                        cpu_relax ();
+                        cpu_relax();
                         break;
 
                         /* fail any uncompleted urbs */
                 default:
-                        io->urbs [i]->dev = NULL;
-                        io->urbs [i]->status = retval;
-                        dev_dbg (&io->dev->dev, "%s, submit --> %d\n",
+                        io->urbs[i]->dev = NULL;
+                        io->urbs[i]->status = retval;
+                        dev_dbg(&io->dev->dev, "%s, submit --> %d\n",
                                 __FUNCTION__, retval);
-                        usb_sg_cancel (io);
+                        usb_sg_cancel(io);
                 }
-                spin_lock_irq (&io->lock);
+                spin_lock_irq(&io->lock);
                 if (retval && (io->status == 0 || io->status == -ECONNRESET))
                         io->status = retval;
         }
         io->count -= entries - i;
         if (io->count == 0)
-                complete (&io->complete);
-        spin_unlock_irq (&io->lock);
+                complete(&io->complete);
+        spin_unlock_irq(&io->lock);
 
         /* OK, yes, this could be packaged as non-blocking.
          * So could the submit loop above ... but it's easier to
          * solve neither problem than to solve both!
          */
-        wait_for_completion (&io->complete);
+        wait_for_completion(&io->complete);
 
-        sg_clean (io);
+        sg_clean(io);
 }
 EXPORT_SYMBOL_GPL(usb_sg_wait);
 
@@ -580,31 +580,31 @@ EXPORT_SYMBOL_GPL(usb_sg_wait);
  * It can also prevents one initialized by usb_sg_init() from starting,
  * so that call just frees resources allocated to the request.
  */
-void usb_sg_cancel (struct usb_sg_request *io)
+void usb_sg_cancel(struct usb_sg_request *io)
 {
-        unsigned long   flags;
+        unsigned long flags;
 
-        spin_lock_irqsave (&io->lock, flags);
+        spin_lock_irqsave(&io->lock, flags);
 
         /* shut everything down, if it didn't already */
         if (!io->status) {
-                int     i;
+                int i;
 
                 io->status = -ECONNRESET;
-                spin_unlock (&io->lock);
+                spin_unlock(&io->lock);
                 for (i = 0; i < io->entries; i++) {
-                        int     retval;
+                        int retval;
 
                         if (!io->urbs [i]->dev)
                                 continue;
-                        retval = usb_unlink_urb (io->urbs [i]);
+                        retval = usb_unlink_urb(io->urbs [i]);
                         if (retval != -EINPROGRESS && retval != -EBUSY)
-                                dev_warn (&io->dev->dev, "%s, unlink --> %d\n",
+                                dev_warn(&io->dev->dev, "%s, unlink --> %d\n",
                                         __FUNCTION__, retval);
                 }
-                spin_lock (&io->lock);
+                spin_lock(&io->lock);
         }
-        spin_unlock_irqrestore (&io->lock, flags);
+        spin_unlock_irqrestore(&io->lock, flags);
 }
 EXPORT_SYMBOL_GPL(usb_sg_cancel);
 
@@ -632,12 +632,13 @@ EXPORT_SYMBOL_GPL(usb_sg_cancel);
  * Returns the number of bytes received on success, or else the status code
  * returned by the underlying usb_control_msg() call.
  */
-int usb_get_descriptor(struct usb_device *dev, unsigned char type, unsigned char index, void *buf, int size)
+int usb_get_descriptor(struct usb_device *dev, unsigned char type,
+                       unsigned char index, void *buf, int size)
 {
         int i;
         int result;
-        
-        memset(buf,0,size);     // Make sure we parse really received data
+
+        memset(buf, 0, size);   /* Make sure we parse really received data */
 
         for (i = 0; i < 3; ++i) {
                 /* retry on length 0 or error; some devices are flakey */
@@ -712,7 +713,7 @@ static void usb_try_string_workarounds(unsigned char *buf, int *length)
 }
 
 static int usb_string_sub(struct usb_device *dev, unsigned int langid,
-                unsigned int index, unsigned char *buf)
+                          unsigned int index, unsigned char *buf)
 {
         int rc;
 
@@ -755,7 +756,7 @@ static int usb_string_sub(struct usb_device *dev, unsigned int langid,
  * @buf: where to put the string
  * @size: how big is "buf"?
  * Context: !in_interrupt ()
- * 
+ *
  * This converts the UTF-16LE encoded strings returned by devices, from
  * usb_get_string_descriptor(), to null-terminated ISO-8859-1 encoded ones
  * that are more usable in most kernel contexts.  Note that all characters
@@ -791,23 +792,23 @@ int usb_string(struct usb_device *dev, int index, char *buf, size_t size)
         if (!dev->have_langid) {
                 err = usb_string_sub(dev, 0, 0, tbuf);
                 if (err < 0) {
-                        dev_err (&dev->dev,
+                        dev_err(&dev->dev,
                                 "string descriptor 0 read error: %d\n",
                                 err);
                         goto errout;
                 } else if (err < 4) {
-                        dev_err (&dev->dev, "string descriptor 0 too short\n");
+                        dev_err(&dev->dev, "string descriptor 0 too short\n");
                         err = -EINVAL;
                         goto errout;
                 } else {
                         dev->have_langid = 1;
-                        dev->string_langid = tbuf[2] | (tbuf[3]<< 8);
-                                /* always use the first langid listed */
-                        dev_dbg (&dev->dev, "default language 0x%04x\n",
+                        dev->string_langid = tbuf[2] | (tbuf[3] << 8);
+                        /* always use the first langid listed */
+                        dev_dbg(&dev->dev, "default language 0x%04x\n",
                                 dev->string_langid);
                 }
         }
-        
+
         err = usb_string_sub(dev, dev->string_langid, index, tbuf);
         if (err < 0)
                 goto errout;
@@ -825,7 +826,9 @@ int usb_string(struct usb_device *dev, int index, char *buf, size_t size)
         err = idx;
 
         if (tbuf[1] != USB_DT_STRING)
-                dev_dbg(&dev->dev, "wrong descriptor type %02x for string %d (\"%s\")\n", tbuf[1], index, buf);
+                dev_dbg(&dev->dev,
+                        "wrong descriptor type %02x for string %d (\"%s\")\n",
+                        tbuf[1], index, buf);
 
  errout:
         kfree(tbuf);
@@ -847,9 +850,15 @@ char *usb_cache_string(struct usb_device *udev, int index)
         char *smallbuf = NULL;
         int len;
 
-        if (index > 0 && (buf = kmalloc(256, GFP_KERNEL)) != NULL) {
-                if ((len = usb_string(udev, index, buf, 256)) > 0) {
-                        if ((smallbuf = kmalloc(++len, GFP_KERNEL)) == NULL)
+        if (index <= 0)
+                return NULL;
+
+        buf = kmalloc(256, GFP_KERNEL);
+        if (buf) {
+                len = usb_string(udev, index, buf, 256);
+                if (len > 0) {
+                        smallbuf = kmalloc(++len, GFP_KERNEL);
+                        if (!smallbuf)
                                 return buf;
                         memcpy(smallbuf, buf, len);
                 }
@@ -888,7 +897,7 @@ int usb_get_device_descriptor(struct usb_device *dev, unsigned int size)
                 return -ENOMEM;
 
         ret = usb_get_descriptor(dev, USB_DT_DEVICE, 0, desc, size);
-        if (ret >= 0) 
+        if (ret >= 0)
                 memcpy(&dev->descriptor, desc, size);
         kfree(desc);
         return ret;
@@ -961,8 +970,8 @@ int usb_clear_halt(struct usb_device *dev, int pipe)
 {
         int result;
         int endp = usb_pipeendpoint(pipe);
-        
-        if (usb_pipein (pipe))
+
+        if (usb_pipein(pipe))
                 endp |= USB_DIR_IN;
 
         /* we don't care if it wasn't halted first. in fact some devices
@@ -1045,7 +1054,7 @@ void usb_disable_interface(struct usb_device *dev, struct usb_interface *intf)
         }
 }
 
-/*
+/**
  * usb_disable_device - Disable all the endpoints for a USB device
  * @dev: the device whose endpoints are being disabled
  * @skip_ep0: 0 to disable endpoint 0, 1 to skip it.
@@ -1060,7 +1069,7 @@ void usb_disable_device(struct usb_device *dev, int skip_ep0)
         int i;
 
         dev_dbg(&dev->dev, "%s nuking %s URBs\n", __FUNCTION__,
-                        skip_ep0 ? "non-ep0" : "all");
+                skip_ep0 ? "non-ep0" : "all");
         for (i = skip_ep0; i < 16; ++i) {
                 usb_disable_endpoint(dev, i);
                 usb_disable_endpoint(dev, i + USB_DIR_IN);
@@ -1078,17 +1087,17 @@ void usb_disable_device(struct usb_device *dev, int skip_ep0)
                         interface = dev->actconfig->interface[i];
                         if (!device_is_registered(&interface->dev))
                                 continue;
-                        dev_dbg (&dev->dev, "unregistering interface %s\n",
+                        dev_dbg(&dev->dev, "unregistering interface %s\n",
                                 interface->dev.bus_id);
                         usb_remove_sysfs_intf_files(interface);
-                        device_del (&interface->dev);
+                        device_del(&interface->dev);
                 }
 
                 /* Now that the interfaces are unbound, nobody should
                  * try to access them.
                  */
                 for (i = 0; i < dev->actconfig->desc.bNumInterfaces; i++) {
-                        put_device (&dev->actconfig->interface[i]->dev);
+                        put_device(&dev->actconfig->interface[i]->dev);
                         dev->actconfig->interface[i] = NULL;
                 }
                 dev->actconfig = NULL;
@@ -1097,8 +1106,7 @@ void usb_disable_device(struct usb_device *dev, int skip_ep0)
         }
 }
 
-
-/*
+/**
  * usb_enable_endpoint - Enable an endpoint for USB communications
  * @dev: the device whose interface is being enabled
  * @ep: the endpoint
@@ -1123,7 +1131,7 @@ void usb_enable_endpoint(struct usb_device *dev, struct usb_host_endpoint *ep)
         ep->enabled = 1;
 }
 
-/*
+/**
  * usb_enable_interface - Enable all the endpoints for an interface
  * @dev: the device whose interface is being enabled
  * @intf: pointer to the interface descriptor
@@ -1179,6 +1187,8 @@ int usb_set_interface(struct usb_device *dev, int interface, int alternate)
         struct usb_host_interface *alt;
         int ret;
         int manual = 0;
+        unsigned int epaddr;
+        unsigned int pipe;
 
         if (dev->state == USB_STATE_SUSPENDED)
                 return -EHOSTUNREACH;
@@ -1233,11 +1243,11 @@ int usb_set_interface(struct usb_device *dev, int interface, int alternate)
                 int i;
 
                 for (i = 0; i < alt->desc.bNumEndpoints; i++) {
-                        unsigned int epaddr =
-                                alt->endpoint[i].desc.bEndpointAddress;
-                        unsigned int pipe =
-        __create_pipe(dev, USB_ENDPOINT_NUMBER_MASK & epaddr)
-        | (usb_endpoint_out(epaddr) ? USB_DIR_OUT : USB_DIR_IN);
+                        epaddr = alt->endpoint[i].desc.bEndpointAddress;
+                        pipe = __create_pipe(dev,
+                                        USB_ENDPOINT_NUMBER_MASK & epaddr) |
+                                        (usb_endpoint_out(epaddr) ?
+                                        USB_DIR_OUT : USB_DIR_IN);
 
                         usb_clear_halt(dev, pipe);
                 }
@@ -1366,7 +1376,8 @@ static int usb_if_uevent(struct device *dev, struct kobj_uevent_env *env)
                 return -ENOMEM;
 
         if (add_uevent_var(env,
-                   "MODALIAS=usb:v%04Xp%04Xd%04Xdc%02Xdsc%02Xdp%02Xic%02Xisc%02Xip%02X",
+                   "MODALIAS=usb:"
+                   "v%04Xp%04Xd%04Xdc%02Xdsc%02Xdp%02Xic%02Xisc%02Xip%02X",
                    le16_to_cpu(usb_dev->descriptor.idVendor),
                    le16_to_cpu(usb_dev->descriptor.idProduct),
                    le16_to_cpu(usb_dev->descriptor.bcdDevice),
@@ -1396,8 +1407,8 @@ struct device_type usb_if_device_type = {
 };
 
 static struct usb_interface_assoc_descriptor *find_iad(struct usb_device *dev,
-                                                       struct usb_host_config *config,
-                                                       u8 inum)
+                                                struct usb_host_config *config,
+                                                u8 inum)
 {
         struct usb_interface_assoc_descriptor *retval = NULL;
         struct usb_interface_assoc_descriptor *intf_assoc;
@@ -1424,7 +1435,6 @@ static struct usb_interface_assoc_descriptor *find_iad(struct usb_device *dev,
         return retval;
 }
 
-
 /*
  * usb_set_configuration - Makes a particular device setting be current
  * @dev: the device whose configuration is being updated
@@ -1542,12 +1552,12 @@ free_interfaces:
          * getting rid of old interfaces means unbinding their drivers.
          */
         if (dev->state != USB_STATE_ADDRESS)
-                usb_disable_device (dev, 1);    // Skip ep0
-
-        if ((ret = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
-                        USB_REQ_SET_CONFIGURATION, 0, configuration, 0,
-                        NULL, 0, USB_CTRL_SET_TIMEOUT)) < 0) {
+                usb_disable_device(dev, 1);     /* Skip ep0 */
 
+        ret = usb_control_msg(dev, usb_sndctrlpipe(dev, 0),
+                              USB_REQ_SET_CONFIGURATION, 0, configuration, 0,
+                              NULL, 0, USB_CTRL_SET_TIMEOUT);
+        if (ret < 0) {
                 /* All the old state is gone, so what else can we do?
                  * The device is probably useless now anyway.
                  */
@@ -1594,11 +1604,11 @@ free_interfaces:
                 intf->dev.bus = &usb_bus_type;
                 intf->dev.type = &usb_if_device_type;
                 intf->dev.dma_mask = dev->dev.dma_mask;
-                device_initialize (&intf->dev);
+                device_initialize(&intf->dev);
                 mark_quiesced(intf);
-                sprintf (&intf->dev.bus_id[0], "%d-%s:%d.%d",
-                         dev->bus->busnum, dev->devpath,
-                         configuration, alt->desc.bInterfaceNumber);
+                sprintf(&intf->dev.bus_id[0], "%d-%s:%d.%d",
+                        dev->bus->busnum, dev->devpath,
+                        configuration, alt->desc.bInterfaceNumber);
         }
         kfree(new_interfaces);
 
@@ -1614,11 +1624,11 @@ free_interfaces:
         for (i = 0; i < nintf; ++i) {
                 struct usb_interface *intf = cp->interface[i];
 
-                dev_dbg (&dev->dev,
+                dev_dbg(&dev->dev,
                         "adding %s (config #%d, interface %d)\n",
                         intf->dev.bus_id, configuration,
                         intf->cur_altsetting->desc.bInterfaceNumber);
-                ret = device_add (&intf->dev);
+                ret = device_add(&intf->dev);
                 if (ret != 0) {
                         dev_err(&dev->dev, "device_add(%s) --> %d\n",
                                 intf->dev.bus_id, ret);