On Sat, 14 Mar 2009 12:59:23 +0100 Jean-Francois Moine <moinejf@xxxxxxx> wrote: > + <entry><constant>V4L2_CID_LEDS</constant></entry> > + <entry>integer</entry> > + <entry>Switch on or off the LEDs or illuminators of the device. > +In the control value, each LED may be coded in one bit (0: off, 1: on) or in > +many bits (light intensity).</entry> > + </row> > + <row> The idea of having some sort of control over the LEDs is interesting, but we should have a better way of controlling it. If the LED may have more than one bit, maybe the better would be to create more than one CID entry. Something like: V4L2_CID_LED_POWER - for showing that the camera is being used V4L2_CID_LED_LIGHT - for normal white light V4L2_CID_LED_INFRARED - for dark light, using infrared ... This way a driver can enumberate what kind of leds are available, and get the power intensity range for each individual one. Cheers, Mauro -- 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