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)
commit6ca3f255f790764f9cfc41d3ac02823d83dfa5ac
tree5ec7281ec5b06fc1b8458fe92d14169b410c744f
parentb3e440eef8a842736d63cc6a6594d80dfbb75fd9
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