Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1391214

Re: [GIT PULL] move ARM LCD display driver to auxdisplay

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

On 28/04/16 12:26, Maxime Ripard wrote:
> On Tue, Apr 26, 2016 at 12:41:29PM +0200, Linus Walleij wrote:
>> On Mon, Apr 25, 2016 at 1:57 PM, Maxime Ripard
>> <maxime.ripard@free-electrons.com> wrote:
>>> On Sat, Apr 23, 2016 at 11:43:58AM +0200, Linus Walleij wrote:
>>>> Hi ARM SoC guys,
>>>>
>>>> these two patches move the ARM character LCD driver from
>>>> misc drivers to the auxdisplay subsystem where it belongs and
>>>> updates the defconfig for the RealView accordingly.
>>>>
>>>> Please pull it into some cleanup branch in the ARM SoC
>>>> tree.
>>>>
>>>> I tried to get some ACK from the auxdisplay maintainer but no
>>>> reaction.
>>>
>>> Auxdisplay is not a new framework, but rather a very old one.
>>>
>>> It got introduced in 2.6.21, and only received a couple of drivers
>>> since.
>>
>> Small problem for me, but Robin is submitting a brand new
>> auxdisplay driver:
>> http://marc.info/?l=linux-kernel&m=146001950512999&w=2
>> http://marc.info/?l=linux-kernel&m=146001988013230&w=2
> 
> My understanding is that Tomi doesn't want any new fbdev drivers, so
> I'm not sure this is the right place for those patches.

Hmm, that one looks like an fbdev driver, not an auxdisplay driver. But,
not relevant for this patch, of course.

>>> It seems to be quite redundant with fbdev deferred_io, which itself is
>>> almost deprecated these days.
>>
>> That is a valid argument for moving cfag12864b* out to
>> drivers/video/* I guess, but it has nothing to do with this
>> driver whatsoever.
>>
>> This driver has nothing to do with fbdev. Nothing at all.
>> It so not connected to any framebuffer.
>>
>> This driver gets moved to auxdisplay to match the existing
>> LCD driver in drivers/auxdisplay/ks0108.c which is a similar
>> hardware, driving an LCD that likewise has nothing to do
>> with fbdev.
> 
> My bad, I thought all auxdisplay drivers were fbdev ones, but I was
> wrong, obviously. And now, I don't really know what auxdisplay is
> about anymore :)
> 
> Still, moving it to a "framework" that has been inactive for so long
> (and you even mention it in your PR) doesn't seem like the right move.

I think this move makes sense. It's only moving a file to a directory
that at least mentions "display", not really moving it into a framework
(because there doesn't seem to be any).

 Tomi

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [GIT PULL] move ARM LCD display driver to auxdisplay Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-04-29 17:00 +0200
  Re: [GIT PULL] move ARM LCD display driver to auxdisplay Robin van der Gracht <robin@protonic.nl> - 2016-05-09 09:10 +0200
    Re: [GIT PULL] move ARM LCD display driver to auxdisplay Linus Walleij <linus.walleij@linaro.org> - 2016-05-09 15:20 +0200
      Re: [GIT PULL] move ARM LCD display driver to auxdisplay Robin van der Gracht <robin@protonic.nl> - 2016-05-09 16:20 +0200
        Re: [GIT PULL] move ARM LCD display driver to auxdisplay Linus Walleij <linus.walleij@linaro.org> - 2016-05-09 16:30 +0200
          Re: [GIT PULL] move ARM LCD display driver to auxdisplay Greg KH <gregkh@linuxfoundation.org> - 2016-05-10 09:20 +0200
            Re: [GIT PULL] move ARM LCD display driver to auxdisplay Robin van der Gracht <robin@protonic.nl> - 2016-05-11 09:50 +0200
              Re: [GIT PULL] move ARM LCD display driver to auxdisplay Greg KH <gregkh@linuxfoundation.org> - 2016-05-11 10:40 +0200
          Re: [GIT PULL] move ARM LCD display driver to auxdisplay Robin van der Gracht <robin@protonic.nl> - 2016-05-12 12:30 +0200

csiph-web