media: coda: flag the last encoded buffer

Use the flagged last output buffer to also flag the corresponding
capture buffer after encoding. This causes the end of stream event
to be issued and the buffer to be dequeued with the last flag set.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>

diff --git a/drivers/media/platform/coda/coda-bit.c b/drivers/media/platform/coda/coda-bit.c
index cbcec57..7bcdfe8 100644 (file)
--- a/drivers/media/platform/coda/coda-bit.c
+++ b/drivers/media/platform/coda/coda-bit.c
@@ -1565,13 +1565,14 @@ static void coda_finish_encode(struct coda_ctx *ctx)
        coda_read(dev, CODA_RET_ENC_PIC_SLICE_NUM);
        coda_read(dev, CODA_RET_ENC_PIC_FLAG);
 
-       if (coda_read(dev, CODA_RET_ENC_PIC_TYPE) == 0) {
+       dst_buf->flags &= ~(V4L2_BUF_FLAG_KEYFRAME |
+                           V4L2_BUF_FLAG_PFRAME |
+                           V4L2_BUF_FLAG_LAST);
+       if (coda_read(dev, CODA_RET_ENC_PIC_TYPE) == 0)
                dst_buf->flags |= V4L2_BUF_FLAG_KEYFRAME;
-               dst_buf->flags &= ~V4L2_BUF_FLAG_PFRAME;
-       } else {
+       else
                dst_buf->flags |= V4L2_BUF_FLAG_PFRAME;
-               dst_buf->flags &= ~V4L2_BUF_FLAG_KEYFRAME;
-       }
+       dst_buf->flags |= src_buf->flags & V4L2_BUF_FLAG_LAST;
 
        v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
 
@@ -1584,8 +1585,9 @@ static void coda_finish_encode(struct coda_ctx *ctx)
        if (ctx->gopcounter < 0)
                ctx->gopcounter = ctx->params.gop_size - 1;
 
-       coda_dbg(1, ctx, "job finished: encoded %c frame (%d)\n",
-                coda_frame_type_char(dst_buf->flags), dst_buf->sequence);
+       coda_dbg(1, ctx, "job finished: encoded %c frame (%d)%s\n",
+                coda_frame_type_char(dst_buf->flags), dst_buf->sequence,
+                (dst_buf->flags & V4L2_BUF_FLAG_LAST) ? " (last)" : "");
 }
 
 static void coda_seq_end_work(struct work_struct *work)