Hi Hans, On Thu, Nov 05, 2020 at 01:56:02PM +0100, Hans Verkuil wrote: > 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. I'll add references to driver documentation I'm about to write to address Mauro's review comments. CCS stands for Camera Command Set. The spec can be found here: <URL:https://www.mipi.org/specifications/camera-command-set> -- Regards, Sakari Ailus
