Hi Mauro, On Thursday 18 March 2010 13:50:50 Laurent Pinchart wrote: > On Thursday 18 March 2010 13:41:36 Mauro Carvalho Chehab wrote: > > Laurent Pinchart wrote: > > > On Thursday 18 March 2010 13:19:57 Mauro Carvalho Chehab wrote: > > >> Laurent Pinchart wrote: > > >>> Those control, as their names imply, control the camera aperture > > >>> settings. > > >>> > > >>> Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > > >>> --- > > >>> > > >>> Documentation/DocBook/v4l/compat.xml | 11 +++++++++++ > > >>> Documentation/DocBook/v4l/controls.xml | 19 +++++++++++++++++++ > > >>> Documentation/DocBook/v4l/videodev2.h.xml | 3 +++ > > >>> include/linux/videodev2.h | 3 +++ > > >>> 4 files changed, 36 insertions(+), 0 deletions(-) > > >>> > > >>> diff --git a/Documentation/DocBook/v4l/compat.xml > > >>> b/Documentation/DocBook/v4l/compat.xml index b9dbdf9..854235b 100644 > > >>> --- a/Documentation/DocBook/v4l/compat.xml > > >>> +++ b/Documentation/DocBook/v4l/compat.xml > > >>> @@ -2332,6 +2332,17 @@ more information.</para> > > >>> > > >>> </listitem> > > >>> > > >>> </orderedlist> > > >>> > > >>> </section> > > >>> > > >>> + <section> > > >>> + <title>V4L2 in Linux 2.6.34</title> > > >>> + <orderedlist> > > >>> + <listitem> > > >>> + <para>Added > > >>> +<constant>V4L2_CID_IRIS_ABSOLUTE</constant> and > > >>> +<constant>V4L2_CID_IRIS_RELATIVE</constant> controls to the > > >>> + <link linkend="camera-controls">Camera controls > > >>> class</link>. > > >>> + </para> > > >>> + </listitem> > > >>> + </orderedlist> > > >>> > > >>> </section> > > >>> > > >>> <section id="other"> > > >>> > > >>> diff --git a/Documentation/DocBook/v4l/controls.xml > > >>> b/Documentation/DocBook/v4l/controls.xml index f464506..c412e89 > > >>> 100644 --- a/Documentation/DocBook/v4l/controls.xml > > >>> +++ b/Documentation/DocBook/v4l/controls.xml > > >>> @@ -1825,6 +1825,25 @@ wide-angle direction. The zoom speed unit is > > >>> driver-specific.</entry> > > >>> > > >>> <row><entry></entry></row> > > >>> > > >>> <row> > > >>> > > >>> + <entry > > >>> spanname="id"><constant>V4L2_CID_IRIS_ABSOLUTE</constant> </entr > > >>> y> + <entry>integer</entry> > > >>> + </row><row><entry spanname="descr">This control sets the > > >>> +camera aperture's to the specified value. The unit is undefined. > > >>> +Positive values open the iris, negative close it.</entry> > > >>> + </row> > > >>> + <row><entry></entry></row> > > >>> + > > >>> + <row> > > >>> + <entry > > >>> spanname="id"><constant>V4L2_CID_IRIS_RELATIVE</constant> </entr > > >>> y> + <entry>integer</entry> > > >>> + </row><row><entry spanname="descr">This control modifies the > > >>> +camera aperture's by the specified amount. The unit is undefined. > > >>> +Positive values open the iris one step further, negative values > > >>> close +it one step further. This is a write-only control.</entry> > > >>> + </row> > > >>> + <row><entry></entry></row> > > >>> + > > >>> + <row> > > >>> > > >>> <entry > > >>> spanname="id"><constant>V4L2_CID_PRIVACY</constant> </entr > > >>> y> <entry>boolean</entry> > > >>> > > >>> </row><row><entry spanname="descr">Prevent video from being > > >>> acquired > > >> > > >> Seems ok to me, but it would be good to add some sort of scale for > > >> those controls. > > > > > > I'd love to, but most iris controllers will just let you specify a > > > value in an arbitrary scale (0 for closed, 255 for fully opened for > > > instance). In that case do we want to force driver developers to > > > measure the aperture in µm units with a micrometer caliper ? :-) > > : > > :) > > > > Well, maybe then you could just comment that higher values means more > > opened apertures. > > Could point, I will do.
I spoke too fast, it's already there :-) -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
