On Sat, 2010-09-04 at 13:10 +0200, Jean-Francois Moine wrote: > Some media devices may have one or many lights (LEDs, illuminators, > lamps..). This patch makes them controlable by the applications. > > Signed-off-by: Jean-Francois Moine <moinejf@xxxxxxx> Jean-Francois, Thanks for the proposal. It looks OK to me. Acked-by: Andy Walls <awalls@xxxxxxxxxxxxxxxx> Regards, Andy > diff --git a/Documentation/DocBook/v4l/controls.xml b/Documentation/DocBook/v4l/controls.xml > index 8408caa..c9b8ca5 100644 > --- a/Documentation/DocBook/v4l/controls.xml > +++ b/Documentation/DocBook/v4l/controls.xml > @@ -312,6 +312,13 @@ minimum value disables backlight compensation.</entry> > information and bits 24-31 must be zero.</entry> > </row> > <row> > + <entry><constant>V4L2_CID_LEDS</constant></entry> > + <entry>integer</entry> > + <entry>Switch on or off the LED(s) or illuminator(s) of the device. > + The control type and values depend on the driver and may be either > + a single boolean (0: off, 1:on) or the index in a menu type.</entry> > + </row> > + <row> > <entry><constant>V4L2_CID_LASTP1</constant></entry> > <entry></entry> > <entry>End of the predefined control IDs (currently > diff --git a/include/linux/videodev2.h b/include/linux/videodev2.h > index 61490c6..3807492 100644 > --- a/include/linux/videodev2.h > +++ b/include/linux/videodev2.h > @@ -1045,8 +1045,10 @@ enum v4l2_colorfx { > > #define V4L2_CID_CHROMA_GAIN (V4L2_CID_BASE+36) > > +#define V4L2_CID_LEDS (V4L2_CID_BASE+37) > + > /* last CID + 1 */ > -#define V4L2_CID_LASTP1 (V4L2_CID_BASE+37) > +#define V4L2_CID_LASTP1 (V4L2_CID_BASE+38) > > /* MPEG-class control IDs defined by V4L2 */ > #define V4L2_CID_MPEG_BASE (V4L2_CTRL_CLASS_MPEG | 0x900) > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html