[media] em28xx: fix wrong data offset for non-interlaced mode in em28xx_copy_video
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Thu, 8 Nov 2012 17:11:33 +0000 (14:11 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 22 Dec 2012 19:55:34 +0000 (17:55 -0200)
em28xx_copy_video uses a wrong offset for the target buffer
when copying the data from an USB isoc packet. This happens
only for the second and all following lines in the packet.
The reason why this bug doesn't cause image corruption with
my test device (SilverCrest Webcam 1.3 MPix) is, that this
device never sends any packets that cross the end of a line.
I don't know if all devices behave like this, so this patch
should be considered for stable.
With the upcoming patches to add support for USB bulk transfers,
em28xx_copy_video will be called once per URB, which will
always trigger this bug.

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

index 766ad125fc08c522539547284c89a83e9fa9a8bb..202e00bb2b6ed857e66f65129f7948170c88fa8f 100644 (file)
@@ -207,15 +207,10 @@ static void em28xx_copy_video(struct em28xx *dev,
        startread = p;
        remain = len;
 
-       if (dev->progressive)
+       if (dev->progressive || buf->top_field)
                fieldstart = outp;
-       else {
-               /* Interlaces two half frames */
-               if (buf->top_field)
-                       fieldstart = outp;
-               else
-                       fieldstart = outp + bytesperline;
-       }
+       else /* interlaced mode, even nr. of lines */
+               fieldstart = outp + bytesperline;
 
        linesdone = dma_q->pos / bytesperline;
        currlinedone = dma_q->pos % bytesperline;
@@ -243,7 +238,10 @@ static void em28xx_copy_video(struct em28xx *dev,
        remain -= lencopy;
 
        while (remain > 0) {
-               startwrite += lencopy + bytesperline;
+               if (dev->progressive)
+                       startwrite += lencopy;
+               else
+                       startwrite += lencopy + bytesperline;
                startread += lencopy;
                if (bytesperline > remain)
                        lencopy = remain;