From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pali =?utf-8?B?Um9ow6Fy?= Subject: Re: [PATCHv6] support for AD5820 camera auto-focus coil Date: Wed, 10 Aug 2016 09:34:03 +0200 Message-ID: <20160810073403.GB30047@pali> References: <20160521054336.GA27123@amd> <573FFF51.1000004@gmail.com> <20160521105607.GA20071@amd> <574049EF.2090208@gmail.com> <20160524090433.GA1277@amd> <20160524091746.GA14536@amd> <20160525212659.GK26360@valkosipuli.retiisi.org.uk> <20160527205140.GA26767@amd> <20160805102611.GA13116@amd> <20160808080955.GA3182@valkosipuli.retiisi.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20160808080955.GA3182@valkosipuli.retiisi.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sakari Ailus Cc: Ivaylo Dimitrov , linux-media@vger.kernel.org, khilman@kernel.org, tony@atomide.com, mchehab@osg.samsung.com, aaro.koskinen@iki.fi, sre@kernel.org, kernel list , Pavel Machek , linux-omap@vger.kernel.org, patrikbachan@gmail.com, linux-arm-kernel , serge@hallyn.com List-Id: linux-omap@vger.kernel.org T24gTW9uZGF5IDA4IEF1Z3VzdCAyMDE2IDExOjA5OjU2IFNha2FyaSBBaWx1cyB3cm90ZToKPiBP biBGcmksIEF1ZyAwNSwgMjAxNiBhdCAxMjoyNjoxMVBNICswMjAwLCBQYXZlbCBNYWNoZWsgd3Jv dGU6Cj4gPiAKPiA+IFRoaXMgYWRkcyBzdXBwb3J0IGZvciBBRDU4MjAgYXV0b2ZvY3VzIGNvaWws IGZvdW5kIGZvciBleGFtcGxlIGluCj4gPiBOb2tpYSBOOTAwIHNtYXJ0cGhvbmUuCj4gCj4gVGhh bmtzLCBQYXZlbCEKPiAKPiBMZXQncyB1c2UgVjRMMl9DSURfRk9DVVNfQUJTT0xVVEUsIGFzIGlz IGluIHRoZSBwYXRjaC4gSWYgd2UgZ2V0IHNvbWV0aGluZwo+IGJldHRlciBpbiB0aGUgZnV0dXJl LCB3ZSdsbCBzd2l0Y2ggdG8gdGhhdCB0aGVuLgoKT2ssIGFuZCB3aGF0IHdpdGggQUQ1ODIwX1JB TVBfVElNRSBhbmQgQUQ1ODIwX1JBTVBfTU9ERT8KCj4gSSd2ZSBhcHBsaWVkIHRoaXMgdG8gYWQ1 ODIwIGJyYW5jaCBpbiBteSB0cmVlLgoKQXJlIHlvdSBnb2luZyB0byBmaW5hbGx5IHNlbmQgdGhp cyBwYXRjaCB0byBtZWRpYSB0cmVlIG9yIHB1bGwgcmVxdWVzdAp0byBMaW51cz8KCi0tIApQYWxp IFJvaMOhcgpwYWxpLnJvaGFyQGdtYWlsLmNvbQoKX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX18KbGludXgtYXJtLWtlcm5lbCBtYWlsaW5nIGxpc3QKbGludXgt YXJtLWtlcm5lbEBsaXN0cy5pbmZyYWRlYWQub3JnCmh0dHA6Ly9saXN0cy5pbmZyYWRlYWQub3Jn L21haWxtYW4vbGlzdGluZm8vbGludXgtYXJtLWtlcm5lbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: pali.rohar@gmail.com (Pali =?utf-8?B?Um9ow6Fy?=) Date: Wed, 10 Aug 2016 09:34:03 +0200 Subject: [PATCHv6] support for AD5820 camera auto-focus coil In-Reply-To: <20160808080955.GA3182@valkosipuli.retiisi.org.uk> References: <20160521054336.GA27123@amd> <573FFF51.1000004@gmail.com> <20160521105607.GA20071@amd> <574049EF.2090208@gmail.com> <20160524090433.GA1277@amd> <20160524091746.GA14536@amd> <20160525212659.GK26360@valkosipuli.retiisi.org.uk> <20160527205140.GA26767@amd> <20160805102611.GA13116@amd> <20160808080955.GA3182@valkosipuli.retiisi.org.uk> Message-ID: <20160810073403.GB30047@pali> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Monday 08 August 2016 11:09:56 Sakari Ailus wrote: > On Fri, Aug 05, 2016 at 12:26:11PM +0200, Pavel Machek wrote: > > > > This adds support for AD5820 autofocus coil, found for example in > > Nokia N900 smartphone. > > Thanks, Pavel! > > Let's use V4L2_CID_FOCUS_ABSOLUTE, as is in the patch. If we get something > better in the future, we'll switch to that then. Ok, and what with AD5820_RAMP_TIME and AD5820_RAMP_MODE? > I've applied this to ad5820 branch in my tree. Are you going to finally send this patch to media tree or pull request to Linus? -- Pali Roh?r pali.rohar at gmail.com From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-wm0-f44.google.com ([74.125.82.44]:38302 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932604AbcHJSwf (ORCPT ); Wed, 10 Aug 2016 14:52:35 -0400 Date: Wed, 10 Aug 2016 09:34:03 +0200 From: Pali =?utf-8?B?Um9ow6Fy?= To: Sakari Ailus Cc: Pavel Machek , Ivaylo Dimitrov , sre@kernel.org, kernel list , linux-arm-kernel , linux-omap@vger.kernel.org, tony@atomide.com, khilman@kernel.org, aaro.koskinen@iki.fi, patrikbachan@gmail.com, serge@hallyn.com, linux-media@vger.kernel.org, mchehab@osg.samsung.com Subject: Re: [PATCHv6] support for AD5820 camera auto-focus coil Message-ID: <20160810073403.GB30047@pali> References: <20160521054336.GA27123@amd> <573FFF51.1000004@gmail.com> <20160521105607.GA20071@amd> <574049EF.2090208@gmail.com> <20160524090433.GA1277@amd> <20160524091746.GA14536@amd> <20160525212659.GK26360@valkosipuli.retiisi.org.uk> <20160527205140.GA26767@amd> <20160805102611.GA13116@amd> <20160808080955.GA3182@valkosipuli.retiisi.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160808080955.GA3182@valkosipuli.retiisi.org.uk> Sender: linux-media-owner@vger.kernel.org List-ID: On Monday 08 August 2016 11:09:56 Sakari Ailus wrote: > On Fri, Aug 05, 2016 at 12:26:11PM +0200, Pavel Machek wrote: > > > > This adds support for AD5820 autofocus coil, found for example in > > Nokia N900 smartphone. > > Thanks, Pavel! > > Let's use V4L2_CID_FOCUS_ABSOLUTE, as is in the patch. If we get something > better in the future, we'll switch to that then. Ok, and what with AD5820_RAMP_TIME and AD5820_RAMP_MODE? > I've applied this to ad5820 branch in my tree. Are you going to finally send this patch to media tree or pull request to Linus? -- Pali Rohár pali.rohar@gmail.com