On Sat, 14 Mar 2009, Mauro Carvalho Chehab wrote: > 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. There is already a sysfs led interface, you could just have the driver export the leds to the led subsystem and use that. -- 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
