Commit fb908568 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: usbdux: tidy up receive_dux_commands()

For aesthetic reasons, pass the comedi_device pointer to this function
instead of the private data pointer. Rename the local variable used
for the private data pointer to the comedi "norm".

Add a local variable for the usb_device pointer to tidy up the
usb_bulk_msg() call.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 49cc49dd
...@@ -860,25 +860,24 @@ static int send_dux_commands(struct comedi_device *dev, int cmd_type) ...@@ -860,25 +860,24 @@ static int send_dux_commands(struct comedi_device *dev, int cmd_type)
&nsent, BULK_TIMEOUT); &nsent, BULK_TIMEOUT);
} }
static int receive_dux_commands(struct usbdux_private *this_usbduxsub, int command) static int receive_dux_commands(struct comedi_device *dev, int command)
{ {
int result = (-EFAULT); struct usbdux_private *devpriv = dev->private;
struct usb_device *usb = devpriv->usbdev;
int ret;
int nrec; int nrec;
int i; int i;
for (i = 0; i < RETRIES; i++) { for (i = 0; i < RETRIES; i++) {
result = usb_bulk_msg(this_usbduxsub->usbdev, ret = usb_bulk_msg(usb, usb_rcvbulkpipe(usb, COMMAND_IN_EP),
usb_rcvbulkpipe(this_usbduxsub->usbdev, devpriv->insn_buffer, SIZEINSNBUF,
COMMAND_IN_EP),
this_usbduxsub->insn_buffer, SIZEINSNBUF,
&nrec, BULK_TIMEOUT); &nrec, BULK_TIMEOUT);
if (result < 0) if (ret < 0)
return result; return ret;
if (le16_to_cpu(this_usbduxsub->insn_buffer[0]) == command) if (le16_to_cpu(devpriv->insn_buffer[0]) == command)
return result; return ret;
} }
/* this is only reached if the data has been requested a couple of /* command not received */
* times */
return -EFAULT; return -EFAULT;
} }
...@@ -1037,7 +1036,7 @@ static int usbdux_ai_insn_read(struct comedi_device *dev, ...@@ -1037,7 +1036,7 @@ static int usbdux_ai_insn_read(struct comedi_device *dev,
} }
for (i = 0; i < insn->n; i++) { for (i = 0; i < insn->n; i++) {
err = receive_dux_commands(this_usbduxsub, SENDSINGLEAD); err = receive_dux_commands(dev, SENDSINGLEAD);
if (err < 0) { if (err < 0) {
up(&this_usbduxsub->sem); up(&this_usbduxsub->sem);
return 0; return 0;
...@@ -1362,7 +1361,7 @@ static int usbdux_dio_insn_bits(struct comedi_device *dev, ...@@ -1362,7 +1361,7 @@ static int usbdux_dio_insn_bits(struct comedi_device *dev,
up(&this_usbduxsub->sem); up(&this_usbduxsub->sem);
return err; return err;
} }
err = receive_dux_commands(this_usbduxsub, SENDDIOBITSCOMMAND); err = receive_dux_commands(dev, SENDDIOBITSCOMMAND);
if (err < 0) { if (err < 0) {
up(&this_usbduxsub->sem); up(&this_usbduxsub->sem);
return err; return err;
...@@ -1392,7 +1391,7 @@ static int usbdux_counter_read(struct comedi_device *dev, ...@@ -1392,7 +1391,7 @@ static int usbdux_counter_read(struct comedi_device *dev,
return err; return err;
} }
err = receive_dux_commands(this_usbduxsub, READCOUNTERCOMMAND); err = receive_dux_commands(dev, READCOUNTERCOMMAND);
if (err < 0) { if (err < 0) {
up(&this_usbduxsub->sem); up(&this_usbduxsub->sem);
return err; return err;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment