2019-12-25 11:08:14

by Guido Günther

[permalink] [raw]
Subject: [PATCH v2 1/6] dt: bindings: lm3692x: Add ti,ovp-microvolt property

This allows to set the overvoltage protection to 17V, 21V, 25V or 29V.

Signed-off-by: Guido Günther <[email protected]>
---
Documentation/devicetree/bindings/leds/leds-lm3692x.txt | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
index 4c2d923f8758..95f6d8c19f20 100644
--- a/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
+++ b/Documentation/devicetree/bindings/leds/leds-lm3692x.txt
@@ -18,6 +18,10 @@ Required properties:
Optional properties:
- enable-gpios : gpio pin to enable/disable the device.
- vled-supply : LED supply
+ - ti,ovp-microvolt: Overvoltage protection in
+ micro-volt, can be 17000000, 21000000, 25000000 or
+ 29000000. If ti,ovp-microvolt is not specified it
+ defaults to 29000000.

Required child properties:
- reg : 0 - Will enable all LED sync paths
@@ -44,6 +48,7 @@ led-controller@36 {

enable-gpios = <&gpio1 28 GPIO_ACTIVE_HIGH>;
vled-supply = <&vbatt>;
+ ti,ovp-microvolt = <25000000>;

led@0 {
reg = <0>;
--
2.23.0


2019-12-26 10:07:29

by Pavel Machek

[permalink] [raw]
Subject: Re: [PATCH v2 1/6] dt: bindings: lm3692x: Add ti,ovp-microvolt property

On Wed 2019-12-25 12:07:14, Guido G?nther wrote:
> This allows to set the overvoltage protection to 17V, 21V, 25V or 29V.
>
> Signed-off-by: Guido G?nther <[email protected]>

Acked-by: Pavel Machek <[email protected]>

I will need Rob's Ack.

> @@ -44,6 +48,7 @@ led-controller@36 {
>
> enable-gpios = <&gpio1 28 GPIO_ACTIVE_HIGH>;
> vled-supply = <&vbatt>;
> + ti,ovp-microvolt = <25000000>;

Wed often put the default values here.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachments:
(No filename) (606.00 B)
 signature.asc (188.00 B)
Digital signature Download all attachments

2019-12-27 12:20:52

by Guido Günther

[permalink] [raw]
Subject: Re: [PATCH v2 1/6] dt: bindings: lm3692x: Add ti,ovp-microvolt property

Hi,
On Thu, Dec 26, 2019 at 11:06:15AM +0100, Pavel Machek wrote:
> On Wed 2019-12-25 12:07:14, Guido G?nther wrote:
> > This allows to set the overvoltage protection to 17V, 21V, 25V or 29V.
> >
> > Signed-off-by: Guido G?nther <[email protected]>
>
> Acked-by: Pavel Machek <[email protected]>

Thanks!

> I will need Rob's Ack.
>
> > @@ -44,6 +48,7 @@ led-controller@36 {
> >
> > enable-gpios = <&gpio1 28 GPIO_ACTIVE_HIGH>;
> > vled-supply = <&vbatt>;
> > + ti,ovp-microvolt = <25000000>;
>
> Wed often put the default values here.

Makes sense, thanks.
-- Guido

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