[PATCH/RFC,v13,04/13] DT: Add documentation for the mfd Maxim max77693

Message ID 1426175114-14876-5-git-send-email-j.anaszewski@samsung.com (mailing list archive)
State Superseded, archived
Headers

Commit Message

Jacek Anaszewski March 12, 2015, 3:45 p.m. UTC
  This patch adds device tree binding documentation for
the flash cell of the Maxim max77693 multifunctional device.

Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Lee Jones <lee.jones@linaro.org>
Cc: Chanwoo Choi <cw00.choi@samsung.com>
Cc: Bryan Wu <cooloney@gmail.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
---
 Documentation/devicetree/bindings/mfd/max77693.txt |   61 ++++++++++++++++++++
 1 file changed, 61 insertions(+)
  

Comments

Lee Jones March 23, 2015, 12:07 p.m. UTC | #1
This patch requires a DT Ack.

> This patch adds device tree binding documentation for
> the flash cell of the Maxim max77693 multifunctional device.
> 
> Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> Cc: Lee Jones <lee.jones@linaro.org>
> Cc: Chanwoo Choi <cw00.choi@samsung.com>
> Cc: Bryan Wu <cooloney@gmail.com>
> Cc: Richard Purdie <rpurdie@rpsys.net>
> ---
>  Documentation/devicetree/bindings/mfd/max77693.txt |   61 ++++++++++++++++++++
>  1 file changed, 61 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/max77693.txt b/Documentation/devicetree/bindings/mfd/max77693.txt
> index 38e6440..15c546e 100644
> --- a/Documentation/devicetree/bindings/mfd/max77693.txt
> +++ b/Documentation/devicetree/bindings/mfd/max77693.txt
> @@ -76,7 +76,53 @@ Optional properties:
>      Valid values: 4300000, 4700000, 4800000, 4900000
>      Default: 4300000
>  
> +- led : the LED submodule device node
> +
> +There are two LED outputs available - FLED1 and FLED2. Each of them can
> +control a separate LED or they can be connected together to double
> +the maximum current for a single connected LED. One LED is represented
> +by one child node.
> +
> +Required properties:
> +- compatible : Must be "maxim,max77693-led".
> +
> +Optional properties:
> +- maxim,trigger-type : Flash trigger type.
> +	Possible trigger types:
> +		LEDS_TRIG_TYPE_EDGE (0) - Rising edge of the signal triggers
> +			the flash,
> +		LEDS_TRIG_TYPE_LEVEL (1) - Strobe pulse length controls duration
> +			of the flash.
> +- maxim,boost-mode :
> +	In boost mode the device can produce up to 1.2A of total current
> +	on both outputs. The maximum current on each output is reduced
> +	to 625mA then. If not enabled explicitly, boost setting defaults to
> +	LEDS_BOOST_FIXED in case both current sources are used.
> +	Possible values:
> +		LEDS_BOOST_OFF (0) - no boost,
> +		LEDS_BOOST_ADAPTIVE (1) - adaptive mode,
> +		LEDS_BOOST_FIXED (2) - fixed mode.
> +- maxim,boost-mvout : Output voltage of the boost module in millivolts.
> +- maxim,mvsys-min : Low input voltage level in millivolts. Flash is not fired
> +	if chip estimates that system voltage could drop below this level due
> +	to flash power consumption.
> +
> +Required properties of the LED child node:
> +- led-sources : see Documentation/devicetree/bindings/leds/common.txt;
> +		device current output identifiers: 0 - FLED1, 1 - FLED2
> +
> +Optional properties of the LED child node:
> +- label : see Documentation/devicetree/bindings/leds/common.txt
> +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> +		Range: 15625 - 250000
> +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> +		Range: 15625 - 1000000
> +- flash-timeout-us : see Documentation/devicetree/bindings/leds/common.txt
> +		Range: 62500 - 1000000
> +
>  Example:
> +#include <dt-bindings/leds/common.h>
> +
>  	max77693@66 {
>  		compatible = "maxim,max77693";
>  		reg = <0x66>;
> @@ -117,5 +163,20 @@ Example:
>  			maxim,thermal-regulation-celsius = <75>;
>  			maxim,battery-overcurrent-microamp = <3000000>;
>  			maxim,charge-input-threshold-microvolt = <4300000>;
> +
> +		led {
> +			compatible = "maxim,max77693-led";
> +			maxim,trigger-type = <LEDS_TRIG_TYPE_LEVEL>;
> +			maxim,boost-mode = <LEDS_BOOST_FIXED>;
> +			maxim,boost-mvout = <5000>;
> +			maxim,mvsys-min = <2400>;
> +
> +			camera_flash: flash-led {
> +				label = "max77693-flash";
> +				led-sources = <0>, <1>;
> +				max-microamp = <500000>;
> +				flash-max-microamp = <1250000>;
> +				flash-timeout-us = <1000000>;
> +			};
>  		};
>  	};
  
Pavel Machek March 23, 2015, 2:22 p.m. UTC | #2
On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> This patch requires a DT Ack.

No, it requires DT people to be notified -- and they were, few times
by now.

They clearly don't care.

> > This patch adds device tree binding documentation for
> > the flash cell of the Maxim max77693 multifunctional device.
> > 
> > Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
> > Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> > Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> > Cc: Lee Jones <lee.jones@linaro.org>
> > Cc: Chanwoo Choi <cw00.choi@samsung.com>
> > Cc: Bryan Wu <cooloney@gmail.com>
> > Cc: Richard Purdie <rpurdie@rpsys.net>

Acked-by: Pavel Machek <pavel@ucw.cz>


> > diff --git a/Documentation/devicetree/bindings/mfd/max77693.txt b/Documentation/devicetree/bindings/mfd/max77693.txt
> > index 38e6440..15c546e 100644
> > --- a/Documentation/devicetree/bindings/mfd/max77693.txt
> > +++ b/Documentation/devicetree/bindings/mfd/max77693.txt
> > @@ -76,7 +76,53 @@ Optional properties:
> >      Valid values: 4300000, 4700000, 4800000, 4900000
> >      Default: 4300000
> >  
> > +- led : the LED submodule device node
> > +
> > +There are two LED outputs available - FLED1 and FLED2. Each of them can
> > +control a separate LED or they can be connected together to double
> > +the maximum current for a single connected LED. One LED is represented
> > +by one child node.
> > +
> > +Required properties:
> > +- compatible : Must be "maxim,max77693-led".
> > +
> > +Optional properties:
> > +- maxim,trigger-type : Flash trigger type.
> > +	Possible trigger types:
> > +		LEDS_TRIG_TYPE_EDGE (0) - Rising edge of the signal triggers
> > +			the flash,
> > +		LEDS_TRIG_TYPE_LEVEL (1) - Strobe pulse length controls duration
> > +			of the flash.
> > +- maxim,boost-mode :
> > +	In boost mode the device can produce up to 1.2A of total current
> > +	on both outputs. The maximum current on each output is reduced
> > +	to 625mA then. If not enabled explicitly, boost setting defaults to
> > +	LEDS_BOOST_FIXED in case both current sources are used.
> > +	Possible values:
> > +		LEDS_BOOST_OFF (0) - no boost,
> > +		LEDS_BOOST_ADAPTIVE (1) - adaptive mode,
> > +		LEDS_BOOST_FIXED (2) - fixed mode.
> > +- maxim,boost-mvout : Output voltage of the boost module in millivolts.
> > +- maxim,mvsys-min : Low input voltage level in millivolts. Flash is not fired
> > +	if chip estimates that system voltage could drop below this level due
> > +	to flash power consumption.
> > +
> > +Required properties of the LED child node:
> > +- led-sources : see Documentation/devicetree/bindings/leds/common.txt;
> > +		device current output identifiers: 0 - FLED1, 1 - FLED2
> > +
> > +Optional properties of the LED child node:
> > +- label : see Documentation/devicetree/bindings/leds/common.txt
> > +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> > +		Range: 15625 - 250000
> > +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> > +		Range: 15625 - 1000000
> > +- flash-timeout-us : see Documentation/devicetree/bindings/leds/common.txt
> > +		Range: 62500 - 1000000
> > +
> >  Example:
> > +#include <dt-bindings/leds/common.h>
> > +
> >  	max77693@66 {
> >  		compatible = "maxim,max77693";
> >  		reg = <0x66>;
> > @@ -117,5 +163,20 @@ Example:
> >  			maxim,thermal-regulation-celsius = <75>;
> >  			maxim,battery-overcurrent-microamp = <3000000>;
> >  			maxim,charge-input-threshold-microvolt = <4300000>;
> > +
> > +		led {
> > +			compatible = "maxim,max77693-led";
> > +			maxim,trigger-type = <LEDS_TRIG_TYPE_LEVEL>;
> > +			maxim,boost-mode = <LEDS_BOOST_FIXED>;
> > +			maxim,boost-mvout = <5000>;
> > +			maxim,mvsys-min = <2400>;
> > +
> > +			camera_flash: flash-led {
> > +				label = "max77693-flash";
> > +				led-sources = <0>, <1>;
> > +				max-microamp = <500000>;
> > +				flash-max-microamp = <1250000>;
> > +				flash-timeout-us = <1000000>;
> > +			};
> >  		};
> >  	};
> 
>
  
