[media] cx88: remove leftover start_video_dma() call
authorHans Verkuil <hans.verkuil@cisco.com>
Mon, 8 Dec 2014 16:23:50 +0000 (13:23 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 17 Dec 2014 01:21:40 +0000 (23:21 -0200)
The start_streaming op is responsible for starting the video dma,
so it shouldn't be called anymore from the buf_queue op.

Unfortunately, this call to start_video_dma() was added to the
start_streaming op, but was forgotten to be removed from the
buf_queue op, which is where it used to be before the vb2 conversion.

Calling this function twice causes very hard to find errors: sometimes
it works, sometimes it doesn't. It took me a whole friggin' day
to track this down, and in the end it was just luck that my eye suddenly
triggered on that line.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/pci/cx88/cx88-video.c

index 25a4b7f3188a2e624be1d76cf257b9118bcd0580..860c98fc72c7f95f12763fc2c0edfaacb8a76890 100644 (file)
@@ -523,7 +523,6 @@ static void buffer_queue(struct vb2_buffer *vb)
 
        if (list_empty(&q->active)) {
                list_add_tail(&buf->list, &q->active);
-               start_video_dma(dev, q, buf);
                buf->count    = q->count++;
                dprintk(2,"[%p/%d] buffer_queue - first active\n",
                        buf, buf->vb.v4l2_buf.index);