linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/3] leds-lp5523: support new LP55231 device
@ 2012-09-04  7:06 Kim, Milo
  2012-09-04  8:05 ` Bryan Wu
  0 siblings, 1 reply; 2+ messages in thread
From: Kim, Milo @ 2012-09-04  7:06 UTC (permalink / raw)
  To: Bryan Wu; +Cc: linux-leds, linux-kernel, Richard Purdie

 New TI LP55231 device has same I2C register layout as LP5523.
 Therefore, all of LED operations can be shared with the LP5523 driver.

 To support new LP55231 device explicitly, the device description
 is added in the configuration file.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
---
 drivers/leds/Kconfig |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig
index 16578d3..0bc80bd 100644
--- a/drivers/leds/Kconfig
+++ b/drivers/leds/Kconfig
@@ -192,11 +192,12 @@ config LEDS_LP5521
 	  programming the engines.
 
 config LEDS_LP5523
-	tristate "LED Support for N.S. LP5523 LED driver chip"
+	tristate "LED Support for TI/National LP5523/55231 LED driver chip"
 	depends on LEDS_CLASS && I2C
 	help
-	  If you say yes here you get support for the National Semiconductor
-	  LP5523 LED driver. It is 9 channel chip with programmable engines.
+	  If you say yes here you get support for TI/National Semiconductor
+	  LP5523/55231 LED driver.
+	  It is 9 channel chip with programmable engines.
 	  Driver provides direct control via LED class and interface for
 	  programming the engines.
 
-- 
1.7.9.5


Best Regards,
Milo


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

* Re: [PATCH 1/3] leds-lp5523: support new LP55231 device
  2012-09-04  7:06 [PATCH 1/3] leds-lp5523: support new LP55231 device Kim, Milo
@ 2012-09-04  8:05 ` Bryan Wu
  0 siblings, 0 replies; 2+ messages in thread
From: Bryan Wu @ 2012-09-04  8:05 UTC (permalink / raw)
  To: Kim, Milo; +Cc: linux-leds, linux-kernel, Richard Purdie

On Tue, Sep 4, 2012 at 3:06 PM, Kim, Milo <Milo.Kim@ti.com> wrote:
>  New TI LP55231 device has same I2C register layout as LP5523.
>  Therefore, all of LED operations can be shared with the LP5523 driver.
>
>  To support new LP55231 device explicitly, the device description
>  is added in the configuration file.
>

Looks good, I will apply this into my for-next

-Bryan

> Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
> ---
>  drivers/leds/Kconfig |    7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig
> index 16578d3..0bc80bd 100644
> --- a/drivers/leds/Kconfig
> +++ b/drivers/leds/Kconfig
> @@ -192,11 +192,12 @@ config LEDS_LP5521
>           programming the engines.
>
>  config LEDS_LP5523
> -       tristate "LED Support for N.S. LP5523 LED driver chip"
> +       tristate "LED Support for TI/National LP5523/55231 LED driver chip"
>         depends on LEDS_CLASS && I2C
>         help
> -         If you say yes here you get support for the National Semiconductor
> -         LP5523 LED driver. It is 9 channel chip with programmable engines.
> +         If you say yes here you get support for TI/National Semiconductor
> +         LP5523/55231 LED driver.
> +         It is 9 channel chip with programmable engines.
>           Driver provides direct control via LED class and interface for
>           programming the engines.
>
> --
> 1.7.9.5
>
>
> 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-09-04  8:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-09-04  7:06 [PATCH 1/3] leds-lp5523: support new LP55231 device Kim, Milo
2012-09-04  8:05 ` 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).