Lee Jones March 23, 2015, 3:02 p.m. UTC | #3
On Mon, 23 Mar 2015, Pavel Machek wrote:
> On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> > This patch requires a DT Ack.
> 
> No, it requires DT people to be notified -- and they were, few times
> by now.
> 
> They clearly don't care.

Well fortunately for the Kernel community, I do care.  And as this
patch adds 3 new DT properties, has been through many iterations
already with vast changes made over that period and there is still
some controversy looming, I'm saying that it _does_ require a DT Ack.

> > > This patch adds device tree binding documentation for
> > > the flash cell of the Maxim max77693 multifunctional device.
> > > 
> > > Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
> > > Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
> > > Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
> > > Cc: Lee Jones <lee.jones@linaro.org>
> > > Cc: Chanwoo Choi <cw00.choi@samsung.com>
> > > Cc: Bryan Wu <cooloney@gmail.com>
> > > Cc: Richard Purdie <rpurdie@rpsys.net>
> 
> Acked-by: Pavel Machek <pavel@ucw.cz>
> 
> 
> > > diff --git a/Documentation/devicetree/bindings/mfd/max77693.txt b/Documentation/devicetree/bindings/mfd/max77693.txt
> > > index 38e6440..15c546e 100644
> > > --- a/Documentation/devicetree/bindings/mfd/max77693.txt
> > > +++ b/Documentation/devicetree/bindings/mfd/max77693.txt
> > > @@ -76,7 +76,53 @@ Optional properties:
> > >      Valid values: 4300000, 4700000, 4800000, 4900000
> > >      Default: 4300000
> > >  
> > > +- led : the LED submodule device node
> > > +
> > > +There are two LED outputs available - FLED1 and FLED2. Each of them can
> > > +control a separate LED or they can be connected together to double
> > > +the maximum current for a single connected LED. One LED is represented
> > > +by one child node.
> > > +
> > > +Required properties:
> > > +- compatible : Must be "maxim,max77693-led".
> > > +
> > > +Optional properties:
> > > +- maxim,trigger-type : Flash trigger type.
> > > +	Possible trigger types:
> > > +		LEDS_TRIG_TYPE_EDGE (0) - Rising edge of the signal triggers
> > > +			the flash,
> > > +		LEDS_TRIG_TYPE_LEVEL (1) - Strobe pulse length controls duration
> > > +			of the flash.
> > > +- maxim,boost-mode :
> > > +	In boost mode the device can produce up to 1.2A of total current
> > > +	on both outputs. The maximum current on each output is reduced
> > > +	to 625mA then. If not enabled explicitly, boost setting defaults to
> > > +	LEDS_BOOST_FIXED in case both current sources are used.
> > > +	Possible values:
> > > +		LEDS_BOOST_OFF (0) - no boost,
> > > +		LEDS_BOOST_ADAPTIVE (1) - adaptive mode,
> > > +		LEDS_BOOST_FIXED (2) - fixed mode.
> > > +- maxim,boost-mvout : Output voltage of the boost module in millivolts.
> > > +- maxim,mvsys-min : Low input voltage level in millivolts. Flash is not fired
> > > +	if chip estimates that system voltage could drop below this level due
> > > +	to flash power consumption.
> > > +
> > > +Required properties of the LED child node:
> > > +- led-sources : see Documentation/devicetree/bindings/leds/common.txt;
> > > +		device current output identifiers: 0 - FLED1, 1 - FLED2
> > > +
> > > +Optional properties of the LED child node:
> > > +- label : see Documentation/devicetree/bindings/leds/common.txt
> > > +- max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> > > +		Range: 15625 - 250000
> > > +- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt
> > > +		Range: 15625 - 1000000
> > > +- flash-timeout-us : see Documentation/devicetree/bindings/leds/common.txt
> > > +		Range: 62500 - 1000000
> > > +
> > >  Example:
> > > +#include <dt-bindings/leds/common.h>
> > > +
> > >  	max77693@66 {
> > >  		compatible = "maxim,max77693";
> > >  		reg = <0x66>;
> > > @@ -117,5 +163,20 @@ Example:
> > >  			maxim,thermal-regulation-celsius = <75>;
> > >  			maxim,battery-overcurrent-microamp = <3000000>;
> > >  			maxim,charge-input-threshold-microvolt = <4300000>;
> > > +
> > > +		led {
> > > +			compatible = "maxim,max77693-led";
> > > +			maxim,trigger-type = <LEDS_TRIG_TYPE_LEVEL>;
> > > +			maxim,boost-mode = <LEDS_BOOST_FIXED>;
> > > +			maxim,boost-mvout = <5000>;
> > > +			maxim,mvsys-min = <2400>;
> > > +
> > > +			camera_flash: flash-led {
> > > +				label = "max77693-flash";
> > > +				led-sources = <0>, <1>;
> > > +				max-microamp = <500000>;
> > > +				flash-max-microamp = <1250000>;
> > > +				flash-timeout-us = <1000000>;
> > > +			};
> > >  		};
> > >  	};
> > 
> > 
>
  
