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

staging: comedi: usbdux: tidy up usbduxsub_pwm_irq()

Rename the local variables to the comedi "norm".

Use dev->class_dev as the device for all dev_printk() messages.
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 e057288f
...@@ -1634,19 +1634,9 @@ static int usbdux_pwm_cancel(struct comedi_device *dev, ...@@ -1634,19 +1634,9 @@ static int usbdux_pwm_cancel(struct comedi_device *dev,
static void usbduxsub_pwm_irq(struct urb *urb) static void usbduxsub_pwm_irq(struct urb *urb)
{ {
struct comedi_device *dev = urb->context;
struct usbdux_private *devpriv = dev->private;
int ret; int ret;
struct usbdux_private *this_usbduxsub;
struct comedi_device *this_comedidev;
struct comedi_subdevice *s;
/* printk(KERN_DEBUG "PWM: IRQ\n"); */
/* the context variable points to the subdevice */
this_comedidev = urb->context;
/* the private structure of the subdevice is struct usbdux_private */
this_usbduxsub = this_comedidev->private;
s = &this_comedidev->subdevices[SUBDEV_DA];
switch (urb->status) { switch (urb->status) {
case 0: case 0:
...@@ -1661,42 +1651,41 @@ static void usbduxsub_pwm_irq(struct urb *urb) ...@@ -1661,42 +1651,41 @@ static void usbduxsub_pwm_irq(struct urb *urb)
* after an unlink command, unplug, ... etc * after an unlink command, unplug, ... etc
* no unlink needed here. Already shutting down. * no unlink needed here. Already shutting down.
*/ */
if (this_usbduxsub->pwm_cmd_running) if (devpriv->pwm_cmd_running)
usbdux_pwm_stop(this_usbduxsub, 0); usbdux_pwm_stop(devpriv, 0);
return; return;
default: default:
/* a real error */ /* a real error */
if (this_usbduxsub->pwm_cmd_running) { if (devpriv->pwm_cmd_running) {
dev_err(&this_usbduxsub->interface->dev, dev_err(dev->class_dev,
"comedi_: Non-zero urb status received in " "Non-zero urb status received in pwm intr context: %d\n",
"pwm intr context: %d\n", urb->status); urb->status);
usbdux_pwm_stop(this_usbduxsub, 0); usbdux_pwm_stop(devpriv, 0);
} }
return; return;
} }
/* are we actually running? */ /* are we actually running? */
if (!(this_usbduxsub->pwm_cmd_running)) if (!devpriv->pwm_cmd_running)
return; return;
urb->transfer_buffer_length = this_usbduxsub->size_pwm_buf; urb->transfer_buffer_length = devpriv->size_pwm_buf;
urb->dev = this_usbduxsub->usbdev; urb->dev = devpriv->usbdev;
urb->status = 0; urb->status = 0;
if (this_usbduxsub->pwm_cmd_running) { if (devpriv->pwm_cmd_running) {
ret = usb_submit_urb(urb, GFP_ATOMIC); ret = usb_submit_urb(urb, GFP_ATOMIC);
if (ret < 0) { if (ret < 0) {
dev_err(&this_usbduxsub->interface->dev, dev_err(dev->class_dev,
"comedi_: pwm urb resubm failed in int-cont. " "pwm urb resubm failed in int-cont. ret=%d",
"ret=%d", ret); ret);
if (ret == EL2NSYNC) if (ret == EL2NSYNC)
dev_err(&this_usbduxsub->interface->dev, dev_err(dev->class_dev,
"buggy USB host controller or bug in " "buggy USB host controller or bug in IRQ handling!\n");
"IRQ handling!\n");
/* don't do an unlink here */ /* don't do an unlink here */
usbdux_pwm_stop(this_usbduxsub, 0); usbdux_pwm_stop(devpriv, 0);
} }
} }
} }
......
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