Re: [PATCH 2/3] ARM/leds: move ARM Versatile LED driver to leds subsystem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Вторник, 11 марта 2014, 12:23 UTC от Pawel Moll <pawel.moll@xxxxxxx>:
> On Tue, 2014-03-11 at 12:18 +0000, Linus Walleij wrote:
> > On Thu, Feb 27, 2014 at 3:07 PM, Alexander Shiyan <shc_work@xxxxxxx> wrote:
> > > Четверг, 27 февраля 2014, 14:58 +01:00 от Linus Walleij <linus.walleij@xxxxxxxxxx>:
> > >> Now that we have converted this driver to a real platform device
> > >> module-based thing, we move the driver down into the LEDs
> > >> subsystem and rename the config option to LEDS_VERSATILE.
> > >
> > > In fact, it can be converted to use basic-mmio-gpio => leds-gpio.
> > 
> > Hm, yeah I see what you mean. However this register is not described
> > as a GPIO register, and on all Versatile/RealView boards these signals
> > are soldered to LEDs, so they are not general purpose at all.
> > 
> > On some systems without "real" GPIO the above would lead to
> > compiling in the entire gpiolib (149 KB) just to do this.
> > 
> > I would agree more with inventing something like leds-mmio as a
> > separate refactoring after this, i.e. a driver for any memory-mapped
> > LED, which should cover a few cases. What do you think about this
> > idea?
> 
> Something like this?
> http://thread.gmane.org/gmane.linux.kernel/1387110 ;-)
> 
> But seriously speaking, I did exactly what Alexander mentioned on VE,
> considering the SYS_LED & alikes as "pseudo-gpios".

It turns out everything has already been invented :)

---
��.n��������+%������w��{.n�����{��W����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f





[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux