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/usbdux.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/comedi/drivers/usbdux.c 
b/drivers/staging/comedi/drivers/usbdux.c
index 053bc50..ad0faf5 100644
--- a/drivers/staging/comedi/drivers/usbdux.c
+++ b/drivers/staging/comedi/drivers/usbdux.c
@@ -513,7 +513,7 @@ static void usbduxsub_ao_isoc_irq(struct urb *urb)
                        dev_err(dev->class_dev,
                                "ao urb resubm failed in int-cont. ret=%d",
                                ret);
-                       if (ret == EL2NSYNC)
+                       if (ret == -EL2NSYNC)
                                dev_err(dev->class_dev,
                                        "buggy USB host controller or bug in 
IRQ handling!\n");
 
@@ -1294,7 +1294,7 @@ static void usbduxsub_pwm_irq(struct urb *urb)
                        dev_err(dev->class_dev,
                                "pwm urb resubm failed in int-cont. ret=%d",
                                ret);
-                       if (ret == EL2NSYNC)
+                       if (ret == -EL2NSYNC)
                                dev_err(dev->class_dev,
                                        "buggy USB host controller or bug in 
IRQ handling!\n");
 
-- 
2.0.3

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

Reply via email to