OSDN Git Service

media: vivid: Set vid_cap_streaming and vid_out_streaming to true
authorVandana BN <bnvandana@gmail.com>
Mon, 9 Sep 2019 09:43:31 +0000 (06:43 -0300)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Tue, 1 Oct 2019 19:54:55 +0000 (16:54 -0300)
When vbi stream is started, followed by video streaming,
the vid_cap_streaming and vid_out_streaming were not being set to true,
which would cause the video stream to stop when vbi stream is stopped.
This patch allows to set vid_cap_streaming and vid_out_streaming to true.
According to Hans Verkuil it appears that these 'if (dev->kthread_vid_cap)'
checks are a left-over from the original vivid development and should never
have been there.

Signed-off-by: Vandana BN <bnvandana@gmail.com>
Cc: <stable@vger.kernel.org> # for v3.18 and up
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/vivid/vivid-vid-cap.c
drivers/media/platform/vivid/vivid-vid-out.c

index 8cbaa0c..2d03073 100644 (file)
@@ -223,9 +223,6 @@ static int vid_cap_start_streaming(struct vb2_queue *vq, unsigned count)
        if (vb2_is_streaming(&dev->vb_vid_out_q))
                dev->can_loop_video = vivid_vid_can_loop(dev);
 
-       if (dev->kthread_vid_cap)
-               return 0;
-
        dev->vid_cap_seq_count = 0;
        dprintk(dev, 1, "%s\n", __func__);
        for (i = 0; i < VIDEO_MAX_FRAME; i++)
index 148b663..a0364ac 100644 (file)
@@ -161,9 +161,6 @@ static int vid_out_start_streaming(struct vb2_queue *vq, unsigned count)
        if (vb2_is_streaming(&dev->vb_vid_cap_q))
                dev->can_loop_video = vivid_vid_can_loop(dev);
 
-       if (dev->kthread_vid_out)
-               return 0;
-
        dev->vid_out_seq_count = 0;
        dprintk(dev, 1, "%s\n", __func__);
        if (dev->start_streaming_error) {