[0/4] power: supply: MAX17040: Add IRQ for low level and alert SOC changes

Message

Matheus Castello July 23, 2018, 4:08 a.m. UTC

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 for send uevents.

Max17040 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 Raspberry Pi Zero W, with a SparkFun Lipo Fuel Gauge module based on
MAXIM MAX17043.

Matheus Castello (4):
  power: supply: max17040: Add IRQ handler for low SOC alert
  power: supply: max17040: Config alert SOC low level threshold from FDT
  dt-bindings: power: supply: Max17040: Add low level SOC alert threshold
  power: supply: max17040: Send uevent in SOC changes

 .../bindings/power/supply/max17040_battery.txt     | 24 ++++++
 drivers/power/supply/max17040_battery.c            | 95 ++++++++++++++++++++++
 2 files changed, 119 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt

Comments

Sebastian Reichel Sept. 16, 2018, 11:45 a.m. UTC | #1

Hi Matheus,

Did I miss a v2 of this patchset, that solves the issues
found by Krzysztof?

-- Sebastian

On Mon, Jul 23, 2018 at 12:08:12AM -0400, 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 for send uevents.
> 
> Max17040 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 Raspberry Pi Zero W, with a SparkFun Lipo Fuel Gauge module based on
> MAXIM MAX17043.
> 
> Matheus Castello (4):
>   power: supply: max17040: Add IRQ handler for low SOC alert
>   power: supply: max17040: Config alert SOC low level threshold from FDT
>   dt-bindings: power: supply: Max17040: Add low level SOC alert threshold
>   power: supply: max17040: Send uevent in SOC changes
> 
>  .../bindings/power/supply/max17040_battery.txt     | 24 ++++++
>  drivers/power/supply/max17040_battery.c            | 95 ++++++++++++++++++++++
>  2 files changed, 119 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/power/supply/max17040_battery.txt
> 
> -- 
> 2.13.3
>

Krzysztof Kozlowski Sept. 17, 2018, 11:32 a.m. UTC | #2

On Sun, 16 Sep 2018 at 22:03, Sebastian Reichel
<[e-mail]> wrote:
>
> Hi Matheus,
>
> Did I miss a v2 of this patchset, that solves the issues
> found by Krzysztof?

I did not see v2. This patchset brings nice feature so it would be a
pity if it were discontinued.

Best regards,
Krzysztof

Matheus Castello Sept. 18, 2018, 3:45 a.m. UTC | #3

Hi Krzysztof and Sebastian,

please forgive me for the delay in working with this patch.
I've been having some personal issues these months, but leaving the 
excuses I still intend to send a v2 for this.

Thanks Krzysztof for review and tips I'll work on it.

Best Regards,
Matheus Castello

On 09/17/2018 08:32 AM, Krzysztof Kozlowski wrote:
> On Sun, 16 Sep 2018 at 22:03, Sebastian Reichel
> <[e-mail]> wrote:
>>
>> Hi Matheus,
>>
>> Did I miss a v2 of this patchset, that solves the issues
>> found by Krzysztof?
> 
> I did not see v2. This patchset brings nice feature so it would be a
> pity if it were discontinued.
> 
> Best regards,
> Krzysztof
>