media: hantro: Remove unneeded hantro_dec_buf_finish

Since now .buf_prepare takes care of setting the
buffer payload size, we can get rid of this,
at least for decoders.

Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/staging/media/hantro/hantro_drv.c b/drivers/staging/media/hantro/hantro_drv.c
index d0097c5fe7d936f6ec03b89807ff78580f5ebeb6..0db8ad4551601511c6c6083e7be03db3b81591fd 100644 (file)
--- a/drivers/staging/media/hantro/hantro_drv.c
+++ b/drivers/staging/media/hantro/hantro_drv.c
@@ -80,15 +80,6 @@ hantro_enc_buf_finish(struct hantro_ctx *ctx, struct vb2_buffer *buf,
        return 0;
 }
 
-static int
-hantro_dec_buf_finish(struct hantro_ctx *ctx, struct vb2_buffer *buf,
-                     unsigned int bytesused)
-{
-       /* For decoders set bytesused as per the output picture. */
-       buf->planes[0].bytesused = ctx->dst_fmt.plane_fmt[0].sizeimage;
-       return 0;
-}
-
 static void hantro_job_finish(struct hantro_dev *vpu,
                              struct hantro_ctx *ctx,
                              unsigned int bytesused,
@@ -431,7 +422,6 @@ static int hantro_open(struct file *filp)
                ctx->buf_finish = hantro_enc_buf_finish;
        } else if (func->id == MEDIA_ENT_F_PROC_VIDEO_DECODER) {
                allowed_codecs = vpu->variant->codec & HANTRO_DECODERS;
-               ctx->buf_finish = hantro_dec_buf_finish;
        } else {
                ret = -ENODEV;
                goto err_ctx_free;