From: Philipp Zabel Date: Fri, 21 Jun 2013 06:55:27 +0000 (-0300) Subject: [media] coda: use vb2_set_plane_payload instead of setting v4l2_planes[0].bytesused... X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=366108f0ecfddc1957e02e9cc42db4c2e1053e24;p=GitHub%2FLineageOS%2Fandroid_kernel_motorola_exynos9610.git [media] coda: use vb2_set_plane_payload instead of setting v4l2_planes[0].bytesused directly As stated in the vb2_buffer documentation, drivers should not directly fill in v4l2_planes[0].bytesused, but should use the vb2_set_plane_payload() function instead. No functional changes. Signed-off-by: Philipp Zabel Signed-off-by: Kamil Debski Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/platform/coda.c b/drivers/media/platform/coda.c index df4ada880e42..ef3bb8bc3b6a 100644 --- a/drivers/media/platform/coda.c +++ b/drivers/media/platform/coda.c @@ -1671,12 +1671,12 @@ static irqreturn_t coda_irq_handler(int irq, void *data) wr_ptr = coda_read(dev, CODA_REG_BIT_WR_PTR(ctx->idx)); /* Calculate bytesused field */ if (dst_buf->v4l2_buf.sequence == 0) { - dst_buf->v4l2_planes[0].bytesused = (wr_ptr - start_ptr) + - ctx->vpu_header_size[0] + - ctx->vpu_header_size[1] + - ctx->vpu_header_size[2]; + vb2_set_plane_payload(dst_buf, 0, wr_ptr - start_ptr + + ctx->vpu_header_size[0] + + ctx->vpu_header_size[1] + + ctx->vpu_header_size[2]); } else { - dst_buf->v4l2_planes[0].bytesused = (wr_ptr - start_ptr); + vb2_set_plane_payload(dst_buf, 0, wr_ptr - start_ptr); } v4l2_dbg(1, coda_debug, &ctx->dev->v4l2_dev, "frame size = %u\n",