[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20171112142918.rqxd2uk7ljyhpctt@earth>
Date: Sun, 12 Nov 2017 15:29:18 +0100
From: Sebastian Reichel <sre@...nel.org>
To: Pavel Machek <pavel@....cz>
Cc: pali.rohar@...il.com, kernel list <linux-kernel@...r.kernel.org>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>,
linux-omap@...r.kernel.org, tony@...mide.com, khilman@...nel.org,
aaro.koskinen@....fi, ivo.g.dimitrov.75@...il.com,
patrikbachan@...il.com, serge@...lyn.com, abcloriens@...il.com,
clayton@...ftyguy.net, martijn@...xit.nl,
sakari.ailus@...ux.intel.com
Subject: Re: [PATCH] omap3-n900: enable autofocus
Hi,
On Sun, Nov 12, 2017 at 12:25:29PM +0100, Pavel Machek wrote:
>
> Connect auto-focus coil to the camera sensor, so that it can be used
> by camera applications.
>
> Signed-off-by: Pavel Machek <pavel@....cz>
I think you should replace auto-focus with lens-focus. This is not
automatic. Otherwise:
Reviewed-by: Sebastian Reichel <sebastian.reichel@...labora.co.uk>
-- Sebastian
> diff --git a/arch/arm/boot/dts/omap3-n900.dts b/arch/arm/boot/dts/omap3-n900.dts
> index 669c51c..59cc743 100644
> --- a/arch/arm/boot/dts/omap3-n900.dts
> +++ b/arch/arm/boot/dts/omap3-n900.dts
> @@ -778,6 +778,8 @@
>
> reset-gpio = <&gpio4 6 GPIO_ACTIVE_HIGH>; /* 102 */
>
> + lens-focus = <&ad5820>;
> +
> port {
> csi_cam1: endpoint {
> bus-type = <3>; /* CCP2 */
>
> --
> (english) http://www.livejournal.com/~pavelmachek
> (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)
Powered by blists - more mailing lists