Re: [PATCH] add V4L2_CID_USER_ALVIUM_BASE and request space for up to 128 controls

[Sakari Ailus <sakari.ailus@linux.intel.com>]

Hi Martin,

On Fri, Oct 09, 2026 at 07:38:24PM +0200, Martin Hecht wrote:
> That patch requests a continoous block of controls to enable various
> features of the Alvium camera family. There are many features like
> triggers, ROI for auto functions like AWB, auto gain and so on.
> 
> Signed-off-by: Martin Hecht <mhecht73@gmail.com>

Thanks for the patch.

Could you also include some patches for actual control definitions in the
same patchset?

> ---
>  include/uapi/linux/v4l2-controls.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h
> index d17e41d51d2e..5ed91c053617 100644
> --- a/include/uapi/linux/v4l2-controls.h
> +++ b/include/uapi/linux/v4l2-controls.h
> @@ -255,6 +255,12 @@ enum v4l2_colorfx {
>   */
>  #define V4L2_CID_USER_IT6625_BASE		(V4L2_CID_USER_BASE + 0x1250)
>  
> +/*
> + * The base for Alvium CSI2 driver controls.
> + * We reserve 128 controls for this driver.
> + */
> +#define V4L2_CID_USER_ALVIUM_BASE		(V4L2_CID_USER_BASE + 0x1260)
> +
>  /* MPEG-class control IDs */
>  /* The MPEG controls are applicable to all codec controls
>   * and the 'MPEG' part of the define is historical */

-- 
Kind regards,

Sakari Ailus