Hi Sakari, Thank you for the patch. On Monday 20 Feb 2017 17:22:19 Sakari Ailus wrote: > The intent of the check the comment is related to is to ensure we are > streaming --- still. Not that streaming wouldn't be enabled yet. Remove > it. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/platform/omap3isp/ispvideo.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/media/platform/omap3isp/ispvideo.c > b/drivers/media/platform/omap3isp/ispvideo.c index 9e9b18c..a3ca2a4 100644 > --- a/drivers/media/platform/omap3isp/ispvideo.c > +++ b/drivers/media/platform/omap3isp/ispvideo.c > @@ -1205,7 +1205,6 @@ isp_video_streamoff(struct file *file, void *fh, enum > v4l2_buf_type type) > > mutex_lock(&video->stream_lock); > > - /* Make sure we're not streaming yet. */ > mutex_lock(&video->queue_lock); > streaming = vb2_is_streaming(&vfh->queue); > mutex_unlock(&video->queue_lock); -- Regards, Laurent Pinchart
