| Message ID | 20180723040816.19455-4-matheus@castello.eng.br (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | power: supply: MAX17040: Add IRQ for low level and alert SOC changes | expand |
On 23 July 2018 at 06:08, 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> > --- > .../bindings/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..e6e4e46c03c4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt > @@ -0,0 +1,24 @@ > +max17040_battery > +~~~~~~~~~~~~~~~~ > + > +Required properties : > + - compatible : "maxim,max17040" Why you skipped "maxim,max77836-battery"? > + > +Optional threshold properties : > + If skipped the power up default value will be used > + - maxim,alert-soc-level : The alert threshold that sets the state of > + charge level where an interrupt is generated. > + max17040 can be configured from 1 up to 32. > + > +Remembering that for the interrupt to be handled it must also be described the > +information of the interruption in the node. Just mention interrupts in optional properties, including the flags. BTW, the driver hard-codes the flags which is in contrast with DT here. Best regards, Krzysztof > + > +Example: > + > + battery-charger@36 { > + compatible = "maxim,max17040"; > + reg = <0x36>; > + maxim,alert-soc-level = <10>; > + interrupt-parent = <&gpio>; > + interrupts = <18 IRQ_TYPE_EDGE_FALLING>; > + }; > -- > 2.13.3 > > -- > 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
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..e6e4e46c03c4 --- /dev/null +++ b/Documentation/devicetree/bindings/power/supply/max17040_battery.txt @@ -0,0 +1,24 @@ +max17040_battery +~~~~~~~~~~~~~~~~ + +Required properties : + - compatible : "maxim,max17040" + +Optional threshold properties : + If skipped the power up default value will be used + - maxim,alert-soc-level : The alert threshold that sets the state of + charge level where an interrupt is generated. + max17040 can be configured from 1 up to 32. + +Remembering that for the interrupt to be handled it must also be described the +information of the interruption in the node. + +Example: + + battery-charger@36 { + compatible = "maxim,max17040"; + reg = <0x36>; + maxim,alert-soc-level = <10>; + interrupt-parent = <&gpio>; + interrupts = <18 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> --- .../bindings/power/supply/max17040_battery.txt | 24 ++++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt