linux-kernel - Re: [PATCH v8 0/5] power: supply: MAX17040: Add IRQ for low level and alert SOC changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20191219005250.w3m7pbjisykxx7iz@earth.universe>
Date:   Thu, 19 Dec 2019 01:52:50 +0100
From:   Sebastian Reichel <sebastian.reichel@...labora.com>
To:     Matheus Castello <matheus@...tello.eng.br>
Cc:     krzk@...nel.org, robh+dt@...nel.org, mark.rutland@....com,
        cw00.choi@...sung.com, b.zolnierkie@...sung.com,
        lee.jones@...aro.org, linux-pm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v8 0/5] power: supply: MAX17040: Add IRQ for low level
 and alert SOC changes

Hi,

On Thu, Dec 05, 2019 at 12:44:05PM -0300, 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.
> 
> Thanks Krzysztof Kozlowski, Lee Jones, Rob Herring and Sebastian Reichel for
> your time reviewing it.

Thanks, I queued all but the MFD patch to power-supply's for-next
branch.

-- Sebastian

> Changes since v7:
> (Suggested by Sebastian Reichel)
> - Fix unrelated change
> - Use device_property_read_u32() and simplify the code
> - Check the irq and of_device_is_compatible in the first patch
> 
> Changes since v6:
> (Suggested by Lee Jones)
> - Use relative paths in documentation
> 
> 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       | 130 +++++++++++++++++-
>  3 files changed, 161 insertions(+), 4 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt
> 
> --
> 2.24.0.rc2
> 

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)