diff options
| author |   Hans Verkuil <hans.verkuil@cisco.com> | 2014-12-08 13:23:50 -0300 |
|---|---|---|
| committer |   Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-12-16 23:21:40 -0200 |
| commit | 389208e1173e097590856ed24a505551510f78d4 (patch) | |
| tree | 2225b7569a3fa1646b9697e7c44f0ec8f620fe3d /MAINTAINERS | |
| parent | [media] cx88: add missing alloc_ctx support (diff) | |
| download | linux-dev-389208e1173e097590856ed24a505551510f78d4.tar.xz linux-dev-389208e1173e097590856ed24a505551510f78d4.zip | |
[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 '')
0 files changed, 0 insertions, 0 deletions