[media] v4l: omap4iss: Remove iss_video streaming field
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Sun, 3 Nov 2013 23:28:24 +0000 (20:28 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 3 Dec 2013 19:48:40 +0000 (17:48 -0200)
The vb2 queue already keeps track of the streaming state, there's no
need to duplicate that in the driver.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Acked-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/staging/media/omap4iss/iss_video.c
drivers/staging/media/omap4iss/iss_video.h

index bab62d762add98e64d0676fe230120461cb78c55..1cdfc43394989f8d136668a89c62838f977d06a2 100644 (file)
@@ -743,11 +743,6 @@ iss_video_streamon(struct file *file, void *fh, enum v4l2_buf_type type)
 
        mutex_lock(&video->stream_lock);
 
-       if (video->streaming) {
-               mutex_unlock(&video->stream_lock);
-               return -EBUSY;
-       }
-
        /* Start streaming on the pipeline. No link touching an entity in the
         * pipeline can be activated or deactivated once streaming is started.
         */
@@ -842,9 +837,6 @@ err_media_entity_pipeline_start:
                video->queue = NULL;
        }
 
-       if (!ret)
-               video->streaming = 1;
-
        mutex_unlock(&video->stream_lock);
        return ret;
 }
@@ -882,7 +874,6 @@ iss_video_streamoff(struct file *file, void *fh, enum v4l2_buf_type type)
        omap4iss_pipeline_set_stream(pipe, ISS_PIPELINE_STREAM_STOPPED);
        vb2_streamoff(&vfh->queue, type);
        video->queue = NULL;
-       video->streaming = 0;
 
        if (video->iss->pdata->set_constraints)
                video->iss->pdata->set_constraints(video->iss, false);
index 8cf1b3500e44c6135b128ea60673f26d49c34db9..35f14d84330bc4ac234dc31a55d1ff74c785d9b2 100644 (file)
@@ -156,9 +156,6 @@ struct iss_video {
        unsigned int bpl_value;         /* bytes per line value */
        unsigned int bpl_padding;       /* padding at end of line */
 
-       /* Entity video node streaming */
-       unsigned int streaming:1;
-
        /* Pipeline state */
        struct iss_pipeline pipe;
        struct mutex stream_lock;       /* pipeline and stream states */