Pavel Machek March 23, 2015, 3:48 p.m. UTC | #4
On Mon 2015-03-23 15:02:13, Lee Jones wrote:
> On Mon, 23 Mar 2015, Pavel Machek wrote:
> > On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> > > This patch requires a DT Ack.
> > 
> > No, it requires DT people to be notified -- and they were, few times
> > by now.
> > 
> > They clearly don't care.
> 
> Well fortunately for the Kernel community, I do care.  And as this
> patch adds 3 new DT properties, has been through many iterations
> already with vast changes made over that period and there is still
> some controversy looming, I'm saying that it _does_ require a DT
> Ack.

Can you help get that ack, then? As a maintainer, you have better
chance getting reply from DT people than patch submitter.
								Pavel
  
Lee Jones March 23, 2015, 3:59 p.m. UTC | #5
On Mon, 23 Mar 2015, Pavel Machek wrote:

> On Mon 2015-03-23 15:02:13, Lee Jones wrote:
> > On Mon, 23 Mar 2015, Pavel Machek wrote:
> > > On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> > > > This patch requires a DT Ack.
> > > 
> > > No, it requires DT people to be notified -- and they were, few times
> > > by now.
> > > 
> > > They clearly don't care.
> > 
> > Well fortunately for the Kernel community, I do care.  And as this
> > patch adds 3 new DT properties, has been through many iterations
> > already with vast changes made over that period and there is still
> > some controversy looming, I'm saying that it _does_ require a DT
> > Ack.
> 
> Can you help get that ack, then? As a maintainer, you have better
> chance getting reply from DT people than patch submitter.

Hopefully they will see my plea a couple of replies back.

