Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1260394 > unrolled thread

[PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information

Started byMilo Kim <milo.kim@ti.com>
First post2015-11-02 06:30 +0100
Last post2015-11-12 01:10 +0100
Articles 4 — 4 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information Milo Kim <milo.kim@ti.com> - 2015-11-02 06:30 +0100
    Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU  device binding information Rob Herring <robh@kernel.org> - 2015-11-06 03:10 +0100
    Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU  device binding information Lee Jones <lee.jones@linaro.org> - 2015-11-11 10:50 +0100
      Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU  device binding information "Kim, Milo" <milo.kim@ti.com> - 2015-11-12 01:10 +0100

#1260394 — [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information

FromMilo Kim <milo.kim@ti.com>
Date2015-11-02 06:30 +0100
Subject[PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information
Message-ID<qqfQS-1lO-29@gated-at.bofh.it>
This patch describes overall binding for TI LMU MFD devices.

Cc: devicetree@vger.kernel.org
Cc: Lee Jones <lee.jones@linaro.org>
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Milo Kim <milo.kim@ti.com>
---
 Documentation/devicetree/bindings/mfd/ti-lmu.txt | 282 +++++++++++++++++++++++
 1 file changed, 282 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/ti-lmu.txt

diff --git a/Documentation/devicetree/bindings/mfd/ti-lmu.txt b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
new file mode 100644
index 0000000..7ccf07e
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
@@ -0,0 +1,282 @@
+TI LMU(Lighting Management Unit) device tree bindings
+
+TI LMU driver supports lighting devices belows.
+
+   Name              Child nodes
+  ------      -------------------------
+  LM3532       Backlight
+  LM3631       Backlight and regulator
+  LM3632       Backlight and regulator
+  LM3633       Backlight, LED and HWMON
+  LM3695       Backlight
+  LM3697       Backlight and HWMON
+
+Required properties:
+  - compatible: Should be one of lists below.
+                "ti,lm3532"
+                "ti,lm3631"
+                "ti,lm3632"
+                "ti,lm3633"
+                "ti,lm3695"
+                "ti,lm3697"
+  - reg: I2C slave address.
+         0x11 is LM3632
+         0x29 is LM3631
+         0x36 is LM3633, LM3697
+         0x38 is LM3532
+         0x63 is LM3695
+
+Optional properties:
+  - 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].
+
+Optional nodes:
+  - hwmon: Hardware fault monitoring driver for LM3633 and LM3697.
+           For the property, please refer to [2].
+  - leds: LED properties for LM3633. Please refer to [3].
+  - regulators: Regulator properties for LM3631 and LM3632.
+          Please refer to [4].
+
+[1] Documentation/devicetree/bindings/video/backlight/ti-lmu-backlight.txt
+[2] Documentation/devicetree/bindings/hwm/ti-lmu-hwmon.txt
+[3] Documentation/devicetree/bindings/leds/leds-lm3633.txt
+[4] Documentation/devicetree/bindings/regulator/lm363x-regulator.txt
+
+Examples:
+
+LM3532 has a backlight device. External GPIO is used for enabling LM3532.
+
+lm3532@38 {
+	compatible = "ti,lm3532";
+	reg = <0x38>;
+
+	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
+
+	backlight {
+		compatible = "ti,lm3532-backlight";
+
+		lcd {
+			hvled1-used;
+			hvled2-used;
+			hvled3-used;
+
+			ramp-up-msec = <30>;
+			ramp-down-msec = <0>;
+
+			backlight-max-microamp = <5000>;
+		};
+	};
+};
+
+LM3631 has 5 regulators with one backlight device.
+
+lm3631@29 {
+	compatible = "ti,lm3631";
+	reg = <0x29>;
+
+	regulators {
+		compatible = "ti,lm363x-regulator";
+
+		vboost {
+			regulator-name = "lcd_boost";
+			regulator-min-microvolt = <4500000>;
+			regulator-max-microvolt = <6350000>;
+			regulator-always-on;
+		};
+
+		vcont {
+			regulator-name = "lcd_vcont";
+			regulator-min-microvolt = <1800000>;
+			regulator-max-microvolt = <3300000>;
+		};
+
+		voref {
+			regulator-name = "lcd_voref";
+			regulator-min-microvolt = <4000000>;
+			regulator-max-microvolt = <6000000>;
+		};
+
+		vpos {
+			regulator-name = "lcd_vpos";
+			regulator-min-microvolt = <4000000>;
+			regulator-max-microvolt = <6000000>;
+			regulator-boot-on;
+		};
+
+		vneg {
+			regulator-name = "lcd_vneg";
+			regulator-min-microvolt = <4000000>;
+			regulator-max-microvolt = <6000000>;
+			regulator-boot-on;
+		};
+	};
+
+	backlight {
+		compatible = "ti,lm3631-backlight";
+
+		lcd {
+			backlight-name = "lcd_bl";
+			hvled1-used;
+			hvled2-used;
+
+			ramp-up-msec = <300>;
+		};
+	};
+};
+
+LM3632 has 3 regulators with one backlight device. External GPIO is
+used for enabling LM3632.
+
+lm3632@11 {
+	compatible = "ti,lm3632";
+	reg = <0x11>;
+
+	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
+
+	regulators {
+		compatible = "ti,lm363x-regulator";
+
+		ti,lcm-en1-gpio = <&pioC 0 GPIO_ACTIVE_HIGH>;
+		ti,lcm-en2-gpio = <&pioC 1 GPIO_ACTIVE_HIGH>;
+
+		vboost {
+			regulator-name = "lcd_boost";
+			regulator-min-microvolt = <4500000>;
+			regulator-max-microvolt = <6400000>;
+			regulator-always-on;
+		};
+
+		vpos {
+			regulator-name = "lcd_vpos";
+			regulator-min-microvolt = <4000000>;
+			regulator-max-microvolt = <6000000>;
+		};
+
+		vneg {
+			regulator-name = "lcd_vneg";
+			regulator-min-microvolt = <4000000>;
+			regulator-max-microvolt = <6000000>;
+		};
+	};
+
+	backlight {
+		compatible = "ti,lm3632-backlight";
+
+		lcd {
+			backlight-name = "lcd";
+			hvled1-used;
+			hvled2-used;
+		};
+	};
+};
+
+LM3633 has multiple backlight channels, LED channels and hardware fault
+monitoring driver. External GPIO is used for enabling LM3633.
+
+lm3633@36 {
+	compatible = "ti,lm3633";
+	reg = <0x36>;
+
+	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
+
+	backlight {
+		compatible = "ti,lm3633-backlight";
+
+		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
+		pwm-names = "lmu-backlight";
+
+		main {
+			backlight-name = "main_lcd";
+			hvled2-used;
+			hvled3-used;
+			backlight-max-microamp = <20000>;
+
+			ramp-up-msec = <500>;
+			ramp-down-msec = <500>;
+		};
+
+		front {
+			backlight-name = "front_lcd";
+			hvled1-used;
+
+			ramp-up-msec = <1000>;
+			ramp-down-msec = <0>;
+
+			pwm-period = <10000>;
+		};
+	};
+
+	leds {
+		compatible = "ti,lm3633-leds";
+
+		chan2 {
+			channel-name = "status";
+			lvled2-used;
+			led-max-microamp = <6000>;
+		};
+
+		chan456 {
+			channel-name = "rgb";
+			lvled4-used;
+			lvled5-used;
+			lvled6-used;
+		};
+	};
+
+	hwmon {
+		compatible = "ti,lm3633-hwmon";
+	};
+};
+
+LM3695 is single backlight device.
+
+lm3695@63 {
+	compatible = "ti,lm3695";
+	reg = <0x63>;
+
+	backlight {
+		compatible = "ti,lm3695-backlight";
+
+		lcd {
+			hvled1-used;
+			hvled2-used;
+			backlight-max-microamp = <20000>;
+		};
+	};
+};
+
+LM3697 has one backlight device and hardware fault monitoring driver.
+External GPIO is used for enabling LM3697.
+
+lm3697@36 {
+	compatible = "ti,lm3697";
+	reg = <0x36>;
+
+	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
+
+	backlight {
+		compatible = "ti,lm3697-backlight";
+
+		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
+		pwm-names = "lmu-backlight";
+
+		lcd {
+			backlight-name = "bl";
+			hvled1-used;
+			hvled2-used;
+			hvled3-used;
+			backlight-max-microamp = <20000>;
+
+			ramp-up-msec = <500>;
+			ramp-down-msec = <500>;
+			pwm-period = <10000>;
+		};
+	};
+
+	hwmon {
+		compatible = "ti,lm3697-hwmon";
+	};
+};
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1263702 — Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information

FromRob Herring <robh@kernel.org>
Date2015-11-06 03:10 +0100
SubjectRe: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information
Message-ID<qrEDw-6Ht-3@gated-at.bofh.it>
In reply to#1260394
On Mon, Nov 02, 2015 at 02:24:20PM +0900, Milo Kim wrote:
> This patch describes overall binding for TI LMU MFD devices.
> 
> Cc: devicetree@vger.kernel.org
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Milo Kim <milo.kim@ti.com>

Acked-by: Rob Herring <robh@kernel.org>

> ---
>  Documentation/devicetree/bindings/mfd/ti-lmu.txt | 282 +++++++++++++++++++++++
>  1 file changed, 282 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/ti-lmu.txt
> 
> diff --git a/Documentation/devicetree/bindings/mfd/ti-lmu.txt b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
> new file mode 100644
> index 0000000..7ccf07e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
> @@ -0,0 +1,282 @@
> +TI LMU(Lighting Management Unit) device tree bindings
> +
> +TI LMU driver supports lighting devices belows.
> +
> +   Name              Child nodes
> +  ------      -------------------------
> +  LM3532       Backlight
> +  LM3631       Backlight and regulator
> +  LM3632       Backlight and regulator
> +  LM3633       Backlight, LED and HWMON
> +  LM3695       Backlight
> +  LM3697       Backlight and HWMON
> +
> +Required properties:
> +  - compatible: Should be one of lists below.
> +                "ti,lm3532"
> +                "ti,lm3631"
> +                "ti,lm3632"
> +                "ti,lm3633"
> +                "ti,lm3695"
> +                "ti,lm3697"
> +  - reg: I2C slave address.
> +         0x11 is LM3632
> +         0x29 is LM3631
> +         0x36 is LM3633, LM3697
> +         0x38 is LM3532
> +         0x63 is LM3695
> +
> +Optional properties:
> +  - 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].
> +
> +Optional nodes:
> +  - hwmon: Hardware fault monitoring driver for LM3633 and LM3697.
> +           For the property, please refer to [2].
> +  - leds: LED properties for LM3633. Please refer to [3].
> +  - regulators: Regulator properties for LM3631 and LM3632.
> +          Please refer to [4].
> +
> +[1] Documentation/devicetree/bindings/video/backlight/ti-lmu-backlight.txt
> +[2] Documentation/devicetree/bindings/hwm/ti-lmu-hwmon.txt
> +[3] Documentation/devicetree/bindings/leds/leds-lm3633.txt
> +[4] Documentation/devicetree/bindings/regulator/lm363x-regulator.txt
> +
> +Examples:
> +
> +LM3532 has a backlight device. External GPIO is used for enabling LM3532.
> +
> +lm3532@38 {
> +	compatible = "ti,lm3532";
> +	reg = <0x38>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3532-backlight";
> +
> +		lcd {
> +			hvled1-used;
> +			hvled2-used;
> +			hvled3-used;
> +
> +			ramp-up-msec = <30>;
> +			ramp-down-msec = <0>;
> +
> +			backlight-max-microamp = <5000>;
> +		};
> +	};
> +};
> +
> +LM3631 has 5 regulators with one backlight device.
> +
> +lm3631@29 {
> +	compatible = "ti,lm3631";
> +	reg = <0x29>;
> +
> +	regulators {
> +		compatible = "ti,lm363x-regulator";
> +
> +		vboost {
> +			regulator-name = "lcd_boost";
> +			regulator-min-microvolt = <4500000>;
> +			regulator-max-microvolt = <6350000>;
> +			regulator-always-on;
> +		};
> +
> +		vcont {
> +			regulator-name = "lcd_vcont";
> +			regulator-min-microvolt = <1800000>;
> +			regulator-max-microvolt = <3300000>;
> +		};
> +
> +		voref {
> +			regulator-name = "lcd_voref";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +
> +		vpos {
> +			regulator-name = "lcd_vpos";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +			regulator-boot-on;
> +		};
> +
> +		vneg {
> +			regulator-name = "lcd_vneg";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +			regulator-boot-on;
> +		};
> +	};
> +
> +	backlight {
> +		compatible = "ti,lm3631-backlight";
> +
> +		lcd {
> +			backlight-name = "lcd_bl";
> +			hvled1-used;
> +			hvled2-used;
> +
> +			ramp-up-msec = <300>;
> +		};
> +	};
> +};
> +
> +LM3632 has 3 regulators with one backlight device. External GPIO is
> +used for enabling LM3632.
> +
> +lm3632@11 {
> +	compatible = "ti,lm3632";
> +	reg = <0x11>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	regulators {
> +		compatible = "ti,lm363x-regulator";
> +
> +		ti,lcm-en1-gpio = <&pioC 0 GPIO_ACTIVE_HIGH>;
> +		ti,lcm-en2-gpio = <&pioC 1 GPIO_ACTIVE_HIGH>;
> +
> +		vboost {
> +			regulator-name = "lcd_boost";
> +			regulator-min-microvolt = <4500000>;
> +			regulator-max-microvolt = <6400000>;
> +			regulator-always-on;
> +		};
> +
> +		vpos {
> +			regulator-name = "lcd_vpos";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +
> +		vneg {
> +			regulator-name = "lcd_vneg";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +	};
> +
> +	backlight {
> +		compatible = "ti,lm3632-backlight";
> +
> +		lcd {
> +			backlight-name = "lcd";
> +			hvled1-used;
> +			hvled2-used;
> +		};
> +	};
> +};
> +
> +LM3633 has multiple backlight channels, LED channels and hardware fault
> +monitoring driver. External GPIO is used for enabling LM3633.
> +
> +lm3633@36 {
> +	compatible = "ti,lm3633";
> +	reg = <0x36>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3633-backlight";
> +
> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
> +		pwm-names = "lmu-backlight";
> +
> +		main {
> +			backlight-name = "main_lcd";
> +			hvled2-used;
> +			hvled3-used;
> +			backlight-max-microamp = <20000>;
> +
> +			ramp-up-msec = <500>;
> +			ramp-down-msec = <500>;
> +		};
> +
> +		front {
> +			backlight-name = "front_lcd";
> +			hvled1-used;
> +
> +			ramp-up-msec = <1000>;
> +			ramp-down-msec = <0>;
> +
> +			pwm-period = <10000>;
> +		};
> +	};
> +
> +	leds {
> +		compatible = "ti,lm3633-leds";
> +
> +		chan2 {
> +			channel-name = "status";
> +			lvled2-used;
> +			led-max-microamp = <6000>;
> +		};
> +
> +		chan456 {
> +			channel-name = "rgb";
> +			lvled4-used;
> +			lvled5-used;
> +			lvled6-used;
> +		};
> +	};
> +
> +	hwmon {
> +		compatible = "ti,lm3633-hwmon";
> +	};
> +};
> +
> +LM3695 is single backlight device.
> +
> +lm3695@63 {
> +	compatible = "ti,lm3695";
> +	reg = <0x63>;
> +
> +	backlight {
> +		compatible = "ti,lm3695-backlight";
> +
> +		lcd {
> +			hvled1-used;
> +			hvled2-used;
> +			backlight-max-microamp = <20000>;
> +		};
> +	};
> +};
> +
> +LM3697 has one backlight device and hardware fault monitoring driver.
> +External GPIO is used for enabling LM3697.
> +
> +lm3697@36 {
> +	compatible = "ti,lm3697";
> +	reg = <0x36>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3697-backlight";
> +
> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
> +		pwm-names = "lmu-backlight";
> +
> +		lcd {
> +			backlight-name = "bl";
> +			hvled1-used;
> +			hvled2-used;
> +			hvled3-used;
> +			backlight-max-microamp = <20000>;
> +
> +			ramp-up-msec = <500>;
> +			ramp-down-msec = <500>;
> +			pwm-period = <10000>;
> +		};
> +	};
> +
> +	hwmon {
> +		compatible = "ti,lm3697-hwmon";
> +	};
> +};
> -- 
> 1.9.1
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1267068 — Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information

FromLee Jones <lee.jones@linaro.org>
Date2015-11-11 10:50 +0100
SubjectRe: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information
Message-ID<qtAcq-11f-15@gated-at.bofh.it>
In reply to#1260394
On Mon, 02 Nov 2015, Milo Kim wrote:

> This patch describes overall binding for TI LMU MFD devices.
> 
> Cc: devicetree@vger.kernel.org
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Milo Kim <milo.kim@ti.com>
> ---
>  Documentation/devicetree/bindings/mfd/ti-lmu.txt | 282 +++++++++++++++++++++++
>  1 file changed, 282 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/ti-lmu.txt
> 
> diff --git a/Documentation/devicetree/bindings/mfd/ti-lmu.txt b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
> new file mode 100644
> index 0000000..7ccf07e
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
> @@ -0,0 +1,282 @@
> +TI LMU(Lighting Management Unit) device tree bindings

' ' here -^

> +TI LMU driver supports lighting devices belows.

s/belows./below:/

> +   Name              Child nodes
> +  ------      -------------------------
> +  LM3532       Backlight
> +  LM3631       Backlight and regulator
> +  LM3632       Backlight and regulator
> +  LM3633       Backlight, LED and HWMON
> +  LM3695       Backlight
> +  LM3697       Backlight and HWMON
> +
> +Required properties:
> +  - compatible: Should be one of lists below.

s/ lists below./:/

> +                "ti,lm3532"
> +                "ti,lm3631"
> +                "ti,lm3632"
> +                "ti,lm3633"
> +                "ti,lm3695"
> +                "ti,lm3697"
> +  - reg: I2C slave address.
> +         0x11 is LM3632
> +         0x29 is LM3631
> +         0x36 is LM3633, LM3697
> +         0x38 is LM3532
> +         0x63 is LM3695

s/is/for/

> +Optional properties:
> +  - 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].
> +
> +Optional nodes:
> +  - hwmon: Hardware fault monitoring driver for LM3633 and LM3697.
> +           For the property, please refer to [2].
> +  - leds: LED properties for LM3633. Please refer to [3].
> +  - regulators: Regulator properties for LM3631 and LM3632.
> +          Please refer to [4].
> +
> +[1] Documentation/devicetree/bindings/video/backlight/ti-lmu-backlight.txt
> +[2] Documentation/devicetree/bindings/hwm/ti-lmu-hwmon.txt
> +[3] Documentation/devicetree/bindings/leds/leds-lm3633.txt
> +[4] Documentation/devicetree/bindings/regulator/lm363x-regulator.txt

