]> git.baikalelectronics.ru Git - kernel.git/commit
V4L/DVB (13140): gspca_jeilinj: once one frame is discarded it keeps discarding all...
authorHans de Goede <hdegoede@redhat.com>
Fri, 9 Oct 2009 06:58:35 +0000 (03:58 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sat, 5 Dec 2009 20:40:31 +0000 (18:40 -0200)
commit33c090406308b388006d99d6e1cccdd8ac468302
tree5ec7281ec5b06fc1b8458fe92d14169b410c744f
parent6afad430f815d2381b26cabe6c985ff22195e0bb
V4L/DVB (13140): gspca_jeilinj: once one frame is discarded it keeps discarding all frames

While checking all gspca sub drivers pkt_scan functions for a bug I found in
1 of them (and after checking also in another), I noticed a bug in the
gspca_jeilinj work queue function, once it has decided to start discard a
frame because the application is not reading fast enough (and thus returning
buffers to fill fast enough), it never stops discarding.

This patch fixes this by simply completely removing the "discarding"
variable, if we need to discard the current frame because there is no buffer
to store it, the "frame" pointer will be NULL, so that is all we need to
check.

I've also moved the gspca_get_i_frame() call and the writing of the jpg
header to the buffer to after the first usb_bulk_msg() call, as we don't
need it before that, and that will give the app slightly more time to
queue a buffer for us to fill.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/gspca/jeilinj.c