[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20161101201122.GF3217@valkosipuli.retiisi.org.uk>
Date: Tue, 1 Nov 2016 22:11:23 +0200
From: Sakari Ailus <sakari.ailus@....fi>
To: Ivaylo Dimitrov <ivo.g.dimitrov.75@...il.com>
Cc: Pavel Machek <pavel@....cz>, sre@...nel.org, pali.rohar@...il.com,
linux-media@...r.kernel.org, galak@...eaurora.org,
mchehab@....samsung.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor
Hi Ivaylo,
On Tue, Nov 01, 2016 at 08:36:57AM +0200, Ivaylo Dimitrov wrote:
> Hi,
>
> On 1.11.2016 00:54, Sakari Ailus wrote:
> >Hi Pavel,
> >
> >On Sun, Oct 23, 2016 at 10:33:15PM +0200, Pavel Machek wrote:
> >>Hi!
> >>
> >>>Thanks, this answered half of my questions already. ;-)
> >>
> >>:-).
> >>
> >>I'll have to go through the patches, et8ek8 driver is probably not
> >>enough to get useful video. platform/video-bus-switch.c is needed for
> >>camera switching, then some omap3isp patches to bind flash and
> >>autofocus into the subdevice.
> >>
> >>Then, device tree support on n900 can be added.
> >
> >I briefly discussed with with Sebastian.
> >
> >Do you think the elusive support for the secondary camera is worth keeping
> >out the main camera from the DT in mainline? As long as there's a reasonable
> >way to get it working, I'd just merge that. If someone ever gets the
> >secondary camera working properly and nicely with the video bus switch,
> >that's cool, we'll somehow deal with the problem then. But frankly I don't
> >think it's very useful even if we get there: the quality is really bad.
> >
>
> Yes, lets merge what we have till now, it will be way easier to improve on
> it once it is part of the mainline.
>
> BTW, I have (had) patched VBS working almost without problems, when it comes
> to it I'll dig it.
I wonder if I'm the only one who wonders what VBS is here. Don't tell me its
the old MS thing. :-) ;-)
--
Kind regards,
Sakari Ailus
e-mail: sakari.ailus@....fi XMPP: sailus@...iisi.org.uk
Powered by blists - more mailing lists