Skip to content

leds: Add IS32FL3207 controller support

Ahmad Byagowi linux-devicetree linux-hardening linux-leds lkml 7 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v7 Latest2 patches 3 repliesIn review
  2. v62 patches 0 repliesSuperseded
  3. v57 patches 13 repliesSuperseded
Earlier versions · 4
VersionPostedPatchesRepliesStatus
v45 patches1 replySuperseded
v35 patches1 replySuperseded
v2 RFC6 patches3 repliesSuperseded
v1 RFC4 patches0 repliesSuperseded

Changes in v7 from the cover letter

- Restore the led-max-microamp schema minimum to 1 uA because the smallest representable nonzero current depends on the RISET value.
- Document the RISET-dependent nonzero current-scaling requirement for individual and multicolor outputs.
- Keep the driver patch unchanged.

Patches in v7 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2dt-bindings: leds: Add IS32FL3207 controllerDocumentation/devicetree/bindings/leds/issi,is32fl3207.yaml MAINTAINERS+291 -01 reply
2/2leds: is32fl3207: Add controller driverMAINTAINERS drivers/leds/rgb/Kconfig drivers/leds/rgb/Makefile +1 more+749 -02 replies

Reviews trailers per patch

1 of 2 patches has a Reviewed-by or Acked-by.

PatchKrzysztof Kozlowski
1/2R-b
2/2

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v7 thread

Cover letter [PATCH v7 0/2] leds: Add IS32FL3207 controller support

Add a binding and LED-class driver for the Lumissil IS32FL3207 18-channel constant-current LED controller. The driver supports individual and multicolor LEDs, per-output current limits, an optional supply, and an optional SDB GPIO.

This is the LED prerequisite split out of the Time Card series. The dependent ptp_ocp board profiles will be resent to net-next after the LED and I2C prerequisites land.

---

Changes since v6:
- Restore the led-max-microamp schema minimum to 1 uA because the smallest representable nonzero current depends on the RISET value.
- Document the RISET-dependent nonzero current-scaling requirement for individual and multicolor outputs.
- Keep the driver patch unchanged.

Krzysztof's Reviewed-by from v3 is not carried because the binding changed materially after that review. It now supports groups beginning at outputs 16 and 17 and documents the RISET/current-limit relationship.

Validation:
- dt-doc-validate
- dt-check-style --mode strict
- yamllint
- git diff --check
- strict checkpatch
- x86-64 Clang syntax check

v6: https://lore.kernel.org/r/[email protected]/

Ahmad Byagowi (2):
dt-bindings: leds: Add IS32FL3207 controller
leds: is32fl3207: Add controller driver

 .../bindings/leds/issi,is32fl3207.yaml        | 285 +++++++
 MAINTAINERS                                   |   7 +
 drivers/leds/rgb/Kconfig                      |  11 +
 drivers/leds/rgb/Makefile                     |   1 +
 drivers/leds/rgb/leds-is32fl3207.c            | 736 ++++++++++++++++++
 5 files changed, 1040 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/leds/issi,is32fl3207.yaml
 create mode 100644 drivers/leds/rgb/leds-is32fl3207.c

base-commit: f8cca63a0a4f3475199d9ab7f86782bdfeb0744d

Signature · 1 line
2.50.1 (Apple Git-155)