Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1609884
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder |
| Date | 2017-03-27 16:30 +0200 |
| Message-ID | <tpDEZ-QR-7@gated-at.bofh.it> (permalink) |
| References | <toy4F-2Fu-3@gated-at.bofh.it> <toyo1-2Pm-9@gated-at.bofh.it> <toyxI-2SC-13@gated-at.bofh.it> <tozk6-3uM-35@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On Fri, Mar 24, 2017 at 4:19 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Andy,
>
> On Fri, Mar 24, 2017 at 3:29 PM, Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
>> On Fri, 2017-03-24 at 15:19 +0100, Geert Uytterhoeven wrote:
>>> On Fri, Mar 24, 2017 at 3:06 PM, Andy Shevchenko
>>> <andriy.shevchenko@linux.intel.com> wrote:
>>> > It looks like panel.c belongs to auxdisplay subsystem.
>>> >
>>> > Move it to drivers/auxdisplay folder.
>>> > No functional changes intended.
>>>
>>> I didn't move it to drivers/auxdisplay/ because it not only provides
>>> auxdisplay functionality, but also keypad functionality.
>>
>> Yes, I have noticed this. But keypad functionality is optional while
>> panel is main one. I think you would agree on this.
>
> I don't care that much, though. Willy?
I think moving all of it to drivers/auxdisplay as it is today, and doing
any further cleanup on the driver over there is a good idea.
On a related note, how about moving arm-charlcd there as well,
and finding a new place for the various ALS drivers?
Arnd
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-03-24 15:10 +0100
[PATCH v1 2/2] auxdisplay: Move arm-charlcd.c to drivers/auxdisplay folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-03-24 15:10 +0100
Re: [PATCH v1 2/2] auxdisplay: Move arm-charlcd.c to drivers/auxdisplay folder Linus Walleij <linus.walleij@linaro.org> - 2017-03-24 16:20 +0100
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-24 15:30 +0100
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-03-24 15:40 +0100
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-24 16:30 +0100
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Willy TARREAU <wtarreau@haproxy.com> - 2017-03-27 10:20 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-27 10:30 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Willy TARREAU <wtarreau@haproxy.com> - 2017-03-27 11:30 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-03-30 13:30 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Willy TARREAU <wtarreau@haproxy.com> - 2017-03-30 15:00 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2017-04-04 18:30 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-27 16:20 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Arnd Bergmann <arnd@arndb.de> - 2017-03-27 16:30 +0200
Re: [PATCH v1 1/2] auxdisplay: Move panel.c to drivers/auxdisplay folder Geert Uytterhoeven <geert@linux-m68k.org> - 2017-03-30 13:50 +0200
csiph-web