staging: comedi: usbduxsigma: tidy up receive_dux_commands()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 23 May 2013 19:50:51 +0000 (12:50 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 30 May 2013 11:40:52 +0000 (20:40 +0900)
Rename the variable used for the private data pointer.

Remove dev_err() when usb_blk_msg() fails, it's just added noise.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/usbduxsigma.c

index cbe8e14a4c5055367a57076dcad2371a83977883..3d8f17a7a25719d795846e10d1f5624a6f14e010 100644 (file)
@@ -740,32 +740,29 @@ static int send_dux_commands(struct usbduxsigma_private *devpriv,
                            &nsent, BULK_TIMEOUT);
 }
 
-static int receive_dux_commands(struct usbduxsigma_private *this_usbduxsub,
+static int receive_dux_commands(struct usbduxsigma_private *devpriv,
                                int command)
 {
-       int result = (-EFAULT);
        int nrec;
+       int ret;
        int i;
 
        for (i = 0; i < RETRIES; i++) {
-               result = usb_bulk_msg(this_usbduxsub->usbdev,
-                                     usb_rcvbulkpipe(this_usbduxsub->usbdev,
-                                                     COMMAND_IN_EP),
-                                     this_usbduxsub->insnBuffer, SIZEINSNBUF,
-                                     &nrec, BULK_TIMEOUT);
-               if (result < 0) {
-                       dev_err(&this_usbduxsub->interface->dev, "comedi%d: "
-                               "insn: USB error %d "
-                               "while receiving DUX command"
-                               "\n", this_usbduxsub->comedidev->minor,
-                               result);
-                       return result;
-               }
-               if (this_usbduxsub->insnBuffer[0] == command)
-                       return result;
+               ret = usb_bulk_msg(devpriv->usbdev,
+                                  usb_rcvbulkpipe(devpriv->usbdev,
+                                                  COMMAND_IN_EP),
+                                  devpriv->insnBuffer, SIZEINSNBUF,
+                                  &nrec, BULK_TIMEOUT);
+               if (ret < 0)
+                       return ret;
+
+               if (devpriv->insnBuffer[0] == command)
+                       return 0;
        }
-       /* this is only reached if the data has been requested a couple of
-        * times */
+       /*
+        * This is only reached if the data has been requested a
+        * couple of times and the command was not received.
+        */
        return -EFAULT;
 }