diff options
author | Hans Verkuil <hans.verkuil@cisco.com> | 2014-12-08 11:23:50 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-12-16 20:21:40 -0500 |
commit | 389208e1173e097590856ed24a505551510f78d4 (patch) | |
tree | 2225b7569a3fa1646b9697e7c44f0ec8f620fe3d /drivers/media | |
parent | 165d00439fa9e9da7f263dc844cbd524cf624074 (diff) |
[media] cx88: remove leftover start_video_dma() call
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>
Diffstat (limited to 'drivers/media')
-rw-r--r-- | drivers/media/pci/cx88/cx88-video.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/pci/cx88/cx88-video.c b/drivers/media/pci/cx88/cx88-video.c index 25a4b7f3188a..860c98fc72c7 100644 --- a/drivers/media/pci/cx88/cx88-video.c +++ b/drivers/media/pci/cx88/cx88-video.c | |||
@@ -523,7 +523,6 @@ static void buffer_queue(struct vb2_buffer *vb) | |||
523 | 523 | ||
524 | if (list_empty(&q->active)) { | 524 | if (list_empty(&q->active)) { |
525 | list_add_tail(&buf->list, &q->active); | 525 | list_add_tail(&buf->list, &q->active); |
526 | start_video_dma(dev, q, buf); | ||
527 | buf->count = q->count++; | 526 | buf->count = q->count++; |
528 | dprintk(2,"[%p/%d] buffer_queue - first active\n", | 527 | dprintk(2,"[%p/%d] buffer_queue - first active\n", |
529 | buf, buf->vb.v4l2_buf.index); | 528 | buf, buf->vb.v4l2_buf.index); |