2017-11-19 14:13:40

by Ladislav Michl

[permalink] [raw]
Subject: Re: bh1770 hacks: get it to work on n950

On Sun, Nov 19, 2017 at 01:41:50PM +0100, Pavel Machek wrote:
> On Fri 2017-11-17 12:35:40, Sebastian Reichel wrote:
> > Hi,
> >
> > On Thu, Nov 16, 2017 at 12:57:03PM +0100, Pavel Machek wrote:
> > > Hi!
> > >
> > > Ok, not for application, but I can get light sensor to work.
> > >
> > > Signed-off-by: Pavel Machek <[email protected]>
> >
> > There is no mainline user of this driver and it has a custom API, so
> > I think the driver should be dropped and an IIO based one should be
> > created instead.
>
> While I agree IIO interface might be better, and would welcome
> patches, I don't think that dropping the driver is right action at
> this point.

That's the most elegant solution at all. How would you later motivate
people to try and test IIO driver? Let's pretend this patch have never
existed until someone notices and starts playing "you broke kernel ABI"
game.

Thank you,
ladis

From 1584498345659948519@xxx Sun Nov 19 12:42:56 +0000 2017
X-GM-THRID: 1584233053474950413
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread