On Fri, Nov 17, 2023 at 12:38:11PM +0100, Rasmus Villemoes wrote: > From: Doug Zobel <douglas.zo...@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.zo...@climate.com/ > > Cc: Doug Zobel <douglas.zo...@climate.com> > Signed-off-by: Rasmus Villemoes <rasmus.villem...@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
signature.asc
Description: PGP signature
