linux-toradex.git/include/media, branch v6.19

media: v4l2-isp: Rename block_info to block_type_info

2025-11-14T14:48:49+00:00

The v4l2_isp_params_block_info structure contains validation information
that apply to a block -type- and not only to a specific ISP block
implementation.

Clarify this by renaming v4l2_isp_params_block_info in
v4l2_isp_params_block_type_info and update the documentation and the
users of v4l2-isp accordingly.

Signed-off-by: Jacopo Mondi 
Reviewed-by: Laurent Pinchart 
Reviewed-by: Sakari Ailus 
Signed-off-by: Hans Verkuil

media: v4l2-core: Introduce v4l2-isp.c

2025-11-14T14:48:48+00:00

Add to the V4L2 framework helper functions to support drivers when
validating a buffer of V4L2 ISP parameters.

Driver shall use v4l2_isp_params_validate_buffer_size() to verify the
size correctness of the data received from userspace, and after having
copied the data to a kernel-only memory location, complete the
validation by calling v4l2_isp_params_validate_buffer().

Reviewed-by: Daniel Scally 
Reviewed-by: Laurent Pinchart 
Reviewed-by: Michael Riesch 
Acked-by: Sakari Ailus 
Tested-by: Lad Prabhakar 
Signed-off-by: Jacopo Mondi 
Signed-off-by: Hans Verkuil

media: v4l: Add helper to get number of active lanes via a pad

2025-11-13T09:57:52+00:00

Sometimes, users will not use all of the MIPI CSI 2 lanes available when
connecting to the MIPI CSI receiver of their device. Add a helper
function that checks the mbus_config for the device driver to allow
users to define the number of active data lanes through the
get_mbus_config op.

If the driver does not implement this op, fall back to using the maximum
number of lanes available.

Reviewed-by: Frank Li 
Reviewed-by: Laurent Pinchart 
Signed-off-by: Isaac Scott 
Signed-off-by: Sakari Ailus 
Signed-off-by: Hans Verkuil

media: saa7146: Replace saa7146_ext_vv.vbi_fops with write function

2025-11-03T14:58:42+00:00

The vbi_fops stored in struct saa7146_ext_vv is a full
v4l2_file_operations, but only its .write field is used. Replace it with
a single vbi_write function pointer to save memory.

Signed-off-by: Laurent Pinchart 
Signed-off-by: Hans Verkuil

media: mc: Make macros to obtain containers const-aware

2025-11-03T14:58:41+00:00

Retain the constness of the graph objects and interfaces in macros to
obtain their containers, by switching to container_of_const().

Signed-off-by: Sakari Ailus 
Signed-off-by: Hans Verkuil

media: v4l2-dev: Make macros to obtain containers const-aware

2025-11-03T14:58:41+00:00

Retain the constness of the object in media_entity_to_video_device() and
to_video_device(), by switching to container_of_const().

Signed-off-by: Sakari Ailus 
Signed-off-by: Hans Verkuil

media: v4l2-subdev: Make media_entity_to_v4l2_subdev() const-aware

2025-11-03T14:58:41+00:00

Retain the constness of the object in media_entity_to_v4l2_subdev(), by
switching to container_of_const().

Signed-off-by: Sakari Ailus 
Signed-off-by: Hans Verkuil

media: v4l2-mem2mem: Don't copy frame flags in v4l2_m2m_buf_copy_metadata()

2025-10-17T09:31:16+00:00

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.

There is no use cases in any upstream driver for copying the flags.
KEY/P/B frames are properties of the bitstream buffer in some formats.
Once decoded, this is no longer a property of the video frame and should
be discarded.

It was considered useful to know if an uncompressed frame was decoded
from a KEY/P/B compressed frame, and to preserve that information if
that same uncompressed frame was passed through another M2M device (e.g.
a scaler). However, the V4L2 documentation makes it clear that the flags
are meant for compressed frames only.

Drop the copy_frame_flags argument from v4l2_m2m_buf_copy_metadata().
The change to drivers was performed with the following Coccinelle
semantic patch:

@@
expression src;
expression dst;
expression flag;
@@
-       v4l2_m2m_buf_copy_metadata(src, dst, flag);
+       v4l2_m2m_buf_copy_metadata(src, dst);

include/media/v4l2-mem2mem.h and drivers/media/v4l2-core/v4l2-mem2mem.c
have been updated manually.

Signed-off-by: Laurent Pinchart 
Reviewed-by: Philipp Zabel 
Reviewed-by: Benjamin Gaignard 
Signed-off-by: Hans Verkuil

media: v4l2-mem2mem: Document that v4l2_m2m_get_vq() never returns NULL

2025-10-17T09:31:14+00:00

The v4l2_m2m_get_vq() never returns a NULL pointer, as the internal
get_queue_ctx() helper always returns a non-NULL pointer. Many drivers
check the return value against NULL, due to a combination of old code
and cargo-cult programming. Even v4l2-mem2mem.c contains unneeded NULL
checks.

Clarify the API by documenting explicitly that a NULL check is not
needed, and simplify the code by removing the unneeded NULL checks from
v4l2-mem2mem.c.

Signed-off-by: Laurent Pinchart 
Reviewed-by: Stefan Klug 
Signed-off-by: Hans Verkuil

media: v4l2-mem2mem: Fix outdated documentation

2025-10-14T13:07:37+00:00

Commit cbd9463da1b1 ("media: v4l2-mem2mem: Avoid calling .device_run in
v4l2_m2m_job_finish") deferred calls to .device_run() to a work queue to
avoid recursive calls when a job is finished right away from
.device_run(). It failed to update the v4l2_m2m_job_finish()
documentation that still states the function must not be called from
.device_run(). Fix it.

Fixes: cbd9463da1b1 ("media: v4l2-mem2mem: Avoid calling .device_run in v4l2_m2m_job_finish")
Cc: stable@vger.kernel.org
Signed-off-by: Laurent Pinchart 
Signed-off-by: Hans Verkuil