linux-kernel - Re: [PATCH 1/1] v4l: Add camera voice coil lens control class, current control

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160617221149.GB31380@amd>
Date:	Sat, 18 Jun 2016 00:11:49 +0200
From:	Pavel Machek <pavel@....cz>
To:	Ivaylo Dimitrov <ivo.g.dimitrov.75@...il.com>
Cc:	Sakari Ailus <sakari.ailus@...ux.intel.com>, pali.rohar@...il.com,
	sre@...nel.org, linux-kernel@...r.kernel.org,
	linux-media@...r.kernel.org, hverkuil@...all.nl,
	laurent.pinchart@...asonboard.com,
	Sakari Ailus <sakari.ailus@....fi>
Subject: Re: [PATCH 1/1] v4l: Add camera voice coil lens control class,
 current control

Hi!

> >I also don't think the FOCUS_ABSOLUTE controls is a really good one for
> >the voice coil lens current. I expect more voice coil lens controls
> >(linear vs. PWM mode, ringing compensation...) to be needed so I think
> >it's worth a new class.

Well, I do agree that that the new class is fine (and have no problem
with your proposal), but I'd like to understand why FOCUS_ABSOLUTE is
unsuitable. I'm actually thinking about writing very simple userspace
camera, without autofocus, where user would just select
"infinity"/"1m"/"50cm" ... and for that FOCUS_ABSOLUTE is equivalent
to VOICE_COIL_CURRENT...

> Right, I still think movement mode should be standard control :)

What options would you like to see for movement mode?

Thanks,
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html