Hi Benjamin, Thank you for the patch. On Fri, Apr 15, 2022 at 01:18:43PM +0200, Benjamin Mugnier wrote: > Add a control base for st-vgxy61 controls and reserve 16 controls. > This is required for HDR mode user control on the st-vgxy61 camera > sensor. > > Signed-off-by: Benjamin Mugnier <benjamin.mugnier@xxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > include/uapi/linux/v4l2-controls.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/uapi/linux/v4l2-controls.h b/include/uapi/linux/v4l2-controls.h > index 705b4043c2de..0bdf77df26ac 100644 > --- a/include/uapi/linux/v4l2-controls.h > +++ b/include/uapi/linux/v4l2-controls.h > @@ -225,6 +225,11 @@ enum v4l2_colorfx { > */ > #define V4L2_CID_USER_ISL7998X_BASE (V4L2_CID_USER_BASE + 0x1180) > > +/* The base for the st-vgxy61 driver controls. > + * We reserve 16 controls for this driver. > + */ > +#define V4L2_CID_USER_STVGXY61_BASE (V4L2_CID_USER_BASE + 0x1190) > + > /* MPEG-class control IDs */ > /* The MPEG controls are applicable to all codec controls > * and the 'MPEG' part of the define is historical */ -- Regards, Laurent Pinchart