From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jacek Anaszewski Subject: Re: [PATCH RESEND 04/16] Documentation: dt-bindings: leds: add LM3633 LED binding information Date: Tue, 03 Nov 2015 17:15:06 +0100 Message-ID: <5638DD8A.5020105@samsung.com> References: <1446441875-1256-1-git-send-email-milo.kim@ti.com> <1446441875-1256-5-git-send-email-milo.kim@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1446441875-1256-5-git-send-email-milo.kim-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Milo Kim Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Milo, Thanks for the patch. Apart from the issues pointed out by Rob, I have one more suggestion below. On 11/02/2015 06:24 AM, Milo Kim wrote: > LM3633 LED device is one of TI LMU device list. > > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Cc: Jacek Anaszewski > Cc: Lee Jones > Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Cc: linux-leds-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Signed-off-by: Milo Kim > --- > .../devicetree/bindings/leds/leds-lm3633.txt | 28 ++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > create mode 100644 Documentation/devicetree/bindings/leds/leds-lm3633.txt > > diff --git a/Documentation/devicetree/bindings/leds/leds-lm3633.txt b/Documentation/devicetree/bindings/leds/leds-lm3633.txt > new file mode 100644 > index 0000000..bb7f213 > --- /dev/null > +++ b/Documentation/devicetree/bindings/leds/leds-lm3633.txt > @@ -0,0 +1,28 @@ > +TI LMU LM3633 LED device tree bindings > + > +Required properties: > + - compatible: "ti,lm3633-leds" > + > +Child nodes: > + Each node matches with LED control bank. > + Please refer to the datasheet [1]. > + > + Required properties of a child node: > + - lvled1-used, lvled2-used, lvled3-used, > + lvled4-used, lvled5-used, lvled6-used: > + Low voltage LED string configuration. Type is . > + Please describe which output LED string is used. > + > + Optional properties of a child node: > + - channel-name: Name string for LED channel identification > + It is used for creating LED sysfs, > + /sys/class/leds//. > + If this property is empty, then default name is set to > + "indicator:" by the driver. > + - led-max-microamp: Max current setting. Type is . > + Unit is microampere. Range is from 5000 to 30000. Could you specify also a step? > +Examples: Please refer to ti-lmu dt-bindings [2]. > + > +[1] http://www.ti.com/product/LM3633/datasheet > +[2] Documentation/devicetree/bindings/mfd/ti-lmu.txt > -- Best Regards, Jacek Anaszewski -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html