Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi! > Hi Pali and Tony, > On Wed, Dec 14, 2016 at 07:08:19AM -0800, Tony Lindgren wrote: > > * Pali Rohár [161214 05:38]: > > > On Monday 08 August 2016 23:41:32 Pavel Machek wrote: > > > > On Mon 2016-08-08 11:09:56, Sakari Ailus wrote: > > > > > On Fri, Aug 05, 2016 at

Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi Pali and Tony, On Wed, Dec 14, 2016 at 07:08:19AM -0800, Tony Lindgren wrote: > * Pali Rohár [161214 05:38]: > > On Monday 08 August 2016 23:41:32 Pavel Machek wrote: > > > On Mon 2016-08-08 11:09:56, Sakari Ailus wrote: > > > > On Fri, Aug 05, 2016 at 12:26:11PM +0200,

Re: [PATCHv6] support for AD5820 camera auto-focus coil

* Pali Rohár [161214 05:38]: > On Monday 08 August 2016 23:41:32 Pavel Machek wrote: > > On Mon 2016-08-08 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

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Monday 08 August 2016 23:41:32 Pavel Machek wrote: > On Mon 2016-08-08 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! > >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

* Pavel Machek [161104 00:45]: > Hi! > > > > > I have to admit I mostly use an N9. > > > > > > Well, if you compare that to development on PC, I prefer PC. > > > > > > Even arm development boards are usually easier, as they don't need too > > > complex userspace, and do have

Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi! > > > I have to admit I mostly use an N9. > > > > Well, if you compare that to development on PC, I prefer PC. > > > > Even arm development boards are usually easier, as they don't need too > > complex userspace, and do have working serial ports. > > > > But I do have a serial adapter for

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Thu, Nov 03, 2016 at 11:27:27AM +0100, Pavel Machek wrote: > Hi! > > > > > > > Yeah. I just compiled it but haven't tested it. I presume it'll > > > > > > work. :-) > > > > > > > > > > I'm testing it on n900. I guess simpler hardware with ad5820 would be > > > > > better for the > > > > >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi! > > > > > Yeah. I just compiled it but haven't tested it. I presume it'll work. > > > > > :-) > > > > > > > > I'm testing it on n900. I guess simpler hardware with ad5820 would be > > > > better for the > > > > test... > > > > > > > > What hardware do you have? > > > > > > N900. What

Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi Pavel, On Thu, Aug 18, 2016 at 12:45:39PM +0200, Pavel Machek wrote: > On Thu 2016-08-11 14:16:34, Sakari Ailus wrote: > > On Tue, Aug 09, 2016 at 01:23:23AM +0200, Pavel Machek wrote: > > > On Wed 2016-08-10 15:01:05, Sakari Ailus wrote: > > > > On Mon, Aug 08, 2016 at 11:41:32PM +0200, Pavel

Re: [PATCHv6] support for AD5820 camera auto-focus coil

Hi! > > Heh. Basically anything is easier to develop for than n900 :-(. > > Is it? > > I actually find the old Nokia devices very practical. It's easy to boot your > own kernel and things just work... until musb broke a bit recently. It > requires reconnecting the usb cable again to function. >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Thu 2016-08-11 14:16:34, Sakari Ailus wrote: > On Tue, Aug 09, 2016 at 01:23:23AM +0200, Pavel Machek wrote: > > On Wed 2016-08-10 15:01:05, Sakari Ailus wrote: > > > On Mon, Aug 08, 2016 at 11:41:32PM +0200, Pavel Machek wrote: > > > > On Mon 2016-08-08 11:09:56, Sakari Ailus wrote: > > > > >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Tue, Aug 09, 2016 at 01:23:23AM +0200, Pavel Machek wrote: > On Wed 2016-08-10 15:01:05, Sakari Ailus wrote: > > On Mon, Aug 08, 2016 at 11:41:32PM +0200, Pavel Machek wrote: > > > On Mon 2016-08-08 11:09:56, Sakari Ailus wrote: > > > > On Fri, Aug 05, 2016 at 12:26:11PM +0200, Pavel Machek

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Wed 2016-08-10 15:01:05, Sakari Ailus wrote: > On Mon, Aug 08, 2016 at 11:41:32PM +0200, Pavel Machek wrote: > > On Mon 2016-08-08 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,

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Mon 2016-08-08 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

Re: [PATCHv6] support for AD5820 camera auto-focus coil

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

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Wed, Aug 10, 2016 at 09:34:03AM +0200, Pali Rohár wrote: > 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. > > > >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Wed 2016-08-10 09:34:03, Pali Rohár wrote: > 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,

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Mon, Aug 08, 2016 at 11:41:32PM +0200, Pavel Machek wrote: > On Mon 2016-08-08 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. > > > >

Re: [PATCHv6] support for AD5820 camera auto-focus coil

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.

Re: [PATCHv6] support for AD5820 camera auto-focus coil

On Friday 05 August 2016 12:26:11 Pavel Machek wrote: > > This adds support for AD5820 autofocus coil, found for example in > Nokia N900 smartphone. > > Signed-off-by: Pavel Machek Acked-by: Pali Rohár > --- > v2: simple cleanups, fix error paths, simplify