linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/28] leds: add common driver for LP5521/5523/55231
@ 2012-10-05  8:09 Kim, Milo
  2012-10-12 16:08 ` Bryan Wu
  0 siblings, 1 reply; 2+ messages in thread
From: Kim, Milo @ 2012-10-05  8:09 UTC (permalink / raw)
  To: Bryan Wu; +Cc: Richard Purdie, linux-leds, linux-kernel

 LP5521, LP5523 and L55231 have common features as below.

 Register access via the I2C.
 Device initialization/de-initialization.
 Create LED class devices for multiple output channels.
 Device attributes for user-space interface.

 Therefore, this series of patch set enables removing duplicate code for
 lp5521/5523 driver and making coherent driver structure.

 All patches have been tested in LP5521, LP5523 and LP55231 boards.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Tested-by: Milo(Woogyom) Kim <milo.kim@ti.com>
---
 Documentation/leds/00-INDEX                  |    2 +
 Documentation/leds/leds-lp5521.txt           |   63 +-
 Documentation/leds/leds-lp5523.txt           |   27 +-
 Documentation/leds/leds-lp55xx.txt           |  104 +++
 arch/arm/mach-omap2/board-rx51-peripherals.c |    8 +-
 arch/arm/mach-ux500/board-mop500.c           |   14 +-
 drivers/leds/Kconfig                         |   15 +-
 drivers/leds/Makefile                        |    1 +
 drivers/leds/leds-lp5521.c                   |  928 +++++++---------------
 drivers/leds/leds-lp5523.c                   | 1065 +++++++-------------------
 drivers/leds/leds-lp55xx-common.c            |  516 +++++++++++++
 drivers/leds/leds-lp55xx-common.h            |  138 ++++
 include/linux/leds-lp5521.h                  |   73 --
 include/linux/leds-lp5523.h                  |   49 --
 include/linux/platform_data/leds-lp55xx.h    |   55 ++
 15 files changed, 1374 insertions(+), 1684 deletions(-)
 create mode 100644 Documentation/leds/leds-lp55xx.txt
 create mode 100644 drivers/leds/leds-lp55xx-common.c
 create mode 100644 drivers/leds/leds-lp55xx-common.h
 delete mode 100644 include/linux/leds-lp5521.h
 delete mode 100644 include/linux/leds-lp5523.h
 create mode 100644 include/linux/platform_data/leds-lp55xx.h

Best Regards,
Milo



^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 00/28] leds: add common driver for LP5521/5523/55231
  2012-10-05  8:09 [PATCH 00/28] leds: add common driver for LP5521/5523/55231 Kim, Milo
@ 2012-10-12 16:08 ` Bryan Wu
  0 siblings, 0 replies; 2+ messages in thread
From: Bryan Wu @ 2012-10-12 16:08 UTC (permalink / raw)
  To: Kim, Milo; +Cc: Richard Purdie, linux-leds, linux-kernel

Sorry for the delay. These days I'm moving and I will review this
patchset after I settle down.

Thanks a lot,
-Bryan

On Fri, Oct 5, 2012 at 4:09 PM, Kim, Milo <Milo.Kim@ti.com> wrote:
>  LP5521, LP5523 and L55231 have common features as below.
>
>  Register access via the I2C.
>  Device initialization/de-initialization.
>  Create LED class devices for multiple output channels.
>  Device attributes for user-space interface.
>
>  Therefore, this series of patch set enables removing duplicate code for
>  lp5521/5523 driver and making coherent driver structure.
>
>  All patches have been tested in LP5521, LP5523 and LP55231 boards.
>
> Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
> Tested-by: Milo(Woogyom) Kim <milo.kim@ti.com>
> ---
>  Documentation/leds/00-INDEX                  |    2 +
>  Documentation/leds/leds-lp5521.txt           |   63 +-
>  Documentation/leds/leds-lp5523.txt           |   27 +-
>  Documentation/leds/leds-lp55xx.txt           |  104 +++
>  arch/arm/mach-omap2/board-rx51-peripherals.c |    8 +-
>  arch/arm/mach-ux500/board-mop500.c           |   14 +-
>  drivers/leds/Kconfig                         |   15 +-
>  drivers/leds/Makefile                        |    1 +
>  drivers/leds/leds-lp5521.c                   |  928 +++++++---------------
>  drivers/leds/leds-lp5523.c                   | 1065 +++++++-------------------
>  drivers/leds/leds-lp55xx-common.c            |  516 +++++++++++++
>  drivers/leds/leds-lp55xx-common.h            |  138 ++++
>  include/linux/leds-lp5521.h                  |   73 --
>  include/linux/leds-lp5523.h                  |   49 --
>  include/linux/platform_data/leds-lp55xx.h    |   55 ++
>  15 files changed, 1374 insertions(+), 1684 deletions(-)
>  create mode 100644 Documentation/leds/leds-lp55xx.txt
>  create mode 100644 drivers/leds/leds-lp55xx-common.c
>  create mode 100644 drivers/leds/leds-lp55xx-common.h
>  delete mode 100644 include/linux/leds-lp5521.h
>  delete mode 100644 include/linux/leds-lp5523.h
>  create mode 100644 include/linux/platform_data/leds-lp55xx.h
>
> Best Regards,
> Milo
>
>



-- 
Bryan Wu <bryan.wu@canonical.com>
Kernel Developer    +86.186-168-78255 Mobile
Canonical Ltd.      www.canonical.com
Ubuntu - Linux for human beings | www.ubuntu.com

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-12 16:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-10-05  8:09 [PATCH 00/28] leds: add common driver for LP5521/5523/55231 Kim, Milo
2012-10-12 16:08 ` Bryan Wu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).