[media] s5p-mfc: use vb2_is_streaming() to check vb2 queue status
authorJavier Martinez Canillas <javier@osg.samsung.com>
Thu, 9 Jun 2016 20:25:14 +0000 (17:25 -0300)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Fri, 8 Jul 2016 17:26:27 +0000 (14:26 -0300)
The streaming field in struct vb2_queue is meant to be private and should
not be used by drivers directly, instead the vb2_is_streaming() function
should be used to check the videobuf2 queue streaming status.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Kamil Debski <k.debski@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/platform/s5p-mfc/s5p_mfc_dec.c

index 038215c198ac4794a8812c4f39b9ffb2c8425f96..d3b01e92ee8027d64fa1ae25cbccd3f1b58b2937 100644 (file)
@@ -423,7 +423,7 @@ static int vidioc_s_fmt(struct file *file, void *priv, struct v4l2_format *f)
        pix_mp = &f->fmt.pix_mp;
        if (ret)
                return ret;
-       if (ctx->vq_src.streaming || ctx->vq_dst.streaming) {
+       if (vb2_is_streaming(&ctx->vq_src) || vb2_is_streaming(&ctx->vq_dst)) {
                v4l2_err(&dev->v4l2_dev, "%s queue busy\n", __func__);
                ret = -EBUSY;
                goto out;
@@ -820,7 +820,7 @@ static int vidioc_decoder_cmd(struct file *file, void *priv,
                if (cmd->flags != 0)
                        return -EINVAL;
 
-               if (!ctx->vq_src.streaming)
+               if (!vb2_is_streaming(&ctx->vq_src))
                        return -EINVAL;
 
                spin_lock_irqsave(&dev->irqlock, flags);