On Wed, 26 Jan 2011, Anatolij Gustschin wrote: > Some camera systems have strong requirement for capturing > an exact number of frames after starting the stream and do > not tolerate losing captured frames. By starting the stream > after the videobuf has queued the buffers, we ensure that > no frame will be lost. > > Signed-off-by: Anatolij Gustschin <agust@xxxxxxx> > --- > drivers/media/video/soc_camera.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/video/soc_camera.c b/drivers/media/video/soc_camera.c > index a66811b..7299de0 100644 > --- a/drivers/media/video/soc_camera.c > +++ b/drivers/media/video/soc_camera.c > @@ -646,11 +646,11 @@ static int soc_camera_streamon(struct file *file, void *priv, > if (icd->streamer != file) > return -EBUSY; > > - v4l2_subdev_call(sd, video, s_stream, 1); > - > /* This calls buf_queue from host driver's videobuf_queue_ops */ > ret = videobuf_streamon(&icd->vb_vidq); > > + v4l2_subdev_call(sd, video, s_stream, 1); > + After a bit more testing I'll make this to + if (!ret) + v4l2_subdev_call(sd, video, s_stream, 1); + Ok? Or you can submit a v2 yourself, if you like - when you fix the comment in the other patch from this series. > return ret; > } > > -- > 1.7.1 Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