s/Documentation/devicetree/bindings/../

> +Examples:

These aren't examples, they're actual DT entries.

Please trim them down to just one, fully enabled node, only to be used
as an 'example'.

> +LM3532 has a backlight device. External GPIO is used for enabling LM3532.
> +
> +lm3532@38 {
> +	compatible = "ti,lm3532";
> +	reg = <0x38>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3532-backlight";
> +
> +		lcd {
> +			hvled1-used;
> +			hvled2-used;
> +			hvled3-used;
> +
> +			ramp-up-msec = <30>;
> +			ramp-down-msec = <0>;
> +
> +			backlight-max-microamp = <5000>;
> +		};
> +	};
> +};
> +
> +LM3631 has 5 regulators with one backlight device.
> +
> +lm3631@29 {
> +	compatible = "ti,lm3631";
> +	reg = <0x29>;
> +
> +	regulators {
> +		compatible = "ti,lm363x-regulator";
> +
> +		vboost {
> +			regulator-name = "lcd_boost";
> +			regulator-min-microvolt = <4500000>;
> +			regulator-max-microvolt = <6350000>;
> +			regulator-always-on;
> +		};
> +
> +		vcont {
> +			regulator-name = "lcd_vcont";
> +			regulator-min-microvolt = <1800000>;
> +			regulator-max-microvolt = <3300000>;
> +		};
> +
> +		voref {
> +			regulator-name = "lcd_voref";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +
> +		vpos {
> +			regulator-name = "lcd_vpos";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +			regulator-boot-on;
> +		};
> +
> +		vneg {
> +			regulator-name = "lcd_vneg";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +			regulator-boot-on;
> +		};
> +	};
> +
> +	backlight {
> +		compatible = "ti,lm3631-backlight";
> +
> +		lcd {
> +			backlight-name = "lcd_bl";
> +			hvled1-used;
> +			hvled2-used;
> +
> +			ramp-up-msec = <300>;
> +		};
> +	};
> +};
> +
> +LM3632 has 3 regulators with one backlight device. External GPIO is
> +used for enabling LM3632.
> +
> +lm3632@11 {
> +	compatible = "ti,lm3632";
> +	reg = <0x11>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	regulators {
> +		compatible = "ti,lm363x-regulator";
> +
> +		ti,lcm-en1-gpio = <&pioC 0 GPIO_ACTIVE_HIGH>;
> +		ti,lcm-en2-gpio = <&pioC 1 GPIO_ACTIVE_HIGH>;
> +
> +		vboost {
> +			regulator-name = "lcd_boost";
> +			regulator-min-microvolt = <4500000>;
> +			regulator-max-microvolt = <6400000>;
> +			regulator-always-on;
> +		};
> +
> +		vpos {
> +			regulator-name = "lcd_vpos";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +
> +		vneg {
> +			regulator-name = "lcd_vneg";
> +			regulator-min-microvolt = <4000000>;
> +			regulator-max-microvolt = <6000000>;
> +		};
> +	};
> +
> +	backlight {
> +		compatible = "ti,lm3632-backlight";
> +
> +		lcd {
> +			backlight-name = "lcd";
> +			hvled1-used;
> +			hvled2-used;
> +		};
> +	};
> +};
> +
> +LM3633 has multiple backlight channels, LED channels and hardware fault
> +monitoring driver. External GPIO is used for enabling LM3633.
> +
> +lm3633@36 {
> +	compatible = "ti,lm3633";
> +	reg = <0x36>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3633-backlight";
> +
> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
> +		pwm-names = "lmu-backlight";
> +
> +		main {
> +			backlight-name = "main_lcd";
> +			hvled2-used;
> +			hvled3-used;
> +			backlight-max-microamp = <20000>;
> +
> +			ramp-up-msec = <500>;
> +			ramp-down-msec = <500>;
> +		};
> +
> +		front {
> +			backlight-name = "front_lcd";
> +			hvled1-used;
> +
> +			ramp-up-msec = <1000>;
> +			ramp-down-msec = <0>;
> +
> +			pwm-period = <10000>;
> +		};
> +	};
> +
> +	leds {
> +		compatible = "ti,lm3633-leds";
> +
> +		chan2 {
> +			channel-name = "status";
> +			lvled2-used;
> +			led-max-microamp = <6000>;
> +		};
> +
> +		chan456 {
> +			channel-name = "rgb";
> +			lvled4-used;
> +			lvled5-used;
> +			lvled6-used;
> +		};
> +	};
> +
> +	hwmon {
> +		compatible = "ti,lm3633-hwmon";
> +	};
> +};
> +
> +LM3695 is single backlight device.
> +
> +lm3695@63 {
> +	compatible = "ti,lm3695";
> +	reg = <0x63>;
> +
> +	backlight {
> +		compatible = "ti,lm3695-backlight";
> +
> +		lcd {
> +			hvled1-used;
> +			hvled2-used;
> +			backlight-max-microamp = <20000>;
> +		};
> +	};
> +};
> +
> +LM3697 has one backlight device and hardware fault monitoring driver.
> +External GPIO is used for enabling LM3697.
> +
> +lm3697@36 {
> +	compatible = "ti,lm3697";
> +	reg = <0x36>;
> +
> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
> +
> +	backlight {
> +		compatible = "ti,lm3697-backlight";
> +
> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
> +		pwm-names = "lmu-backlight";
> +
> +		lcd {
> +			backlight-name = "bl";
> +			hvled1-used;
> +			hvled2-used;
> +			hvled3-used;
> +			backlight-max-microamp = <20000>;
> +
> +			ramp-up-msec = <500>;
> +			ramp-down-msec = <500>;
> +			pwm-period = <10000>;
> +		};
> +	};
> +
> +	hwmon {
> +		compatible = "ti,lm3697-hwmon";
> +	};
> +};

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1267547 — Re: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information

From"Kim, Milo" <milo.kim@ti.com>
Date2015-11-12 01:10 +0100
SubjectRe: [PATCH RESEND 01/16] Documentation: dt-bindings: mfd: add TI LMU device binding information
Message-ID<qtNCF-1vu-9@gated-at.bofh.it>
In reply to#1267068
Thanks for all your comments. I'll create the 2nd patch-set in few weeks.

Best regards,
Milo

On 11/11/2015 6:49 PM, Lee Jones wrote:
> On Mon, 02 Nov 2015, Milo Kim wrote:
>
>> This patch describes overall binding for TI LMU MFD devices.
>>
>> Cc: devicetree@vger.kernel.org
>> Cc: Lee Jones <lee.jones@linaro.org>
>> Cc: linux-kernel@vger.kernel.org
>> Signed-off-by: Milo Kim <milo.kim@ti.com>
>> ---
>>   Documentation/devicetree/bindings/mfd/ti-lmu.txt | 282 +++++++++++++++++++++++
>>   1 file changed, 282 insertions(+)
>>   create mode 100644 Documentation/devicetree/bindings/mfd/ti-lmu.txt
>>
>> diff --git a/Documentation/devicetree/bindings/mfd/ti-lmu.txt b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
>> new file mode 100644
>> index 0000000..7ccf07e
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/mfd/ti-lmu.txt
>> @@ -0,0 +1,282 @@
>> +TI LMU(Lighting Management Unit) device tree bindings
>
> ' ' here -^
>
>> +TI LMU driver supports lighting devices belows.
>
> s/belows./below:/
>
>> +   Name              Child nodes
>> +  ------      -------------------------
>> +  LM3532       Backlight
>> +  LM3631       Backlight and regulator
>> +  LM3632       Backlight and regulator
>> +  LM3633       Backlight, LED and HWMON
>> +  LM3695       Backlight
>> +  LM3697       Backlight and HWMON
>> +
>> +Required properties:
>> +  - compatible: Should be one of lists below.
>
> s/ lists below./:/
>
>> +                "ti,lm3532"
>> +                "ti,lm3631"
>> +                "ti,lm3632"
>> +                "ti,lm3633"
>> +                "ti,lm3695"
>> +                "ti,lm3697"
>> +  - reg: I2C slave address.
>> +         0x11 is LM3632
>> +         0x29 is LM3631
>> +         0x36 is LM3633, LM3697
>> +         0x38 is LM3532
>> +         0x63 is LM3695
>
> s/is/for/
>
>> +Optional properties:
>> +  - 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].
>> +
>> +Optional nodes:
>> +  - hwmon: Hardware fault monitoring driver for LM3633 and LM3697.
>> +           For the property, please refer to [2].
>> +  - leds: LED properties for LM3633. Please refer to [3].
>> +  - regulators: Regulator properties for LM3631 and LM3632.
>> +          Please refer to [4].
>> +
>> +[1] Documentation/devicetree/bindings/video/backlight/ti-lmu-backlight.txt
>> +[2] Documentation/devicetree/bindings/hwm/ti-lmu-hwmon.txt
>> +[3] Documentation/devicetree/bindings/leds/leds-lm3633.txt
>> +[4] Documentation/devicetree/bindings/regulator/lm363x-regulator.txt
>
> s/Documentation/devicetree/bindings/../
>
>> +Examples:
>
> These aren't examples, they're actual DT entries.
>
> Please trim them down to just one, fully enabled node, only to be used
> as an 'example'.
>
>> +LM3532 has a backlight device. External GPIO is used for enabling LM3532.
>> +
>> +lm3532@38 {
>> +	compatible = "ti,lm3532";
>> +	reg = <0x38>;
>> +
>> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
>> +
>> +	backlight {
>> +		compatible = "ti,lm3532-backlight";
>> +
>> +		lcd {
>> +			hvled1-used;
>> +			hvled2-used;
>> +			hvled3-used;
>> +
>> +			ramp-up-msec = <30>;
>> +			ramp-down-msec = <0>;
>> +
>> +			backlight-max-microamp = <5000>;
>> +		};
>> +	};
>> +};
>> +
>> +LM3631 has 5 regulators with one backlight device.
>> +
>> +lm3631@29 {
>> +	compatible = "ti,lm3631";
>> +	reg = <0x29>;
>> +
>> +	regulators {
>> +		compatible = "ti,lm363x-regulator";
>> +
>> +		vboost {
>> +			regulator-name = "lcd_boost";
>> +			regulator-min-microvolt = <4500000>;
>> +			regulator-max-microvolt = <6350000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vcont {
>> +			regulator-name = "lcd_vcont";
>> +			regulator-min-microvolt = <1800000>;
>> +			regulator-max-microvolt = <3300000>;
>> +		};
>> +
>> +		voref {
>> +			regulator-name = "lcd_voref";
>> +			regulator-min-microvolt = <4000000>;
>> +			regulator-max-microvolt = <6000000>;
>> +		};
>> +
>> +		vpos {
>> +			regulator-name = "lcd_vpos";
>> +			regulator-min-microvolt = <4000000>;
>> +			regulator-max-microvolt = <6000000>;
>> +			regulator-boot-on;
>> +		};
>> +
>> +		vneg {
>> +			regulator-name = "lcd_vneg";
>> +			regulator-min-microvolt = <4000000>;
>> +			regulator-max-microvolt = <6000000>;
>> +			regulator-boot-on;
>> +		};
>> +	};
>> +
>> +	backlight {
>> +		compatible = "ti,lm3631-backlight";
>> +
>> +		lcd {
>> +			backlight-name = "lcd_bl";
>> +			hvled1-used;
>> +			hvled2-used;
>> +
>> +			ramp-up-msec = <300>;
>> +		};
>> +	};
>> +};
>> +
>> +LM3632 has 3 regulators with one backlight device. External GPIO is
>> +used for enabling LM3632.
>> +
>> +lm3632@11 {
>> +	compatible = "ti,lm3632";
>> +	reg = <0x11>;
>> +
>> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
>> +
>> +	regulators {
>> +		compatible = "ti,lm363x-regulator";
>> +
>> +		ti,lcm-en1-gpio = <&pioC 0 GPIO_ACTIVE_HIGH>;
>> +		ti,lcm-en2-gpio = <&pioC 1 GPIO_ACTIVE_HIGH>;
>> +
>> +		vboost {
>> +			regulator-name = "lcd_boost";
>> +			regulator-min-microvolt = <4500000>;
>> +			regulator-max-microvolt = <6400000>;
>> +			regulator-always-on;
>> +		};
>> +
>> +		vpos {
>> +			regulator-name = "lcd_vpos";
>> +			regulator-min-microvolt = <4000000>;
>> +			regulator-max-microvolt = <6000000>;
>> +		};
>> +
>> +		vneg {
>> +			regulator-name = "lcd_vneg";
>> +			regulator-min-microvolt = <4000000>;
>> +			regulator-max-microvolt = <6000000>;
>> +		};
>> +	};
>> +
>> +	backlight {
>> +		compatible = "ti,lm3632-backlight";
>> +
>> +		lcd {
>> +			backlight-name = "lcd";
>> +			hvled1-used;
>> +			hvled2-used;
>> +		};
>> +	};
>> +};
>> +
>> +LM3633 has multiple backlight channels, LED channels and hardware fault
>> +monitoring driver. External GPIO is used for enabling LM3633.
>> +
>> +lm3633@36 {
>> +	compatible = "ti,lm3633";
>> +	reg = <0x36>;
>> +
>> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
>> +
>> +	backlight {
>> +		compatible = "ti,lm3633-backlight";
>> +
>> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
>> +		pwm-names = "lmu-backlight";
>> +
>> +		main {
>> +			backlight-name = "main_lcd";
>> +			hvled2-used;
>> +			hvled3-used;
>> +			backlight-max-microamp = <20000>;
>> +
>> +			ramp-up-msec = <500>;
>> +			ramp-down-msec = <500>;
>> +		};
>> +
>> +		front {
>> +			backlight-name = "front_lcd";
>> +			hvled1-used;
>> +
>> +			ramp-up-msec = <1000>;
>> +			ramp-down-msec = <0>;
>> +
>> +			pwm-period = <10000>;
>> +		};
>> +	};
>> +
>> +	leds {
>> +		compatible = "ti,lm3633-leds";
>> +
>> +		chan2 {
>> +			channel-name = "status";
>> +			lvled2-used;
>> +			led-max-microamp = <6000>;
>> +		};
>> +
>> +		chan456 {
>> +			channel-name = "rgb";
>> +			lvled4-used;
>> +			lvled5-used;
>> +			lvled6-used;
>> +		};
>> +	};
>> +
>> +	hwmon {
>> +		compatible = "ti,lm3633-hwmon";
>> +	};
>> +};
>> +
>> +LM3695 is single backlight device.
>> +
>> +lm3695@63 {
>> +	compatible = "ti,lm3695";
>> +	reg = <0x63>;
>> +
>> +	backlight {
>> +		compatible = "ti,lm3695-backlight";
>> +
>> +		lcd {
>> +			hvled1-used;
>> +			hvled2-used;
>> +			backlight-max-microamp = <20000>;
>> +		};
>> +	};
>> +};
>> +
>> +LM3697 has one backlight device and hardware fault monitoring driver.
>> +External GPIO is used for enabling LM3697.
>> +
>> +lm3697@36 {
>> +	compatible = "ti,lm3697";
>> +	reg = <0x36>;
>> +
>> +	enable-gpios = <&pioC 2 GPIO_ACTIVE_HIGH>;
>> +
>> +	backlight {
>> +		compatible = "ti,lm3697-backlight";
>> +
>> +		pwms = <&pwm0 0 10000 0>; /* pwm number, period, polarity */
>> +		pwm-names = "lmu-backlight";
>> +
>> +		lcd {
>> +			backlight-name = "bl";
>> +			hvled1-used;
>> +			hvled2-used;
>> +			hvled3-used;
>> +			backlight-max-microamp = <20000>;
>> +
>> +			ramp-up-msec = <500>;
>> +			ramp-down-msec = <500>;
>> +			pwm-period = <10000>;
>> +		};
>> +	};
>> +
>> +	hwmon {
>> +		compatible = "ti,lm3697-hwmon";
>> +	};
>> +};
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web