On Saturday, September 11, 2010 10:24:26 Jean-Francois Moine wrote: > Hi, > > This new proposal cancels the previous patch 'Illuminators and status > LED controls'. > > Cheers. > > diff --git a/Documentation/DocBook/v4l/controls.xml b/Documentation/DocBook/v4l/controls.xml index 8408caa..4226a9e 100644 --- a/Documentation/DocBook/v4l/controls.xml +++ b/Documentation/DocBook/v4l/controls.xml @@ -312,10 +312,17 @@ minimum value disables backlight compensation.</entry> information and bits 24-31 must be zero.</entry> </row> <row> + <entry><constant>V4L2_CID_ILLUMINATORS_0</constant> + <constant>V4L2_CID_ILLUMINATORS_1</constant></entry> + <entry>oolean</entry> Typo: 'oolean' -> boolean :-) + <entry>Switch on or off the illuminator 0 or 1 of the device + (usually a microscope).</entry> + </row> + <row> <entry><constant>V4L2_CID_LASTP1</constant></entry> <entry></entry> <entry>End of the predefined control IDs (currently -<constant>V4L2_CID_BG_COLOR</constant> + 1).</entry> +<constant>V4L2_CID_ILLUMINATORS_1</constant> + 1).</entry> </row> <row> <entry><constant>V4L2_CID_PRIVATE_BASE</constant></entry> I would recommend naming these controls ILLUMINATOR_1 and _2 instead of _0 and _1. The average end-user starts counting at 1, not 0. Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco -- 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