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 | 165d00439fa9e9da7f263dc844cbd524cf624074 (diff) | |
download | linux-389208e1173e097590856ed24a505551510f78d4.tar.gz linux-389208e1173e097590856ed24a505551510f78d4.tar.bz2 linux-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 'MAINTAINERS')
0 files changed, 0 insertions, 0 deletions