On 02/07/18 14:00, Florian Echtler wrote: > This patch adds parameter definitions and module parameters for the four > userspace controls that the SUR40 can currently provide. > > Signed-off-by: Florian Echtler <floe@xxxxxxxxxxxxxx> > --- > drivers/input/touchscreen/sur40.c | 28 ++++++++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/drivers/input/touchscreen/sur40.c b/drivers/input/touchscreen/sur40.c > index 8375b06..8a5b031 100644 > --- a/drivers/input/touchscreen/sur40.c > +++ b/drivers/input/touchscreen/sur40.c > @@ -149,6 +149,34 @@ struct sur40_image_header { > #define SUR40_TOUCH 0x02 > #define SUR40_TAG 0x04 > > +/* video controls */ > +#define SUR40_BRIGHTNESS_MAX 0xFF > +#define SUR40_BRIGHTNESS_MIN 0x00 > +#define SUR40_BRIGHTNESS_DEF 0xFF > + > +#define SUR40_CONTRAST_MAX 0x0F > +#define SUR40_CONTRAST_MIN 0x00 > +#define SUR40_CONTRAST_DEF 0x0A It's kernel style to use lowercase hex values. > + > +#define SUR40_GAIN_MAX 0x09 > +#define SUR40_GAIN_MIN 0x00 > +#define SUR40_GAIN_DEF 0x08 > + > +#define SUR40_BACKLIGHT_MAX 0x01 > +#define SUR40_BACKLIGHT_MIN 0x00 > +#define SUR40_BACKLIGHT_DEF 0x01 > + > +/* module parameters */ > +static uint brightness = SUR40_BRIGHTNESS_DEF; > +module_param(brightness, uint, 0644); > +MODULE_PARM_DESC(brightness, "set initial brightness"); No range in the param description? I think you can do some macro magic so you can use the MIN/MAX defines in the string. Regards, Hans > +static uint contrast = SUR40_CONTRAST_DEF; > +module_param(contrast, uint, 0644); > +MODULE_PARM_DESC(contrast, "set initial contrast"); > +static uint gain = SUR40_GAIN_DEF; > +module_param(gain, uint, 0644); > +MODULE_PARM_DESC(gain, "set initial gain"); > + > static const struct v4l2_pix_format sur40_pix_format[] = { > { > .pixelformat = V4L2_TCH_FMT_TU08, > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html