mbox

[GIT,PULL] Versatile LEDs conversion

Message ID CACRpkdb4VhN5Dt6ZoUNgwphxUD7Q69waF9vfO_=SenbvA=WSeA@mail.gmail.com
State New
Headers show

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git

Message

Linus Walleij March 28, 2014, 10:02 a.m. UTC
Hi ARM SoC folks,

here is a Versatile LEDs patch stack, it has been finished for a while,
but delayed waiting for an ACK from the LEDs subsystem maintainer
which arrived today. (My bad for not hunting it more actively.)

I pretty sure the patches are safe and they need some rotation in linux-next,
but I understand if you want to wait until after v3.15-rc1 before pulling this
in. (I'll nag you later in that case.)

Yours,
Linus Walleij

The following changes since commit cfbf8d4857c26a8a307fb7cd258074c9dcd8c691:

  Linux 3.14-rc4 (2014-02-23 17:40:03 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
tags/versatile-leds

for you to fetch changes up to 9b58d187fdce9df05fd331ad5790bf503d7e3dfe:

  ARM: plat-versatile: update defconfigs for Versatile LEDs
(2014-03-28 10:54:31 +0100)

----------------------------------------------------------------
This contains a christmas-decoration fix from Russell
King, converts the Versatile LEDs to a platform device
and moves the driver to the LEDs subsystem.

----------------------------------------------------------------
Linus Walleij (3):
      ARM: plat-versatile: convert LEDs to platform device
      ARM/leds: move ARM Versatile LED driver to leds subsystem
      ARM: plat-versatile: update defconfigs for Versatile LEDs

Russell King (1):
      ARM: plat-versatile: LEDs initialise to off state

 arch/arm/configs/realview-smp_defconfig            |  2 +
 arch/arm/configs/realview_defconfig                |  2 +
 arch/arm/configs/versatile_defconfig               |  3 ++
 arch/arm/mach-realview/core.c                      | 15 +++++++
 arch/arm/mach-realview/core.h                      |  1 +
 arch/arm/mach-realview/realview_eb.c               |  1 +
 arch/arm/mach-realview/realview_pb1176.c           |  1 +
 arch/arm/mach-realview/realview_pb11mp.c           |  1 +
 arch/arm/mach-realview/realview_pba8.c             |  1 +
 arch/arm/mach-realview/realview_pbx.c              |  1 +
 arch/arm/mach-versatile/core.c                     | 16 +++++++
 arch/arm/plat-versatile/Kconfig                    |  6 ---
 arch/arm/plat-versatile/Makefile                   |  1 -
 drivers/leds/Kconfig                               |  8 ++++
 drivers/leds/Makefile                              |  1 +
 .../leds.c => drivers/leds/leds-versatile.c        | 51 ++++++++++++----------
 16 files changed, 82 insertions(+), 29 deletions(-)
 rename arch/arm/plat-versatile/leds.c => drivers/leds/leds-versatile.c (68%)

Comments

Linus Walleij April 29, 2014, 6:19 p.m. UTC | #1
ARM SoC guys:

PING on this pull request!

Yours,
Linus Walleij

On Fri, Mar 28, 2014 at 3:02 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> Hi ARM SoC folks,
>
> here is a Versatile LEDs patch stack, it has been finished for a while,
> but delayed waiting for an ACK from the LEDs subsystem maintainer
> which arrived today. (My bad for not hunting it more actively.)
>
> I pretty sure the patches are safe and they need some rotation in linux-next,
> but I understand if you want to wait until after v3.15-rc1 before pulling this
> in. (I'll nag you later in that case.)
>
> Yours,
> Linus Walleij
>
> The following changes since commit cfbf8d4857c26a8a307fb7cd258074c9dcd8c691:
>
>   Linux 3.14-rc4 (2014-02-23 17:40:03 -0800)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/linusw/linux-integrator.git
> tags/versatile-leds
>
> for you to fetch changes up to 9b58d187fdce9df05fd331ad5790bf503d7e3dfe:
>
>   ARM: plat-versatile: update defconfigs for Versatile LEDs
> (2014-03-28 10:54:31 +0100)
>
> ----------------------------------------------------------------
> This contains a christmas-decoration fix from Russell
> King, converts the Versatile LEDs to a platform device
> and moves the driver to the LEDs subsystem.
>
> ----------------------------------------------------------------
> Linus Walleij (3):
>       ARM: plat-versatile: convert LEDs to platform device
>       ARM/leds: move ARM Versatile LED driver to leds subsystem
>       ARM: plat-versatile: update defconfigs for Versatile LEDs
>
> Russell King (1):
>       ARM: plat-versatile: LEDs initialise to off state
>
>  arch/arm/configs/realview-smp_defconfig            |  2 +
>  arch/arm/configs/realview_defconfig                |  2 +
>  arch/arm/configs/versatile_defconfig               |  3 ++
>  arch/arm/mach-realview/core.c                      | 15 +++++++
>  arch/arm/mach-realview/core.h                      |  1 +
>  arch/arm/mach-realview/realview_eb.c               |  1 +
>  arch/arm/mach-realview/realview_pb1176.c           |  1 +
>  arch/arm/mach-realview/realview_pb11mp.c           |  1 +
>  arch/arm/mach-realview/realview_pba8.c             |  1 +
>  arch/arm/mach-realview/realview_pbx.c              |  1 +
>  arch/arm/mach-versatile/core.c                     | 16 +++++++
>  arch/arm/plat-versatile/Kconfig                    |  6 ---
>  arch/arm/plat-versatile/Makefile                   |  1 -
>  drivers/leds/Kconfig                               |  8 ++++
>  drivers/leds/Makefile                              |  1 +
>  .../leds.c => drivers/leds/leds-versatile.c        | 51 ++++++++++++----------
>  16 files changed, 82 insertions(+), 29 deletions(-)
>  rename arch/arm/plat-versatile/leds.c => drivers/leds/leds-versatile.c (68%)
Olof Johansson May 5, 2014, 4:11 a.m. UTC | #2
On Fri, Mar 28, 2014 at 11:02:57AM +0100, Linus Walleij wrote:
> Hi ARM SoC folks,
> 
> here is a Versatile LEDs patch stack, it has been finished for a while,
> but delayed waiting for an ACK from the LEDs subsystem maintainer
> which arrived today. (My bad for not hunting it more actively.)
> 
> I pretty sure the patches are safe and they need some rotation in linux-next,
> but I understand if you want to wait until after v3.15-rc1 before pulling this
> in. (I'll nag you later in that case.)

Merged into next/cleanup.


-Olof
Linus Walleij May 7, 2014, 1:09 p.m. UTC | #3
On Tue, May 6, 2014 at 5:58 AM, Arnd Bergmann <arnd@arndb.de> wrote:

> I got a build error today, since LEDS_VERSATILE can be built-in while LEDS_CLASS
> is a module:
>
> drivers/built-in.o: In function `versatile_leds_probe':
> :(.text+0x155020): undefined reference to `led_classdev_register'
>
> I suggest we turn this option into 'tristate' so that the dependency
> tracking works correctly.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Quite tricky these bool/tristate things :-/

Yours,
Linus Walleij
Olof Johansson May 11, 2014, 3:13 a.m. UTC | #4
On Wed, May 07, 2014 at 06:09:52AM -0700, Linus Walleij wrote:
> On Tue, May 6, 2014 at 5:58 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> 
> > I got a build error today, since LEDS_VERSATILE can be built-in while LEDS_CLASS
> > is a module:
> >
> > drivers/built-in.o: In function `versatile_leds_probe':
> > :(.text+0x155020): undefined reference to `led_classdev_register'
> >
> > I suggest we turn this option into 'tristate' so that the dependency
> > tracking works correctly.
> >
> > Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> 
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> 
> Quite tricky these bool/tristate things :-/
> 

Applied onto next/cleanup.


-Olof