Hi, (CC'ing linux-media this time, please discard the previous mail) On Tuesday 14 December 2010 12:27:32 Kim, HeungJun wrote: > Hi Laurent and Hans, > > I am working on V4L2 subdev for M5MOLS by Fujitsu. > and I wanna listen your comments about Auto Focus mode of my ideas. > the details is in the following link discussed at the past. > Although the situation(adding the more various functions at the M5MOLS > or any other MEGA camera sensor, I worked.)is changed, > so I wanna continue this threads for now. > > http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg03543.html > > First of all, the at least two more mode of auto-focus exists in the > M5MOLS camera sensor. So, considering defined V4L2 controls and the > controls in the M5MOLS, I suggest like this: > > +enum v4l2_focus_auto_type { > + V4L2_FOCUS_AUTO_NORMAL = 0, > + V4L2_FOCUS_AUTO_MACRO = 1, > + V4L2_FOCUS_AUTO_POSITION = 2, > +}; > +#define V4L2_CID_FOCUS_POSITION (V4L2_CID_CAMERA_CLASS_BASE+13) > > -#define V4L2_CID_ZOOM_ABSOLUTE (V4L2_CID_CAMERA_CLASS_BASE+13) > -#define V4L2_CID_ZOOM_RELATIVE (V4L2_CID_CAMERA_CLASS_BASE+14) > +#define V4L2_CID_ZOOM_ABSOLUTE (V4L2_CID_CAMERA_CLASS_BASE+14) > +#define V4L2_CID_ZOOM_RELATIVE (V4L2_CID_CAMERA_CLASS_BASE+15) > > > The M5MOLS(or other recent camera sensor) can have at least 2 mode although > in any cases : *MACRO* and *NORMAL* mode. plus, M5MOLS supports > positioning focus mode, AKA. POSITION AF mode. > > The MACRO mode scan short range, and this mode can be used at the > circumstance in the short distance with object and camera lens. So, It has > fast lens movement, but the command FOCUSING dosen't works well at the > long distance object. > > On the other hand, NORMAL mode can this. As the words, It's general and > normal focus mode. The M5MOLS scan fully in the mode. > > In the Position AF mode, the position(expressed x,y) is given at the > M5MOLS, and then the M5MOLS focus this area. But, the time given the > position, is normally touch the lcd screen at the mobile device, in my > case. If the time is given from button, it's no big problem *when*. But, > in touch-lcd screen case, the position is read at the touch screen driver, > before command FOCUS to camera sensor. It's the why I add another > CID(V4L2_CID_FOCUS_POSITION). I'm pretty sure that some devices would require a rectangle instead of coordinates to define the focus point. Even a rectangle might not be enough. It would help if we could get feedback from camera designers here. Hans, should we add a new control type to pass coordinates/rectangles ? :-) -- 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