Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm

From	Linus Walleij <linus.walleij@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm
Date	2017-05-23 00:40 +0200
Message-ID	<tK49A-385-13@gated-at.bofh.it> (permalink)
References	<tJFXA-4dN-17@gated-at.bofh.it> <tJFXA-4dN-15@gated-at.bofh.it> <tJWvo-6NX-9@gated-at.bofh.it> <tJXUu-7yP-25@gated-at.bofh.it> <tK3wS-2DV-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, May 22, 2017 at 11:49 PM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Mon, May 22, 2017 at 5:56 PM, Wolfram Sang <wsa@the-dreams.de> wrote:
>>
>>> Should we also remove the drivers or move them into staging?
>>
>> I'd prefer to keep them. If they are needed again, DT conversion is
>> likely and easier than restarting from scratch. Guenter Roeck also
>> prefers to not unnecessarily annoy people who might have out-of-tree
>> board code. So, as long as they are not painful, let's keep them?
>
> The one reason I can think of for removing them is that we don't
> want ambient light sensor drivers in drivers/misc any more and
> instead of adding DT probing code would also expect new users to
> migrate to drivers/iio/light/, which already has drivers for
> apds9300 and apds9960 but not apds990x, as well as bh1750 and
> bh1780 but not bh1770.

This (apds990x) and bh1770 were added by Samu Onkalo
for Nokia's upstreaming efforts.

Samu, what are the hardware targets using this? Something that
has a userspace etc that we can test?

Like Nokia 900 or so?

Yours,
Linus Walleij

Thread

[PATCH 1/2] misc: apds990x: move header file out of I2C realm Wolfram Sang <wsa@the-dreams.de> - 2017-05-21 22:50 +0200
  Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm Arnd Bergmann <arnd@arndb.de> - 2017-05-22 16:30 +0200
    Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm Wolfram Sang <wsa@the-dreams.de> - 2017-05-22 18:00 +0200
      Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm Arnd Bergmann <arnd@arndb.de> - 2017-05-23 00:00 +0200
        Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm Linus Walleij <linus.walleij@linaro.org> - 2017-05-23 00:40 +0200
        Re: [PATCH 1/2] misc: apds990x: move header file out of I2C realm Wolfram Sang <wsa@the-dreams.de> - 2017-05-23 08:30 +0200

csiph-web