From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-ee0-f46.google.com ([74.125.83.46]:46545 "EHLO mail-ee0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750935Ab1LJOmq (ORCPT ); Sat, 10 Dec 2011 09:42:46 -0500 Received: by eekc4 with SMTP id c4so832598eek.19 for ; Sat, 10 Dec 2011 06:42:45 -0800 (PST) Message-ID: <4EE36FE1.1080601@gmail.com> Date: Sat, 10 Dec 2011 15:42:41 +0100 From: Sylwester Nawrocki MIME-Version: 1.0 To: Sakari Ailus CC: linux-media@vger.kernel.org, laurent.pinchart@ideasonboard.com, hverkuil@xs4all.nl, riverful.kim@samsung.com, s.nawrocki@samsung.com, Kyungmin Park Subject: Re: [RFC/PATCH 1/5] v4l: Convert V4L2_CID_FOCUS_AUTO control to a menu control References: <1323011776-15967-1-git-send-email-snjw23@gmail.com> <1323011776-15967-2-git-send-email-snjw23@gmail.com> <20111210103344.GF1967@valkosipuli.localdomain> In-Reply-To: <20111210103344.GF1967@valkosipuli.localdomain> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Sakari, On 12/10/2011 11:33 AM, Sakari Ailus wrote: > On Sun, Dec 04, 2011 at 04:16:12PM +0100, Sylwester Nawrocki wrote: >> Change the V4L2_CID_FOCUS_AUTO control type from boolean to a menu >> type. In case of boolean control we had values 0 and 1 corresponding >> to manual and automatic focus respectively. >> >> The V4L2_CID_FOCUS_AUTO menu control has currently following items: >> 0 - V4L2_FOCUS_MANUAL, >> 1 - V4L2_FOCUS_AUTO, >> 2 - V4L2_FOCUS_AUTO_MACRO, >> 3 - V4L2_FOCUS_AUTO_CONTINUOUS. > > I would put the macro mode to a separate menu since it's configuration for > how the regular AF works rather than really different mode. Yes, makes sense. Most likely there could be also continuous macro auto focus.. I don't have yet an idea what could be a name for that new menu though. Many Samsung devices have also something like guided auto focus, where the application can specify location in the frame for focusing on. IIRC this could be also single-shot or continuous. So it could make sense to group MACRO and "guided" auto focus in one menu, what do you think ? >> @@ -567,6 +576,7 @@ const char *v4l2_ctrl_get_name(u32 id) >> case V4L2_CID_PRIVACY: return "Privacy"; >> case V4L2_CID_IRIS_ABSOLUTE: return "Iris, Absolute"; >> case V4L2_CID_IRIS_RELATIVE: return "Iris, Relative"; >> + case V4L2_CID_DO_AUTO_FOCUS: return "Do Auto Focus"; > > I'd perhaps use "begin" or "start". How does the user learn the autofocus I agree, it's not something than is finished after G_CTRL call returns. V4L2_CID_START_AUTO_FOCUS sounds better to me. > has finished? I think this looks like quite similar problem than telling the > flash strobe status to the user. The solution could also be similar to that. I guess you're suggesting an auto focus status control? Together with the control events it would be nice interface for notifying the applications. -- Regards, Sylwester