If that fails I'll go poke them via other means.  Failing that I'll go
see them in person and continually hit them with wet fish until one of
them relents.
  
Sylwester Nawrocki March 23, 2015, 6:28 p.m. UTC | #6
On 23/03/15 16:59, Lee Jones wrote:
> On Mon, 23 Mar 2015, Pavel Machek wrote:
> 
>> > On Mon 2015-03-23 15:02:13, Lee Jones wrote:
>>> > > On Mon, 23 Mar 2015, Pavel Machek wrote:
>>>> > > > On Mon 2015-03-23 12:07:43, Lee Jones wrote:
>>>>> > > > > This patch requires a DT Ack.
>>>> > > > 
>>>> > > > No, it requires DT people to be notified -- and they were, few times
>>>> > > > by now.
>>>> > > > 
>>>> > > > They clearly don't care.
>>> > > 
>>> > > Well fortunately for the Kernel community, I do care.  And as this
>>> > > patch adds 3 new DT properties, has been through many iterations
>>> > > already with vast changes made over that period and there is still
>>> > > some controversy looming, I'm saying that it _does_ require a DT
>>> > > Ack.
>> > 
>> > Can you help get that ack, then? As a maintainer, you have better
>> > chance getting reply from DT people than patch submitter.
>
> Hopefully they will see my plea a couple of replies back.
> 
> If that fails I'll go poke them via other means.  Failing that I'll go
> see them in person and continually hit them with wet fish until one of
> them relents.

Nice, as we can see it doesn't take _that_ much to be a really good
subsystem maintainer. I wish you good luck and hope there will be no
need to realize the very last steps of that clever plan.
  
Pavel Machek April 2, 2015, 2:50 p.m. UTC | #7
On Mon 2015-03-23 15:59:37, Lee Jones wrote:
> On Mon, 23 Mar 2015, Pavel Machek wrote:
> 
> > On Mon 2015-03-23 15:02:13, Lee Jones wrote:
> > > On Mon, 23 Mar 2015, Pavel Machek wrote:
> > > > On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> > > > > This patch requires a DT Ack.
> > > > 
> > > > No, it requires DT people to be notified -- and they were, few times
> > > > by now.
> > > > 
> > > > They clearly don't care.
> > > 
> > > Well fortunately for the Kernel community, I do care.  And as this
> > > patch adds 3 new DT properties, has been through many iterations
> > > already with vast changes made over that period and there is still
> > > some controversy looming, I'm saying that it _does_ require a DT
> > > Ack.
> > 
> > Can you help get that ack, then? As a maintainer, you have better
> > chance getting reply from DT people than patch submitter.
> 
> Hopefully they will see my plea a couple of replies back.
> 
> If that fails I'll go poke them via other means.  Failing that I'll go
> see them in person and continually hit them with wet fish until one of
> them relents.

