[PATCH v5 0/5] leds: add new LED driver for TI LP5812

Nam Tran posted 5 patches 8 months, 1 week ago
There is a newer version of this series
.../ABI/testing/sysfs-bus-i2c-devices-lp5812  |  144 +
.../devicetree/bindings/leds/ti,lp5812.yaml   |  119 +
Documentation/leds/leds-lp5812.rst            |   79 +
MAINTAINERS                                   |   12 +
.../arm/boot/dts/broadcom/bcm2711-rpi-4-b.dts |   60 +
drivers/leds/Kconfig                          |   16 +
drivers/leds/Makefile                         |    1 +
drivers/leds/leds-lp5812.c                    | 2797 +++++++++++++++++
drivers/leds/leds-lp5812.h                    |  350 +++
9 files changed, 3578 insertions(+)
create mode 100644 Documentation/ABI/testing/sysfs-bus-i2c-devices-lp5812
create mode 100644 Documentation/devicetree/bindings/leds/ti,lp5812.yaml
create mode 100644 Documentation/leds/leds-lp5812.rst
create mode 100644 drivers/leds/leds-lp5812.c
create mode 100644 drivers/leds/leds-lp5812.h
[PATCH v5 0/5] leds: add new LED driver for TI LP5812
Posted by Nam Tran 8 months, 1 week ago
This patch series adds support for the TI/National Semiconductor LP5812
4x3 matrix RGB LED driver. The driver supports features such as autonomous
animation and time-cross-multiplexing (TCM) for dynamic LED effects.

Signed-off-by: Nam Tran <trannamatk@gmail.com>
---
Changes in v5:
- Rebase on v6.15-rc2
- Removed unused functions (lp5812_dump_regs, lp5812_update_bit).
- Address Krzysztof's review comments
- Link to v4: https://lore.kernel.org/linux-leds/20250405183246.198568-1-trannamatk@gmail.com/

---
Nam Tran (5):
  dt-bindings: leds: add TI/National Semiconductor LP5812 LED Driver
  leds: add TI/National Semiconductor LP5812 LED Driver
  docs: ABI: Document LP5812 LED sysfs interfaces
  docs: leds: Document TI LP5812 LED driver
  arm64: dts: Add LP5812 LED node for Raspberry Pi 4 Model B

 .../ABI/testing/sysfs-bus-i2c-devices-lp5812  |  144 +
 .../devicetree/bindings/leds/ti,lp5812.yaml   |  119 +
 Documentation/leds/leds-lp5812.rst            |   79 +
 MAINTAINERS                                   |   12 +
 .../arm/boot/dts/broadcom/bcm2711-rpi-4-b.dts |   60 +
 drivers/leds/Kconfig                          |   16 +
 drivers/leds/Makefile                         |    1 +
 drivers/leds/leds-lp5812.c                    | 2797 +++++++++++++++++
 drivers/leds/leds-lp5812.h                    |  350 +++
 9 files changed, 3578 insertions(+)
 create mode 100644 Documentation/ABI/testing/sysfs-bus-i2c-devices-lp5812
 create mode 100644 Documentation/devicetree/bindings/leds/ti,lp5812.yaml
 create mode 100644 Documentation/leds/leds-lp5812.rst
 create mode 100644 drivers/leds/leds-lp5812.c
 create mode 100644 drivers/leds/leds-lp5812.h


base-commit: 8ffd015db85fea3e15a77027fda6c02ced4d2444
-- 
2.25.1
Re: [PATCH v5 0/5] leds: add new LED driver for TI LP5812
Posted by Krzysztof Kozlowski 8 months, 1 week ago
On 14/04/2025 16:57, Nam Tran wrote:
> This patch series adds support for the TI/National Semiconductor LP5812
> 4x3 matrix RGB LED driver. The driver supports features such as autonomous
> animation and time-cross-multiplexing (TCM) for dynamic LED effects.
> 
> Signed-off-by: Nam Tran <trannamatk@gmail.com>
> ---
> Changes in v5:
> - Rebase on v6.15-rc2
> - Removed unused functions (lp5812_dump_regs, lp5812_update_bit).
> - Address Krzysztof's review comments
Which ones? What changed?

Best regards,
Krzysztof
Re: [PATCH v5 0/5] leds: add new LED driver for TI LP5812
Posted by Nam Tran 8 months ago
On Mon, 14 Apr 2025, Krzysztof Kozlowski wrote:

>On 14/04/2025 16:57, Nam Tran wrote:
>> This patch series adds support for the TI/National Semiconductor LP5812
>> 4x3 matrix RGB LED driver. The driver supports features such as autonomous
>> animation and time-cross-multiplexing (TCM) for dynamic LED effects.
>> 
>> Signed-off-by: Nam Tran <trannamatk@gmail.com>
>> ---
>> Changes in v5:
>> - Rebase on v6.15-rc2
>> - Removed unused functions (lp5812_dump_regs, lp5812_update_bit).
>> - Address Krzysztof's review comments
>Which ones? What changed?

Apologies for the delayed reply.

I’ve updated the ABI document based on your previous comments.
Regarding the binding, I’ve removed redundant descriptions and fixed the indentation as you pointed out.

I’ll make sure to describe the changes more clearly in the next patch submission.

Best regards,
Nam Tran