Thread (17 messages) flat view 17 messages, 4 authors, 2018-06-21

Re: [PATCH v2 1/2] dt-bindings: leds: Add SC27xx breathing light controller documentation

From: Pavel Machek <hidden>
Date: 2018-05-09 14:25:30
Also in: linux-leds, lkml

Hi!
quoted hunk ↗ jump to hunk
This patch adds the binding documentation for Spreadtrum SC27xx series
breathing light controller, which supports 3 outputs: red LED, green
LED and blue LED.

Signed-off-by: Baolin Wang <redacted>
---
Changes since v1:
 - Change the compatible string to be one explicit SoC name.
 - Change the child node name.
 - Change to be upper case for the first character.
---
 .../devicetree/bindings/leds/leds-sc27xx-bltc.txt  |   41 ++++++++++++++++++++
 1 file changed, 41 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/leds/leds-sc27xx-bltc.txt
diff --git a/Documentation/devicetree/bindings/leds/leds-sc27xx-bltc.txt b/Documentation/devicetree/bindings/leds/leds-sc27xx-bltc.txt
new file mode 100644
index 0000000..b3de7fc
--- /dev/null
+++ b/Documentation/devicetree/bindings/leds/leds-sc27xx-bltc.txt
@@ -0,0 +1,41 @@
+LEDs connected to Spreadtrum SC27XX PMIC breathing light controller
+
+The SC27xx breathing light controller supports to 3 outputs:
+red LED, green LED and blue LED. Each LED can work at normal
+PWM mode or breath light mode.
+Required properties:
+- compatible: Should be "sprd,sc2731-bltc".
If this is for 2731, I'd say so in the title and the filename...?
+- #address-cells: Must be 1.
+- #size-cells: Must be 0.
+- reg: Specify controller address.
the controller...
+Required child properties:
+- reg: Number of LED line (could be from 0 to 2).
Uff. "Port this LED is connected to"?

With that fixed...

Acked-by: Pavel Machek <redacted>

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help