On Sat, 29 Jan 2011 20:16:42 +0100 (CET) Guennadi Liakhovetski <g.liakhovetski@xxxxxx> wrote: ... > > --- 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. I'll submit a v2 patch since I have to resubmit the other patch, too. Thanks, Anatolij -- 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