On 07/10/2020 10:45, Sakari Ailus wrote: > Add a V4L2 controls for analogue gai constants required to control > analogue gain. The values are device specific and thus need to be obtained > from the driver. > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > MAINTAINERS | 1 + > include/uapi/linux/ccs.h | 14 ++++++++++++++ > 2 files changed, 15 insertions(+) > create mode 100644 include/uapi/linux/ccs.h > > diff --git a/MAINTAINERS b/MAINTAINERS > index b72f666b8b60..c173e503b0b7 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -11569,6 +11569,7 @@ F: drivers/media/i2c/ccs/ > F: drivers/media/i2c/ccs-pll.c > F: drivers/media/i2c/ccs-pll.h > F: include/uapi/linux/smiapp.h > +F: include/uapi/linux/ccs.h > > MIPS > M: Thomas Bogendoerfer <tsbogend@xxxxxxxxxxxxxxxx> > diff --git a/include/uapi/linux/ccs.h b/include/uapi/linux/ccs.h > new file mode 100644 > index 000000000000..bcdce95955b0 > --- /dev/null > +++ b/include/uapi/linux/ccs.h > @@ -0,0 +1,14 @@ > +/* SPDX-License-Identifier: GPL-2.0-only AND BSD-3-Clause */ > +/* Copyright (C) 2020 Intel Corporation */ > + > +#ifndef __UAPI_CCS_H__ > +#define __UAPI_CCS_H__ What does CCS stand for? Provide a reference to the standard as well. Just looking at this header doesn't give you any clue as to what it related to. Regards, Hans > + > +#include <linux/videodev2.h> > + > +#define V4L2_CID_CCS_ANALOGUE_GAIN_M0 (V4L2_CID_USER_CCS_BASE + 1) > +#define V4L2_CID_CCS_ANALOGUE_GAIN_C0 (V4L2_CID_USER_CCS_BASE + 2) > +#define V4L2_CID_CCS_ANALOGUE_GAIN_M1 (V4L2_CID_USER_CCS_BASE + 3) > +#define V4L2_CID_CCS_ANALOGUE_GAIN_C1 (V4L2_CID_USER_CCS_BASE + 4) > + > +#endif >
