[PATCH] media: v4l2-mem2mem: Don't copy frame flags for frame-based devices
Frank Li
Frank.li at nxp.com
Thu Oct 9 07:40:40 PDT 2025
On Thu, Oct 09, 2025 at 02:11:43PM +0300, Laurent Pinchart wrote:
> The v4l2_m2m_buf_copy_metadata() function takes a boolean
> copy_frame_flags argument. When true, it causes the function to copy the
> V4L2_BUF_FLAG_KEYFRAME, V4L2_BUF_FLAG_BFRAME and V4L2_BUF_FLAG_PFRAME
> flags from the output buffer to the capture buffer.
>
> Many frame-based M2M drivers (e.g. for JPEG encoders, scalers,
> dewarpers, 2D blenders, ...) set the argument to true, while the frame
> flags are not applicable to those drivers as they have no concept of
> key, B or P frames. Set the argument to false to avoid further
> cargo-cult mistakes.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Frank Li <Frank.Li at nxp.com>
> ---
> drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c | 4 ++--
> drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c | 4 ++--
> drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c | 4 ++--
> drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c | 2 +-
> drivers/media/platform/nxp/dw100/dw100.c | 2 +-
> drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c | 2 +-
> drivers/media/platform/rockchip/rga/rga.c | 2 +-
> drivers/media/platform/st/stm32/dma2d/dma2d.c | 2 +-
> drivers/media/platform/sunxi/sun8i-di/sun8i-di.c | 2 +-
> drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c | 2 +-
> drivers/media/test-drivers/vim2m.c | 2 +-
> drivers/staging/media/imx/imx-media-csc-scaler.c | 2 +-
> 12 files changed, 15 insertions(+), 15 deletions(-)
>
> diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
> index 35c70ec3ad2c..6bd5036430dc 100644
> --- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
> +++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_core.c
> @@ -1619,7 +1619,7 @@ static void mtk_jpegenc_worker(struct work_struct *work)
> if (!dst_buf)
> goto getbuf_fail;
>
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> mtk_jpegenc_set_hw_param(ctx, hw_id, src_buf, dst_buf);
> ret = pm_runtime_get_sync(comp_jpeg[hw_id]->dev);
> @@ -1715,7 +1715,7 @@ static void mtk_jpegdec_worker(struct work_struct *work)
> if (!dst_buf)
> goto getbuf_fail;
>
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
> jpeg_src_buf = mtk_jpeg_vb2_to_srcbuf(&src_buf->vb2_buf);
> jpeg_dst_buf = mtk_jpeg_vb2_to_srcbuf(&dst_buf->vb2_buf);
>
> diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
> index e78e1d11093c..556865100872 100644
> --- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
> +++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_dec_hw.c
> @@ -530,7 +530,7 @@ static void mtk_jpegdec_timeout_work(struct work_struct *work)
>
> src_buf = cjpeg->hw_param.src_buffer;
> dst_buf = cjpeg->hw_param.dst_buffer;
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> mtk_jpeg_dec_reset(cjpeg->reg_base);
> clk_disable_unprepare(cjpeg->jdec_clk.clks->clk);
> @@ -560,7 +560,7 @@ static irqreturn_t mtk_jpegdec_hw_irq_handler(int irq, void *priv)
> ctx = jpeg->hw_param.curr_ctx;
> src_buf = jpeg->hw_param.src_buffer;
> dst_buf = jpeg->hw_param.dst_buffer;
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> irq_status = mtk_jpeg_dec_get_int_status(jpeg->reg_base);
> dec_irq_ret = mtk_jpeg_dec_enum_result(irq_status);
> diff --git a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
> index 9ab27aee302a..4c8427b3c384 100644
> --- a/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
> +++ b/drivers/media/platform/mediatek/jpeg/mtk_jpeg_enc_hw.c
> @@ -261,7 +261,7 @@ static void mtk_jpegenc_timeout_work(struct work_struct *work)
>
> src_buf = cjpeg->hw_param.src_buffer;
> dst_buf = cjpeg->hw_param.dst_buffer;
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> mtk_jpeg_enc_reset(cjpeg->reg_base);
> clk_disable_unprepare(cjpeg->venc_clk.clks->clk);
> @@ -289,7 +289,7 @@ static irqreturn_t mtk_jpegenc_hw_irq_handler(int irq, void *priv)
> ctx = jpeg->hw_param.curr_ctx;
> src_buf = jpeg->hw_param.src_buffer;
> dst_buf = jpeg->hw_param.dst_buffer;
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> irq_status = readl(jpeg->reg_base + JPEG_ENC_INT_STS) &
> JPEG_ENC_INT_STATUS_MASK_ALLIRQ;
> diff --git a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c
> index 59ce5cce0698..dba46a69c6be 100644
> --- a/drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c
> +++ b/drivers/media/platform/mediatek/mdp3/mtk-mdp3-m2m.c
> @@ -51,7 +51,7 @@ static void mdp_m2m_process_done(void *priv, int vb_state)
> ctx->curr_param.frame_no = ctx->frame_count[MDP_M2M_SRC];
> src_vbuf->sequence = ctx->frame_count[MDP_M2M_SRC]++;
> dst_vbuf->sequence = ctx->frame_count[MDP_M2M_DST]++;
> - v4l2_m2m_buf_copy_metadata(src_vbuf, dst_vbuf, true);
> + v4l2_m2m_buf_copy_metadata(src_vbuf, dst_vbuf, false);
>
> v4l2_m2m_buf_done(src_vbuf, vb_state);
> v4l2_m2m_buf_done(dst_vbuf, vb_state);
> diff --git a/drivers/media/platform/nxp/dw100/dw100.c b/drivers/media/platform/nxp/dw100/dw100.c
> index 7a0ee44d9e1f..b73302d54635 100644
> --- a/drivers/media/platform/nxp/dw100/dw100.c
> +++ b/drivers/media/platform/nxp/dw100/dw100.c
> @@ -1483,7 +1483,7 @@ static void dw100_start(struct dw100_ctx *ctx, struct vb2_v4l2_buffer *in_vb,
> V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE),
> in_vb->sequence, out_vb->sequence);
>
> - v4l2_m2m_buf_copy_metadata(in_vb, out_vb, true);
> + v4l2_m2m_buf_copy_metadata(in_vb, out_vb, false);
>
> /* Now, let's deal with hardware ... */
> dw100_hw_master_bus_disable(dw_dev);
> diff --git a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> index 37e0670f98c5..e1dda1e834e4 100644
> --- a/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> +++ b/drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c
> @@ -1537,7 +1537,7 @@ static void mxc_jpeg_device_run(void *priv)
> src_buf->sequence = q_data_out->sequence++;
> dst_buf->sequence = q_data_cap->sequence++;
>
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> jpeg_src_buf = vb2_to_mxc_buf(&src_buf->vb2_buf);
> if (q_data_cap->fmt->mem_planes != dst_buf->vb2_buf.num_planes) {
> diff --git a/drivers/media/platform/rockchip/rga/rga.c b/drivers/media/platform/rockchip/rga/rga.c
> index 48b88da59da0..075b684fb178 100644
> --- a/drivers/media/platform/rockchip/rga/rga.c
> +++ b/drivers/media/platform/rockchip/rga/rga.c
> @@ -75,7 +75,7 @@ static irqreturn_t rga_isr(int irq, void *prv)
> WARN_ON(!src);
> WARN_ON(!dst);
>
> - v4l2_m2m_buf_copy_metadata(src, dst, true);
> + v4l2_m2m_buf_copy_metadata(src, dst, false);
>
> dst->sequence = ctx->csequence++;
>
> diff --git a/drivers/media/platform/st/stm32/dma2d/dma2d.c b/drivers/media/platform/st/stm32/dma2d/dma2d.c
> index 643913adc1f3..4184bdb96e6d 100644
> --- a/drivers/media/platform/st/stm32/dma2d/dma2d.c
> +++ b/drivers/media/platform/st/stm32/dma2d/dma2d.c
> @@ -483,7 +483,7 @@ static void device_run(void *prv)
>
> src->sequence = frm_out->sequence++;
> dst->sequence = frm_cap->sequence++;
> - v4l2_m2m_buf_copy_metadata(src, dst, true);
> + v4l2_m2m_buf_copy_metadata(src, dst, false);
>
> if (clk_enable(dev->gate))
> goto end;
> diff --git a/drivers/media/platform/sunxi/sun8i-di/sun8i-di.c b/drivers/media/platform/sunxi/sun8i-di/sun8i-di.c
> index 3e7f2df70408..11a6c7f5212e 100644
> --- a/drivers/media/platform/sunxi/sun8i-di/sun8i-di.c
> +++ b/drivers/media/platform/sunxi/sun8i-di/sun8i-di.c
> @@ -71,7 +71,7 @@ static void deinterlace_device_run(void *priv)
> src = v4l2_m2m_next_src_buf(ctx->fh.m2m_ctx);
> dst = v4l2_m2m_next_dst_buf(ctx->fh.m2m_ctx);
>
> - v4l2_m2m_buf_copy_metadata(src, dst, true);
> + v4l2_m2m_buf_copy_metadata(src, dst, false);
>
> deinterlace_write(dev, DEINTERLACE_MOD_ENABLE,
> DEINTERLACE_MOD_ENABLE_EN);
> diff --git a/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c b/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
> index abd10b218aa1..f6e2f11a20dd 100644
> --- a/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
> +++ b/drivers/media/platform/sunxi/sun8i-rotate/sun8i_rotate.c
> @@ -70,7 +70,7 @@ static void rotate_device_run(void *priv)
> src = v4l2_m2m_next_src_buf(ctx->fh.m2m_ctx);
> dst = v4l2_m2m_next_dst_buf(ctx->fh.m2m_ctx);
>
> - v4l2_m2m_buf_copy_metadata(src, dst, true);
> + v4l2_m2m_buf_copy_metadata(src, dst, false);
>
> val = ROTATE_GLB_CTL_MODE(ROTATE_MODE_COPY_ROTATE);
> if (ctx->hflip)
> diff --git a/drivers/media/test-drivers/vim2m.c b/drivers/media/test-drivers/vim2m.c
> index dc82830a35a5..3e8476165721 100644
> --- a/drivers/media/test-drivers/vim2m.c
> +++ b/drivers/media/test-drivers/vim2m.c
> @@ -482,7 +482,7 @@ static int device_process(struct vim2m_ctx *ctx,
>
> out_vb->sequence = q_data_out->sequence++;
> in_vb->sequence = q_data_in->sequence++;
> - v4l2_m2m_buf_copy_metadata(in_vb, out_vb, true);
> + v4l2_m2m_buf_copy_metadata(in_vb, out_vb, false);
>
> if (ctx->mode & MEM2MEM_VFLIP) {
> start = height - 1;
> diff --git a/drivers/staging/media/imx/imx-media-csc-scaler.c b/drivers/staging/media/imx/imx-media-csc-scaler.c
> index 19fd31cb9bb0..770ba3fbaba2 100644
> --- a/drivers/staging/media/imx/imx-media-csc-scaler.c
> +++ b/drivers/staging/media/imx/imx-media-csc-scaler.c
> @@ -96,7 +96,7 @@ static void ipu_ic_pp_complete(struct ipu_image_convert_run *run, void *_ctx)
> src_buf = v4l2_m2m_src_buf_remove(ctx->fh.m2m_ctx);
> dst_buf = v4l2_m2m_dst_buf_remove(ctx->fh.m2m_ctx);
>
> - v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, true);
> + v4l2_m2m_buf_copy_metadata(src_buf, dst_buf, false);
>
> src_buf->sequence = ctx->sequence++;
> dst_buf->sequence = src_buf->sequence;
>
> base-commit: e5f0a698b34ed76002dc5cff3804a61c80233a7a
> --
> Regards,
>
> Laurent Pinchart
>
More information about the Linux-rockchip
mailing list