usb: storage: freecom: Fixed several coding style issues.
authorMartin Enderleit <martin@martin-desktop.(none)>
Sat, 10 Jul 2010 14:50:12 +0000 (16:50 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 10 Aug 2010 21:35:39 +0000 (14:35 -0700)
Fixed several coding style issues in freecom.c.

Signed-off-by: Martin Enderleit <menderleit@gmail.com>
Cc: Matthew Dharm <mdharm-usb@one-eyed-alien.net>
Cc: Daniel Mack <daniel@caiaq.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

drivers/usb/storage/freecom.c

index 54cc942..6542ca4 100644 (file)
@@ -269,7 +269,7 @@ static int freecom_transport(struct scsi_cmnd *srb, struct us_data *us)
        /* The firmware will time-out commands after 20 seconds. Some commands
         * can legitimately take longer than this, so we use a different
         * command that only waits for the interrupt and then sends status,
-        * without having to send a new ATAPI command to the device. 
+        * without having to send a new ATAPI command to the device.
         *
         * NOTE: There is some indication that a data transfer after a timeout
         * may not work, but that is a condition that should never happen.
@@ -324,14 +324,14 @@ static int freecom_transport(struct scsi_cmnd *srb, struct us_data *us)
 
        /* Find the length we desire to read. */
        switch (srb->cmnd[0]) {
-               case INQUIRY:
-               case REQUEST_SENSE:             /* 16 or 18 bytes? spec says 18, lots of devices only have 16 */
-               case MODE_SENSE:
-               case MODE_SENSE_10:
-                       length = le16_to_cpu(fst->Count);
-                       break;
-               default:
-                       length = scsi_bufflen(srb);
+       case INQUIRY:
+       case REQUEST_SENSE:     /* 16 or 18 bytes? spec says 18, lots of devices only have 16 */
+       case MODE_SENSE:
+       case MODE_SENSE_10:
+               length = le16_to_cpu(fst->Count);
+               break;
+       default:
+               length = scsi_bufflen(srb);
        }
 
        /* verify that this amount is legal */
@@ -414,7 +414,7 @@ static int freecom_transport(struct scsi_cmnd *srb, struct us_data *us)
                /* should never hit here -- filtered in usb.c */
                US_DEBUGP ("freecom unimplemented direction: %d\n",
                                us->srb->sc_data_direction);
-               // Return fail, SCSI seems to handle this better.
+               /* Return fail, SCSI seems to handle this better. */
                return USB_STOR_TRANSPORT_FAILED;
                break;
        }
@@ -494,8 +494,7 @@ static void pdump (void *ibuffer, int length)
                                offset = 0;
                        }
                        offset += sprintf (line+offset, "%08x:", i);
-               }
-               else if ((i & 7) == 0) {
+               } else if ((i & 7) == 0) {
                        offset += sprintf (line+offset, " -");
                }
                offset += sprintf (line+offset, " %02x", buffer[i] & 0xff);