[media] em28xx: remove double checks for urb->status == -ENOENT in urb_data_copy...
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Thu, 8 Nov 2012 17:11:44 +0000 (14:11 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 22 Dec 2012 20:00:59 +0000 (18:00 -0200)
This check is already done in the URB handler
em28xx_irq_callback before calling these functions.

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/usb/em28xx/em28xx-dvb.c
drivers/media/usb/em28xx/em28xx-video.c

index fe4d11c..7583cb7 100644 (file)
@@ -134,11 +134,8 @@ static inline int em28xx_dvb_isoc_copy(struct em28xx *dev, struct urb *urb)
        if ((dev->state & DEV_DISCONNECTED) || (dev->state & DEV_MISCONFIGURED))
                return 0;
 
-       if (urb->status < 0) {
+       if (urb->status < 0)
                print_err_status(dev, -1, urb->status);
-               if (urb->status == -ENOENT)
-                       return 0;
-       }
 
        for (i = 0; i < urb->number_of_packets; i++) {
                int status = urb->iso_frame_desc[i].status;
index a5c1a42..6bb0b1d 100644 (file)
@@ -429,11 +429,8 @@ static inline int em28xx_isoc_copy(struct em28xx *dev, struct urb *urb)
        if ((dev->state & DEV_DISCONNECTED) || (dev->state & DEV_MISCONFIGURED))
                return 0;
 
-       if (urb->status < 0) {
+       if (urb->status < 0)
                print_err_status(dev, -1, urb->status);
-               if (urb->status == -ENOENT)
-                       return 0;
-       }
 
        buf = dev->usb_ctl.vid_buf;
        if (buf != NULL)
@@ -525,11 +522,8 @@ static inline int em28xx_isoc_copy_vbi(struct em28xx *dev, struct urb *urb)
        if ((dev->state & DEV_DISCONNECTED) || (dev->state & DEV_MISCONFIGURED))
                return 0;
 
-       if (urb->status < 0) {
+       if (urb->status < 0)
                print_err_status(dev, -1, urb->status);
-               if (urb->status == -ENOENT)
-                       return 0;
-       }
 
        buf = dev->usb_ctl.vid_buf;
        if (buf != NULL)