s5p-mfc: fix state check from encoder queue_setup 88/40388/1
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Wed, 13 May 2015 04:44:39 +0000 (13:44 +0900)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Wed, 3 Jun 2015 07:48:06 +0000 (16:48 +0900)
MFCINST_GOT_INST state is set to encoder context with set_format
only for catpure buffer. In queue_setup of encoder called during
reqbufs, it is checked MFCINST_GOT_INST state for both capture
and output buffer. So this patch fixes to encoder to check
MFCINST_GOT_INST state only for capture buffer from queue_setup.

Change-Id: I58f449eb48f990a2dcd4ecc06bc775293d1b6396
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
drivers/media/platform/s5p-mfc/s5p_mfc_enc.c

index f05a60b..80b8e99 100644 (file)
@@ -1635,11 +1635,12 @@ static int s5p_mfc_queue_setup(struct vb2_queue *vq,
        struct s5p_mfc_ctx *ctx = fh_to_ctx(vq->drv_priv);
        struct s5p_mfc_dev *dev = ctx->dev;
 
-       if (ctx->state != MFCINST_GOT_INST) {
-               mfc_err("inavlid state: %d\n", ctx->state);
-               return -EINVAL;
-       }
        if (vq->type == V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE) {
+               if (ctx->state != MFCINST_GOT_INST) {
+                       mfc_err("inavlid state: %d\n", ctx->state);
+                       return -EINVAL;
+               }
+
                if (ctx->dst_fmt)
                        *plane_count = ctx->dst_fmt->num_planes;
                else