* [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).