[PATCHv2 2.6.40 3/3] ARM:simpad: add GPIO based device definitions.

Dmitry Artamonow mad_soft at inbox.ru
Tue Apr 19 09:52:10 EDT 2011


On 15:00 Mon 18 Apr     , Jochen Friedrich wrote:
> Register LED, keyboard, polled keyboard and I2C platform
> devices based on GPIOs.
> 
> V2: Remove the legacy ARM LED code for simpad devices.

Yeah, that looks like a good move! IMO, legacy ARM leds code should go
away, as we have now proper generic LED subsystem, and the sooner we
convert all legacy users to leds-class drivers, the better.

> Signed-off-by: Jochen Friedrich <jochen at scram.de>
> ---
>  arch/arm/mach-sa1100/Makefile      |    1 -
>  arch/arm/mach-sa1100/leds-simpad.c |   97 ------------------------------------
>  arch/arm/mach-sa1100/simpad.c      |   95 ++++++++++++++++++++++++++++++++++-
>  3 files changed, 94 insertions(+), 99 deletions(-)
>  delete mode 100644 arch/arm/mach-sa1100/leds-simpad.c

But, while removing leds-simpad.c, it seems you forgot to update both
Makefile and corresponding entries in leds.[ch], so I suspect it will
break build with CONFIG_LEDS=y.

And, while you're at it, it's probably a good idea to move all these LED
changes into separate patch, and not to mix them with gpio-keys/i2c/etc
devices registration.

-- 
Best regards,
Dmitry "MAD" Artamonow




More information about the linux-arm-kernel mailing list