Re: [PATCH v4] media: Driver for Toshiba et8ek8 5MP sensor

[Ivaylo Dimitrov]

On  1.11.2016 22:11, Sakari Ailus wrote:
> 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. :-) ;-)

Oh, sorry, I thought that V(ideo) B(us) S(witch) is clear in the context :)