diff options
author | H Hartley Sweeten <hsweeten@visionengravers.com> | 2013-02-06 00:30:01 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-02-06 02:07:13 (GMT) |
commit | 4c56a2b63979ec1992fddc087c4660fde7d8e6c0 (patch) | |
tree | edc0a91bff0a5b0eda19e2f9dbc37baa814b2f1a | |
parent | 6f25a527eac10d9ba277a2de7a2a9d3e92c08a52 (diff) | |
download | linux-fsl-qoriq-4c56a2b63979ec1992fddc087c4660fde7d8e6c0.tar.xz |
staging: comedi: vmk80xx: remove CONFIG_COMEDI_DEBUG code
If CONFIG_COMEDI_DEBUG is defined a macro is enabled to output some
printk(KERN_REBUG ...) messages. These are just added noise. Remove
it.
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>
-rw-r--r-- | drivers/staging/comedi/drivers/vmk80xx.c | 46 |
1 files changed, 8 insertions, 38 deletions
diff --git a/drivers/staging/comedi/drivers/vmk80xx.c b/drivers/staging/comedi/drivers/vmk80xx.c index 5ba9632..8ec5985 100644 --- a/drivers/staging/comedi/drivers/vmk80xx.c +++ b/drivers/staging/comedi/drivers/vmk80xx.c @@ -123,18 +123,6 @@ enum { #define URB_RCV_FLAG (1 << 0) #define URB_SND_FLAG (1 << 1) -#ifdef CONFIG_COMEDI_DEBUG -static int dbgcm = 1; -#else -static int dbgcm; -#endif - -#define dbgcm(fmt, arg...) \ -do { \ - if (dbgcm) \ - printk(KERN_DEBUG fmt, ##arg); \ -} while (0) - enum vmk80xx_model { VMK8055_MODEL, VMK8061_MODEL @@ -212,12 +200,6 @@ static void vmk80xx_tx_callback(struct urb *urb) { struct vmk80xx_private *devpriv = urb->context; unsigned long *flags = &devpriv->flags; - int stat = urb->status; - - if (stat && !(stat == -ENOENT - || stat == -ECONNRESET || stat == -ESHUTDOWN)) - dbgcm("comedi#: vmk80xx: %s - nonzero urb status (%d)\n", - __func__, stat); if (!test_bit(TRANS_OUT_BUSY, flags)) return; @@ -241,26 +223,16 @@ static void vmk80xx_rx_callback(struct urb *urb) case -ESHUTDOWN: break; default: - dbgcm("comedi#: vmk80xx: %s - nonzero urb status (%d)\n", - __func__, stat); - goto resubmit; - } - - goto exit; -resubmit: - if (test_bit(TRANS_IN_RUNNING, flags) && devpriv->intf) { - usb_anchor_urb(urb, &devpriv->rx_anchor); - - if (!usb_submit_urb(urb, GFP_KERNEL)) - goto exit; + /* Try to resubmit the urb */ + if (test_bit(TRANS_IN_RUNNING, flags) && devpriv->intf) { + usb_anchor_urb(urb, &devpriv->rx_anchor); - dev_err(&urb->dev->dev, - "comedi#: vmk80xx: %s - submit urb failed\n", - __func__); - - usb_unanchor_urb(urb); + if (usb_submit_urb(urb, GFP_KERNEL)) + usb_unanchor_urb(urb); + } + break; } -exit: + clear_bit(TRANS_IN_BUSY, flags); wake_up_interruptible(&devpriv->read_wait); @@ -1187,8 +1159,6 @@ static int vmk80xx_auto_attach(struct comedi_device *dev, if (vmk80xx_check_data_link(devpriv)) { vmk80xx_read_eeprom(devpriv, IC6_VERSION); dev_info(&intf->dev, "%s\n", devpriv->fw.ic6_vers); - } else { - dbgcm("comedi#: vmk80xx: no conn. to CPU\n"); } } |