Hi! > >> index 000000000000..3256dec21075 > >> --- /dev/null > >> +++ b/Documentation/devicetree/bindings/leds/leds-lm3697.txt > >> @@ -0,0 +1,86 @@ > >> +* Texas Instruments - LM3697 Highly Efficient White LED Driver > >> + > >> +The LM3697 11-bit LED driver provides high- > >> +performance backlight dimming for 1, 2, or 3 series > >> +LED strings while delivering up to 90% efficiency. > > > > Hmm. We already have second set of bindings for 3697: > > > > ./devicetree/bindings/mfd/ti-lmu.txt > > > > (Sorry for not noticing that earlier). Advantage is that those have > > had discussion with device tree people and have acks: > > > > What to do there? > > UGH! IMO this should have not been accepted without the support code. > I think this MFD driver is complete over kill to try to commonize features > into a MFD device. The LM3631 and LM3632 seem to be the only true MFD devices > here since they provide regulator support. They are not yet in mainline; but they have Rob's acknowledges. ...but I see there are improvements possible. > The LM3697 fault monitoring is only for test purposes according to the data > sheet. Not sure the customer can trust this or they should be warned at boot > that the Fault monitoring is for test purposes only. Ok, but that's topic for the driver, not for binding, right? > And I think Jacek pointed out that the bindings references in this bindings > don't even exist. > > I am thinking we need to deprecate this MFD driver and consolidate these drivers > in the LED directory as we indicated before. I did not find any ti-lmu support > code. > > ti-lmu common core code and then the LED children appending the feature differentiation. > Need some maintainer weigh in here. Hehe. I'm maintnainer. Fun. Is there something obviously wrong with 287cce719d85311f61d1b6b7f7b0d93f7907cd46 + d774c7e447ac911e73a1b3c775e6d89f0422218c ? If not, as it already had discussion/Acks so I'd prefer that one. We may move it to LEDs directory if neccessary, or something like that... Best regards, Pavel > > commit 287cce719d85311f61d1b6b7f7b0d93f7907cd46 > > Author: Milo Kim > > Date: Tue Feb 28 15:45:14 2017 +0900 > > > > dt-bindings: mfd: Add TI LMU device binding information > > > > This patch describes overall binding for TI LMU MFD devices. > > > > Signed-off-by: Milo Kim > > Acked-by: Rob Herring > > Acked-by: Tony Lindgren > > Signed-off-by: Lee Jones > > > > > > commit d774c7e447ac911e73a1b3c775e6d89f0422218c > > Author: Sebastian Reichel > > Date: Mon Aug 27 09:15:08 2018 -0700 > > > > dt-bindings: mfd: ti-lmu: update for backlight > > > > Update binding to integrate the backlight feature directly into > > the main node, as suggested by Rob Herring. > > > > Signed-off-by: Sebastian Reichel > > > > diff --git a/Documentation/devicetree/bindings/mfd/ti-lmu.txt b/Documentation/devicetree/bindings/mfd/ti-lmu.txt > > index c885cf8..b3433e9 100644 > > --- a/Documentation/devicetree/bindings/mfd/ti-lmu.txt > > +++ b/Documentation/devicetree/bindings/mfd/ti-lmu.txt > > @@ -28,10 +28,9 @@ Required properties: > > > > Optional property: > > - enable-gpios: A GPIO specifier for hardware enable pin. > > - > > -Required node: > > - - backlight: All LMU devices have backlight child nodes. > > - For the properties, please refer to [1]. > > + - pwm-names: Should be either "lmu-backlight" or unset > > + - pwm: This should be a PWM specifier following ../pwm/pwm.txt and must > > + only be specified, if the backlight should be used in PWM mode. > > > > Optional nodes: > > - fault-monitor: Hardware fault monitoring driver for LM3633 and LM3697. > > @@ -42,8 +41,31 @@ Optional nodes: > > - leds: LED properties for LM3633. Please refer to [2]. > > - regulators: Regulator properties for LM3631 and LM3632. > > Please refer to [3]. > > + - bank0, bank1, bank2: This contains the backlight configuration > > + for each backlight control bank. > > + > > +Required properties in the bank subnodes: > > + - label: A string describing the backlight. Should contain "keyboard" > > + for a keyboard backlight and "lcd" for LCD panel backlights. > > + - ti,led-sources: A list of channels, that should be driven. Each channel > > + should only be driven by one bank. > > + > > +Optional properties in the bank subnodes: > > + - default-brightness-level: Backlight initial brightness value. > > + Type is . It is set as soon as backlight > > + device is created. > > + 0 ~ 2047 = LM3631, LM3632, LM3633, LM3695 and > > + LM3697 > > + 0 ~ 255 = LM3532 > > + - ti,ramp-up-ms, ti,ramp-down-ms: Light dimming effect properties. > > + Type is . Unit is millisecond. > > + 0 ~ 65 ms = LM3532 > > + 0 ~ 4000 ms = LM3631 > > + 0 ~ 16000 ms = LM3633 and LM3697 > > + - pwm-period: PWM period. Only valid in PWM brightness mode. > > + Type is . If this property is missing, then control > > + mode is set to I2C by default. > > > > -[1] ../leds/backlight/ti-lmu-backlight.txt > > [2] ../leds/leds-lm3633.txt > > [3] ../regulator/lm363x-regulator.txt > > > > @@ -53,14 +75,11 @@ lm3532@38 { > > > > enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>; > > > > - backlight { > > - compatible = "ti,lm3532-backlight"; > > - > > - lcd { > > - led-sources = <0 1 2>; > > - ramp-up-msec = <30>; > > - ramp-down-msec = <0>; > > - }; > > + bank0 { > > + label = "lcd"; > > + led-sources = <0 1 2>; > > + ramp-up-msec = <30>; > > + ramp-down-msec = <0>; > > }; > > }; > > > > @@ -105,13 +124,10 @@ lm3631@29 { > > }; > > }; > > > > - backlight { > > - compatible = "ti,lm3631-backlight"; > > - > > - lcd_bl { > > - led-sources = <0 1>; > > - ramp-up-msec = <300>; > > - }; > > + bank0 { > > + label = "lcd_bl"; > > + led-sources = <0 1>; > > + ramp-up-msec = <300>; > > }; > > }; > > > > @@ -147,16 +163,13 @@ lm3632@11 { > > }; > > }; > > > > - backlight { > > - compatible = "ti,lm3632-backlight"; > > - > > - pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */ > > - pwm-names = "lmu-backlight"; > > + pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */ > > + pwm-names = "lmu-backlight"; > > > > - lcd { > > - led-sources = <0 1>; > > - pwm-period = <10000>; > > - }; > > + bank0 { > > + label = "lcd"; > > + led-sources = <0 1>; > > + pwm-period = <10000>; > > }; > > }; > > > > @@ -166,22 +179,18 @@ lm3633@36 { > > > > enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>; > > > > - backlight { > > - compatible = "ti,lm3633-backlight"; > > - > > - main { > > - label = "main_lcd"; > > - led-sources = <1 2>; > > - ramp-up-msec = <500>; > > - ramp-down-msec = <500>; > > - }; > > + bank0 { > > + label = "main_lcd"; > > + led-sources = <1 2>; > > + ramp-up-msec = <500>; > > + ramp-down-msec = <500>; > > + }; > > > > - front { > > - label = "front_lcd"; > > - led-sources = <0>; > > - ramp-up-msec = <1000>; > > - ramp-down-msec = <0>; > > - }; > > + bank1 { > > + label = "front_lcd"; > > + led-sources = <0>; > > + ramp-up-msec = <1000>; > > + ramp-down-msec = <0>; > > }; > > > > leds { > > @@ -211,13 +220,9 @@ lm3695@63 { > > > > enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>; > > > > - backlight { > > - compatible = "ti,lm3695-backlight"; > > - > > - lcd { > > - label = "bl"; > > - led-sources = <0 1>; > > - }; > > + bank0 { > > + label = "lcd_bl"; > > + led-sources = <0 1>; > > }; > > }; > > > > @@ -227,14 +232,10 @@ lm3697@36 { > > > > enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>; > > > > - backlight { > > - compatible = "ti,lm3697-backlight"; > > - > > - lcd { > > - led-sources = <0 1 2>; > > - ramp-up-msec = <200>; > > - ramp-down-msec = <200>; > > - }; > > + bank0 { > > + led-sources = <0 1 2>; > > + ramp-up-msec = <200>; > > + ramp-down-msec = <200>; > > }; > > > > fault-monitor { > > > > -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html