[v2,00/16] AB8500 charger to use power_supply_battery_info

Message

Linus Walleij Nov. 20, 2021, 3:53 p.m. UTC

This is the first set of patches starting to migrate the
AB8500 charging code to use the struct power_supply_battery_info.
We drop some cruft along the road.

This series does not add anything to the struct, just reuse
what is already there. Adding new stuff comes in the next
patch series.

The AB8500 charging code has not been in working condition for
some time, but it is slowly getting there. Some of this is
just regular maintenance.

ChangeLog v1->v2:
- Fixed review comments on patch 1.

Linus Walleij (16):
  power: supply: ab8500: Use core battery parser
  power: supply: ab8500: Sink current tables into charger code
  power: supply: ab8500: Standardize operating temperature
  power: supply: ab8500: Drop unused battery types
  power: supply: ab8500: Use only one battery type
  power: supply: ab8500: Standardize design capacity
  power: supply: ab8500: Standardize technology
  power: supply: ab8500: Standardize voltages
  power: supply: ab8500_fg: Init battery data in bind()
  power: supply: ab8500: Standardize internal resistance
  power: supply: ab8500: Standardize termination current
  power: supply: ab8500: Make recharge capacity a constant
  power: supply: ab8500: Standardize CC current
  power: supply: ab8500: Standardize CV voltage
  power: supply: ab8500: Standardize temp res lookup
  power: supply: ab8500: Standardize capacity lookup

 drivers/power/supply/ab8500-bm.h       | 123 ++----
 drivers/power/supply/ab8500-chargalg.h |   8 +-
 drivers/power/supply/ab8500_bmdata.c   | 572 ++++++-------------------
 drivers/power/supply/ab8500_btemp.c    |  61 ++-
 drivers/power/supply/ab8500_chargalg.c | 315 +++++++-------
 drivers/power/supply/ab8500_charger.c  | 536 ++++++++++++-----------
 drivers/power/supply/ab8500_fg.c       | 371 ++++++++--------
 7 files changed, 804 insertions(+), 1182 deletions(-)

Comments

Sebastian Reichel Nov. 22, 2021, 4:38 p.m. UTC | #1

Hi,

On Sat, Nov 20, 2021 at 04:53:10PM +0100, Linus Walleij wrote:
> This is the first set of patches starting to migrate the
> AB8500 charging code to use the struct power_supply_battery_info.
> We drop some cruft along the road.
> 
> This series does not add anything to the struct, just reuse
> what is already there. Adding new stuff comes in the next
> patch series.
> 
> The AB8500 charging code has not been in working condition for
> some time, but it is slowly getting there. Some of this is
> just regular maintenance.
> 
> ChangeLog v1->v2:
> - Fixed review comments on patch 1.

Thanks, queued.

-- Sebastian

> Linus Walleij (16):
>   power: supply: ab8500: Use core battery parser
>   power: supply: ab8500: Sink current tables into charger code
>   power: supply: ab8500: Standardize operating temperature
>   power: supply: ab8500: Drop unused battery types
>   power: supply: ab8500: Use only one battery type
>   power: supply: ab8500: Standardize design capacity
>   power: supply: ab8500: Standardize technology
>   power: supply: ab8500: Standardize voltages
>   power: supply: ab8500_fg: Init battery data in bind()
>   power: supply: ab8500: Standardize internal resistance
>   power: supply: ab8500: Standardize termination current
>   power: supply: ab8500: Make recharge capacity a constant
>   power: supply: ab8500: Standardize CC current
>   power: supply: ab8500: Standardize CV voltage
>   power: supply: ab8500: Standardize temp res lookup
>   power: supply: ab8500: Standardize capacity lookup
> 
>  drivers/power/supply/ab8500-bm.h       | 123 ++----
>  drivers/power/supply/ab8500-chargalg.h |   8 +-
>  drivers/power/supply/ab8500_bmdata.c   | 572 ++++++-------------------
>  drivers/power/supply/ab8500_btemp.c    |  61 ++-
>  drivers/power/supply/ab8500_chargalg.c | 315 +++++++-------
>  drivers/power/supply/ab8500_charger.c  | 536 ++++++++++++-----------
>  drivers/power/supply/ab8500_fg.c       | 371 ++++++++--------
>  7 files changed, 804 insertions(+), 1182 deletions(-)
> 
> -- 
> 2.31.1
>