| Message ID | 20191107031710.5672-1-matheus@castello.eng.br (mailing list archive) |
|---|---|
| Headers | show |
| Series | power: supply: MAX17040: Add IRQ for low level and alert SOC changes | expand |
On Thu, 07 Nov 2019, Matheus Castello wrote: > This series add IRQ handler for low level SOC alert, define a devicetree > binding attribute to configure the alert level threshold and check for > changes in SOC and power supply status for send uevents. > > Max17043/17044 have a pin for alert host about low level state of charge and > this alert can be configured in a threshold from 1% up to 32% of SOC. > > Tested on Toradex Colibri iMX7D, with a SparkFun Lipo Fuel Gauge module > based on MAXIM MAX17043. > > Thanks Krzysztof for your time reviewing it. Let me know what you think about > the fixes. > > Changes since v5: > (Suggested by Krzysztof Kozlowski) > - Rearrange code and add max17040_enable_alert_irq on patch 1/5 > - Remove useless dev_info Just something to bear in mind in the future: When submitting subsequent versions, could you please do so as separate sets? Attaching them to previous submissions gets confusing real quick.
This series add IRQ handler for low level SOC alert, define a devicetree binding attribute to configure the alert level threshold and check for changes in SOC and power supply status for send uevents. Max17043/17044 have a pin for alert host about low level state of charge and this alert can be configured in a threshold from 1% up to 32% of SOC. Tested on Toradex Colibri iMX7D, with a SparkFun Lipo Fuel Gauge module based on MAXIM MAX17043. Thanks Krzysztof for your time reviewing it. Let me know what you think about the fixes. Changes since v5: (Suggested by Krzysztof Kozlowski) - Rearrange code and add max17040_enable_alert_irq on patch 1/5 - Remove useless dev_info Changes since v4: (Suggested by Krzysztof Kozlowski) - Fix code style and alignment issues - Keep IRQF_TRIGGER_FALLING | IRQF_ONESHOT instead client->flags (Suggested by Rob Herring) - Add reference to the MFD description - Fix the dt-bindings commit description Matheus Castello (5): power: supply: max17040: Add IRQ handler for low SOC alert dt-bindings: power: supply: Max17040: Add DT bindings for max17040 fuel gauge devicetree: mfd: max14577: Add reference to max14040_battery.txt descriptions power: supply: max17040: Config alert SOC low level threshold from FDT power: supply: max17040: Send uevent in SOC and status change .../devicetree/bindings/mfd/max14577.txt | 2 + .../power/supply/max17040_battery.txt | 33 ++++ drivers/power/supply/max17040_battery.c | 141 +++++++++++++++++- 3 files changed, 171 insertions(+), 5 deletions(-) create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt -- 2.24.0.rc2