Message ID | 20190415012635.6369-3-matheus@castello.eng.br |
---|---|
State | Changes Requested, archived |
Headers | show |
Series | power: supply: MAX17040: Add IRQ for low level and alert SOC changes | expand |
Context | Check | Description |
---|---|---|
robh/checkpatch | success |
On Mon, 15 Apr 2019 at 03:50, Matheus Castello <matheus@castello.eng.br> wrote: > > For configure low level state of charge threshold alert signaled from > max17040 we add "maxim,alert-soc-level" property. > > Signed-off-by: Matheus Castello <matheus@castello.eng.br> > --- > .../power/supply/max17040_battery.txt | 24 +++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt > > diff --git a/Documentation/devicetree/bindings/power/supply/max17040_battery.txt b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt > new file mode 100644 > index 000000000000..9b2cc67d556f > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt > @@ -0,0 +1,24 @@ > +max17040_battery > +~~~~~~~~~~~~~~~~ > + > +Required properties : > + - compatible : "maxim,max17040" or "maxim,max77836-battery" > + > +Optional properties : > +- maxim,alert-soc-level : The alert threshold that sets the state of > + charge level where an interrupt is generated. > + Can be configured from 1 up to 32. If skipped > + the power up default value of 4 will be used. > +- interrupt-parent : The GPIO bank from the interrupt line. It does not have to be GPIO so: "phandle of the parent interrupt controller" > +- interrupts : Interrupt line see Documentation/devicetree/ > + bindings/interrupt-controller/interrupts.txt > + > +Example: > + > + battery-charger@36 { It is a fuel gauge, not battery charger. Best regards, Krzysztof > + compatible = "maxim,max17040"; > + reg = <0x36>; > + maxim,alert-soc-level = <10>; > + interrupt-parent = <&gpio7>; > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > + }; > -- > 2.17.0 >
On Sun, Apr 14, 2019 at 10:26:33PM -0300, Matheus Castello wrote: > For configure low level state of charge threshold alert signaled from > max17040 we add "maxim,alert-soc-level" property. > > Signed-off-by: Matheus Castello <matheus@castello.eng.br> > --- > .../power/supply/max17040_battery.txt | 24 +++++++++++++++++++ > 1 file changed, 24 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt > > diff --git a/Documentation/devicetree/bindings/power/supply/max17040_battery.txt b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt > new file mode 100644 > index 000000000000..9b2cc67d556f > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt > @@ -0,0 +1,24 @@ > +max17040_battery > +~~~~~~~~~~~~~~~~ > + > +Required properties : > + - compatible : "maxim,max17040" or "maxim,max77836-battery" This is really a charger, not a battery. > + > +Optional properties : > +- maxim,alert-soc-level : The alert threshold that sets the state of > + charge level where an interrupt is generated. > + Can be configured from 1 up to 32. If skipped > + the power up default value of 4 will be used. Units? This is a low or high alert? Does a common property make sense here? > +- interrupt-parent : The GPIO bank from the interrupt line. Drop this. interrupt-parent is implied. > +- interrupts : Interrupt line see Documentation/devicetree/ > + bindings/interrupt-controller/interrupts.txt > + > +Example: > + > + battery-charger@36 { > + compatible = "maxim,max17040"; > + reg = <0x36>; > + maxim,alert-soc-level = <10>; > + interrupt-parent = <&gpio7>; > + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; Usually there are battery properties that need to be described too... > + }; > -- > 2.17.0 >
On 4/29/19 7:13 PM, Rob Herring wrote: > On Sun, Apr 14, 2019 at 10:26:33PM -0300, Matheus Castello wrote: >> For configure low level state of charge threshold alert signaled from >> max17040 we add "maxim,alert-soc-level" property. >> >> Signed-off-by: Matheus Castello <matheus@castello.eng.br> >> --- >> .../power/supply/max17040_battery.txt | 24 +++++++++++++++++++ >> 1 file changed, 24 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt >> >> diff --git a/Documentation/devicetree/bindings/power/supply/max17040_battery.txt b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt >> new file mode 100644 >> index 000000000000..9b2cc67d556f >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt >> @@ -0,0 +1,24 @@ >> +max17040_battery >> +~~~~~~~~~~~~~~~~ >> + >> +Required properties : >> + - compatible : "maxim,max17040" or "maxim,max77836-battery" > > This is really a charger, not a battery. > max17040 is a fuel gauge, max77836 MUIC has it integrated. Because of this we use it in the compatible list. >> + >> +Optional properties : >> +- maxim,alert-soc-level : The alert threshold that sets the state of >> + charge level where an interrupt is generated. >> + Can be configured from 1 up to 32. If skipped >> + the power up default value of 4 will be used. > > Units? This is a low or high alert? Does a common property make sense > here? > It is a low level alert. I will change the name of the property to "maxim,alert-low-soc-level" to make this clear and I will put the percent unit in the description. I do not find any common property that I can use here, if I am wrong let me know. >> +- interrupt-parent : The GPIO bank from the interrupt line. > > Drop this. interrupt-parent is implied. Ok, I will do. > >> +- interrupts : Interrupt line see Documentation/devicetree/ >> + bindings/interrupt-controller/interrupts.txt >> + >> +Example: >> + >> + battery-charger@36 { >> + compatible = "maxim,max17040"; >> + reg = <0x36>; >> + maxim,alert-soc-level = <10>; >> + interrupt-parent = <&gpio7>; >> + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; > > Usually there are battery properties that need to be described too... > I will fix this for "battery-fuel-gauge@36". I will also add the description for wake-source as optional property. Thanks. Best Regards, Matheus Castello >> + }; >> -- >> 2.17.0 >>
diff --git a/Documentation/devicetree/bindings/power/supply/max17040_battery.txt b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt new file mode 100644 index 000000000000..9b2cc67d556f --- /dev/null +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt @@ -0,0 +1,24 @@ +max17040_battery +~~~~~~~~~~~~~~~~ + +Required properties : + - compatible : "maxim,max17040" or "maxim,max77836-battery" + +Optional properties : +- maxim,alert-soc-level : The alert threshold that sets the state of + charge level where an interrupt is generated. + Can be configured from 1 up to 32. If skipped + the power up default value of 4 will be used. +- interrupt-parent : The GPIO bank from the interrupt line. +- interrupts : Interrupt line see Documentation/devicetree/ + bindings/interrupt-controller/interrupts.txt + +Example: + + battery-charger@36 { + compatible = "maxim,max17040"; + reg = <0x36>; + maxim,alert-soc-level = <10>; + interrupt-parent = <&gpio7>; + interrupts = <2 IRQ_TYPE_EDGE_FALLING>; + };
For configure low level state of charge threshold alert signaled from max17040 we add "maxim,alert-soc-level" property. Signed-off-by: Matheus Castello <matheus@castello.eng.br> --- .../power/supply/max17040_battery.txt | 24 +++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt