These should both be checking the -errno.

Reported by: coverity
Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com>
Cc: Ian Abbott <abbo...@mev.co.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/comedi/drivers/usbduxsigma.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/comedi/drivers/usbduxsigma.c 
b/drivers/staging/comedi/drivers/usbduxsigma.c
index 94a09c1..da33b8c 100644
--- a/drivers/staging/comedi/drivers/usbduxsigma.c
+++ b/drivers/staging/comedi/drivers/usbduxsigma.c
@@ -451,7 +451,7 @@ static void usbduxsigma_ao_urb_complete(struct urb *urb)
                dev_err(dev->class_dev,
                        "%s: urb resubmit failed (%d)\n",
                        __func__, ret);
-               if (ret == EL2NSYNC)
+               if (ret == -EL2NSYNC)
                        dev_err(dev->class_dev,
                                "buggy USB host controller or bug in IRQ 
handler\n");
                usbduxsigma_ao_stop(dev, 0);    /* w/o unlink */
@@ -1148,7 +1148,7 @@ static void usbduxsigma_pwm_urb_complete(struct urb *urb)
        if (ret < 0) {
                dev_err(dev->class_dev, "%s: urb resubmit failed (%d)\n",
                        __func__, ret);
-               if (ret == EL2NSYNC)
+               if (ret == -EL2NSYNC)
                        dev_err(dev->class_dev,
                                "buggy USB host controller or bug in IRQ 
handler\n");
                usbduxsigma_pwm_stop(dev, 0);   /* w/o unlink */
-- 
2.0.3

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to