Commit 366108f0 authored by Philipp Zabel's avatar Philipp Zabel Committed by Mauro Carvalho Chehab

[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: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: default avatarKamil Debski <k.debski@samsung.com>
Signed-off-by: default avatarMauro Carvalho Chehab <m.chehab@samsung.com>
parent e2c257f4
...@@ -1671,12 +1671,12 @@ static irqreturn_t coda_irq_handler(int irq, void *data) ...@@ -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)); wr_ptr = coda_read(dev, CODA_REG_BIT_WR_PTR(ctx->idx));
/* Calculate bytesused field */ /* Calculate bytesused field */
if (dst_buf->v4l2_buf.sequence == 0) { if (dst_buf->v4l2_buf.sequence == 0) {
dst_buf->v4l2_planes[0].bytesused = (wr_ptr - start_ptr) + vb2_set_plane_payload(dst_buf, 0, wr_ptr - start_ptr +
ctx->vpu_header_size[0] + ctx->vpu_header_size[0] +
ctx->vpu_header_size[1] + ctx->vpu_header_size[1] +
ctx->vpu_header_size[2]; ctx->vpu_header_size[2]);
} else { } 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", v4l2_dbg(1, coda_debug, &ctx->dev->v4l2_dev, "frame size = %u\n",
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment