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". Pawel -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html