[PATCH v2 6/6] led: add TI LP5562 LED driver

Tom Rini trini at konsulko.com
Wed Nov 29 16:23:11 CET 2023


On Fri, Nov 17, 2023 at 12:38:11PM +0100, Rasmus Villemoes wrote:

> From: Doug Zobel <douglas.zobel at climate.com>
> 
> Driver for the TI LP5562 4 channel LED controller. Supports
> independent on/off control of all 4 channels. Supports LED_BLINK on 3
> independent channels: blue/green/red. The white channel can blink, but
> shares the blue channel blink rate.
> 
> Heavily based on patch originally from Doug Zobel [1].
> 
> I have modified it so it matches the DT bindings in the linux tree,
> and also follows the linux driver implementation more closely. This
> should address Tom's concerns, and also matches my goal of making the
> U-Boot driver work with our existing .dts which is known to work in
> linux.
> 
> As our boards only have the R,G,B outputs connected, I have not
> actually tested how the white channel behaves, but the R,G,B work
> exactly as expected.
> 
> [1] https://lore.kernel.org/u-boot/1547150757-1561-1-git-send-email-douglas.zobel@climate.com/
> 
> Cc: Doug Zobel <douglas.zobel at climate.com>
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes at prevas.dk>
> ---
>  doc/device-tree-bindings/leds/leds-lp5562.txt |  63 ++
>  drivers/led/Kconfig                           |   8 +
>  drivers/led/Makefile                          |   1 +
>  drivers/led/led_lp5562.c                      | 578 ++++++++++++++++++
>  4 files changed, 650 insertions(+)
>  create mode 100644 doc/device-tree-bindings/leds/leds-lp5562.txt
>  create mode 100644 drivers/led/led_lp5562.c

If there's no further comments, I'll update the MAINTAINERS file to list
this driver under your name as well, OK?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20231129/63448db9/attachment.sig>


More information about the U-Boot mailing list