Can I request footage of the last steps on the youtube? (IOW ping, I
don't think I seen any replies from DT maintainers...)
									Pavel
  
Lee Jones April 2, 2015, 7:33 p.m. UTC | #8
On Thu, 02 Apr 2015, Pavel Machek wrote:

> On Mon 2015-03-23 15:59:37, Lee Jones wrote:
> > On Mon, 23 Mar 2015, Pavel Machek wrote:
> > 
> > > On Mon 2015-03-23 15:02:13, Lee Jones wrote:
> > > > On Mon, 23 Mar 2015, Pavel Machek wrote:
> > > > > On Mon 2015-03-23 12:07:43, Lee Jones wrote:
> > > > > > This patch requires a DT Ack.
> > > > > 
> > > > > No, it requires DT people to be notified -- and they were, few times
> > > > > by now.
> > > > > 
> > > > > They clearly don't care.
> > > > 
> > > > Well fortunately for the Kernel community, I do care.  And as this
> > > > patch adds 3 new DT properties, has been through many iterations
> > > > already with vast changes made over that period and there is still
> > > > some controversy looming, I'm saying that it _does_ require a DT
> > > > Ack.
> > > 
> > > Can you help get that ack, then? As a maintainer, you have better
> > > chance getting reply from DT people than patch submitter.
> > 
> > Hopefully they will see my plea a couple of replies back.
> > 
> > If that fails I'll go poke them via other means.  Failing that I'll go
> > see them in person and continually hit them with wet fish until one of
> > them relents.
> 
> Can I request footage of the last steps on the youtube?

Too late, it's been and gone.

> (IOW ping, I don't think I seen any replies from DT maintainers...)

Then you haven't been looking the right place [1]. ;)

[1] http://comments.gmane.org/gmane.linux.leds/2245
  

Patch

diff --git a/Documentation/devicetree/bindings/mfd/max77693.txt b/Documentation/devicetree/bindings/mfd/max77693.txt
index 38e6440..15c546e 100644
--- a/Documentation/devicetree/bindings/mfd/max77693.txt
+++ b/Documentation/devicetree/bindings/mfd/max77693.txt
@@ -76,7 +76,53 @@  Optional properties:
     Valid values: 4300000, 4700000, 4800000, 4900000
     Default: 4300000
 
+- led : the LED submodule device node
+
+There are two LED outputs available - FLED1 and FLED2. Each of them can
+control a separate LED or they can be connected together to double
+the maximum current for a single connected LED. One LED is represented
+by one child node.
+
+Required properties:
+- compatible : Must be "maxim,max77693-led".
+
+Optional properties:
+- maxim,trigger-type : Flash trigger type.
+	Possible trigger types:
+		LEDS_TRIG_TYPE_EDGE (0) - Rising edge of the signal triggers
+			the flash,
+		LEDS_TRIG_TYPE_LEVEL (1) - Strobe pulse length controls duration
+			of the flash.
+- maxim,boost-mode :
+	In boost mode the device can produce up to 1.2A of total current
+	on both outputs. The maximum current on each output is reduced
+	to 625mA then. If not enabled explicitly, boost setting defaults to
+	LEDS_BOOST_FIXED in case both current sources are used.
+	Possible values:
+		LEDS_BOOST_OFF (0) - no boost,
+		LEDS_BOOST_ADAPTIVE (1) - adaptive mode,
+		LEDS_BOOST_FIXED (2) - fixed mode.
+- maxim,boost-mvout : Output voltage of the boost module in millivolts.
+- maxim,mvsys-min : Low input voltage level in millivolts. Flash is not fired
+	if chip estimates that system voltage could drop below this level due
+	to flash power consumption.
+
+Required properties of the LED child node:
+- led-sources : see Documentation/devicetree/bindings/leds/common.txt;
+		device current output identifiers: 0 - FLED1, 1 - FLED2
+
+Optional properties of the LED child node:
+- label : see Documentation/devicetree/bindings/leds/common.txt
+- max-microamp : see Documentation/devicetree/bindings/leds/common.txt
+		Range: 15625 - 250000
+- flash-max-microamp : see Documentation/devicetree/bindings/leds/common.txt
+		Range: 15625 - 1000000
+- flash-timeout-us : see Documentation/devicetree/bindings/leds/common.txt
+		Range: 62500 - 1000000
+
 Example:
+#include <dt-bindings/leds/common.h>
+
 	max77693@66 {
 		compatible = "maxim,max77693";
 		reg = <0x66>;
@@ -117,5 +163,20 @@  Example:
 			maxim,thermal-regulation-celsius = <75>;
 			maxim,battery-overcurrent-microamp = <3000000>;
 			maxim,charge-input-threshold-microvolt = <4300000>;
+
+		led {
+			compatible = "maxim,max77693-led";
+			maxim,trigger-type = <LEDS_TRIG_TYPE_LEVEL>;
+			maxim,boost-mode = <LEDS_BOOST_FIXED>;
+			maxim,boost-mvout = <5000>;
+			maxim,mvsys-min = <2400>;
+
+			camera_flash: flash-led {
+				label = "max77693-flash";
+				led-sources = <0>, <1>;
+				max-microamp = <500000>;
+				flash-max-microamp = <1250000>;
+				flash-timeout-us = <1000000>;
+			};
 		};
 	};