Hi, On Thursday 03 March 2011 15:34:36 Kim HeungJun wrote: > 2011. 3. 3., 오후 10:24, Laurent Pinchart 작성: > > On Thursday 03 March 2011 14:17:10 Kim HeungJun wrote: > >> 2011. 3. 3., 오후 7:08, Laurent Pinchart 작성: > >>> On Thursday 03 March 2011 03:16:34 Kim, HeungJun wrote: > >>>> Document about the type changes and the enumeration of the auto focus > >>>> control. > >>>> > >>>> Signed-off-by: Heungjun Kim <riverful.kim@xxxxxxxxxxx> > >>>> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > >>>> --- > >>>> Documentation/DocBook/v4l/controls.xml | 31 > >>>> +++++++++++++++++++++++++--- Documentation/DocBook/v4l/videodev2.h.xml > >>>> | 6 +++++ > >>>> 2 files changed, 33 insertions(+), 4 deletions(-) > >>>> > >>>> diff --git a/Documentation/DocBook/v4l/controls.xml > >>>> b/Documentation/DocBook/v4l/controls.xml index 2fae3e8..889fa84 100644 > >>>> --- a/Documentation/DocBook/v4l/controls.xml > >>>> +++ b/Documentation/DocBook/v4l/controls.xml > >>>> @@ -1801,12 +1801,35 @@ negative values towards infinity. This is a > >>>> write-only control.</entry> </row> > >>>> > >>>> <row><entry></entry></row> > >>>> > >>>> - <row> > >>>> + <row id="v4l2-focus-auto-type"> > >>>> > >>>> <entry > >>>> > >>>> spanname="id"><constant>V4L2_CID_FOCUS_AUTO</constant> </entry> - > >>>> <entry>boolean</entry> > >>>> + <entry>enum v4l2_focus_auto_type</entry> > >>>> > >>>> </row><row><entry spanname="descr">Enables automatic focus > >>>> > >>>> -adjustments. The effect of manual focus adjustments while this > >>>> feature -is enabled is undefined, drivers should ignore such > >>>> requests.</entry> +adjustments of the normal or macro or > >>>> continuous(CAF) mode. The effect of +manual focus adjustments while > >>>> this feature is enabled is undefined, +drivers should ignore such > >>>> requests. Possible values are:</entry> + </row> > >>>> + <row> > >>>> + <entrytbl spanname="descr" cols="2"> > >>>> + <tbody valign="top"> > >>>> + <row> > >>>> + <entry><constant>V4L2_FOCUS_MANUAL</constant> </entry> > >>>> + <entry>Manual focus mode.</entry> > >>>> + </row> > >>>> + <row> > >>>> + <entry><constant>V4L2_FOCUS_AUTO</constant> </entry> > >>>> + <entry>Auto focus mode with normal operation.</entry> > >>>> + </row> > >>>> + <row> > >>>> + <entry><constant>V4L2_FOCUS_MACRO</constant> </entry> > >>>> + <entry>Auto focus mode with macro operation.</entry> > >>>> + </row> > >>>> + <row> > >>>> + <entry><constant>V4L2_FOCUS_CONTINUOUS</constant> </entry> > >>>> + <entry>Auto focus mode with continuous(CAF) operation.</entry> > >>> > >>> I should have asked this sooner, but what's the difference between > >>> normal AF and continuous AF ? > >> > >> Let's assume that the whole focus process(of course the sensor-internal > >> process automatically) is below : 1) pointing the object focussed > >> > >> : mostly this is possible to be a middle spot, or pre-given orient x > >> : & y set from register. > >> > >> (The mode using given orient x & y, may be defined V4L2_FOCUS_TOUCH > >> or > >> > >> whatever, but I know touch auto focus suggestion is determined nothing > >> so far, because of that, after this I will suggest to discuss about it > >> one more.) 2) starting and completing the move of the motor to control > >> lens, using the focusing internal algorithm 3) check whether the lens > >> position is proper or not, using the focusing internal algorithm, too. > >> (If the position is not proper, the image may looks defocused) > >> > >> In the Normal AF mode, the sensor do the whole focusing process once, > >> and after completing to move lens, even though the sensor failed > >> focusing process. On the other hand, the sensor repeats 1~3 in the > >> Continuous AF mode periodically, regardless of success of failure. > > > > OK, that was my understanding as well. How will that work with the > > proposed focus menu control ? Don't you need a button control for normal > > auto-focus ? > > You mean that the normal auto-focus control type should be a button, right? > For now, in our case the user application assumes that such modes(normal, > MACRO, CAF, TOUCH or oriented mode, or something) are already ready in the > driver. So, I did't need the normal auto-focus is a button type. > > But, IMHO, the focus control of camera is right to follow the next > procedure, 1) choosing the modes as I said upper. (because, the camera > dose not have one more focus mode.) 2) adjusting the value of the mode set > right before. > 3) doing the action of the mode. > > 1) is the same call the control V4L2_AUTO_FOCUS of menu type. > 2), 3) is the same call the control dedicated focus control. Currently, the > focus control excepting manual is not needed another control. If touch > mode are needed, we can add another enum value, e.g. V4L2_FOCUS_ORIENT. > and add another dedicated control, e.g., V4L2_CID_FOCUS_ORIENT_X, > V4L2_CID_FOCUS_ORIENT_Y. > > I think this as I use digital camera. We normally follow such procedure, > using the digital camera. > > If I get your words wrongly, give me some clue :) What happens when the user selects the V4L2_FOCUS_AUTO menu entry ? Will the camera start a one-shot auto-focus algorithm ? In that case, if the user is not happy with the result and wants to perform a new auto-focus, how will that be possible ? The V4L2_CID_FOCUS_AUTO control will already be in V4L2_FOCUS_AUTO mode. -- Regards, Laurent Pinchart -- 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