[0/5] power: supply: ab8500 charger cleanup

Message

Linus Walleij Dec. 12, 2020, 10:57 a.m. UTC

This patch set is a first set of cleanups for the AB8500
charging code, I'm trying to bring this to a working
state. The code has seen some bitrot since it was merged
a few years back.

Notably it makes the code probe properly when enabled.

Linus Walleij (5):
  power: supply: ab8500: Use local helper
  power: supply: ab8500: Convert to dev_pm_ops
  power: supply: ab8500_charger: Oneshot threaded IRQs
  power: supply: ab8500_fg: Request all IRQs as threaded
  power: supply: ab8500: Use dev_err_probe() for IIO channels

 drivers/power/supply/ab8500_btemp.c    |  66 +++++++--------
 drivers/power/supply/ab8500_charger.c  |  99 ++++++++++-------------
 drivers/power/supply/ab8500_fg.c       | 106 +++++++++----------------
 drivers/power/supply/abx500_chargalg.c |  19 ++---
 4 files changed, 116 insertions(+), 174 deletions(-)

Comments

Sebastian Reichel Dec. 12, 2020, 11:40 p.m. UTC | #1

Hi,

On Sat, Dec 12, 2020 at 11:57:07AM +0100, Linus Walleij wrote:
> This patch set is a first set of cleanups for the AB8500
> charging code, I'm trying to bring this to a working
> state. The code has seen some bitrot since it was merged
> a few years back.
> 
> Notably it makes the code probe properly when enabled.

Thanks, queued!

-- Sebastian

> 
> Linus Walleij (5):
>   power: supply: ab8500: Use local helper
>   power: supply: ab8500: Convert to dev_pm_ops
>   power: supply: ab8500_charger: Oneshot threaded IRQs
>   power: supply: ab8500_fg: Request all IRQs as threaded
>   power: supply: ab8500: Use dev_err_probe() for IIO channels
> 
>  drivers/power/supply/ab8500_btemp.c    |  66 +++++++--------
>  drivers/power/supply/ab8500_charger.c  |  99 ++++++++++-------------
>  drivers/power/supply/ab8500_fg.c       | 106 +++++++++----------------
>  drivers/power/supply/abx500_chargalg.c |  19 ++---
>  4 files changed, 116 insertions(+), 174 deletions(-)
> 
> -- 
> 2.26.2
>