mbox series

[v3,0/5] Improve the SC27XX fuel gauge controller

Message ID cover.1575863274.git.baolin.wang7@gmail.com
Headers show
Series Improve the SC27XX fuel gauge controller | expand

Message

Baolin Wang Dec. 9, 2019, 3:56 a.m. UTC
Hi,

This patch set adds one battery resistance-temperature table to optimize
the real battery internal resistance in different tempertures, and
calibrates the resistance of coulomb counter to improve the accuracy
of the coulomb counter.

Any comments are welcome. Thanks.

Changes from v2:
 - Add reviewed tag from Rob.
 - Rebased.

Changes from v1:
 - Fix the order of values in resistance-temp-table property's description.
 - Add an unit suffix for FGU resistance property.

Baolin Wang (4):
  dt-bindings: power: Introduce one property to describe the battery
    resistance with temperature changes
  power: supply: core: Add battery internal resistance temperature
    table support
  dt-bindings: power: sc27xx: Add a new property to describe the real
    resistance of coulomb counter chip
  power: supply: sc27xx: Calibrate the resistance of coulomb counter

Yuanjiang Yu (1):
  power: supply: sc27xx: Optimize the battery resistance with measuring
    temperature

 .../devicetree/bindings/power/supply/battery.txt   |    5 ++
 .../devicetree/bindings/power/supply/sc27xx-fg.txt |    3 +
 drivers/power/supply/power_supply_core.c           |   67 +++++++++++++++++++-
 drivers/power/supply/sc27xx_fuel_gauge.c           |   49 +++++++++++++-
 include/linux/power_supply.h                       |   10 +++
 5 files changed, 130 insertions(+), 4 deletions(-)

Comments

Sebastian Reichel Dec. 19, 2019, 12:54 a.m. UTC | #1
Hi,

On Mon, Dec 09, 2019 at 11:56:20AM +0800, Baolin Wang wrote:
> This patch set adds one battery resistance-temperature table to optimize
> the real battery internal resistance in different tempertures, and
> calibrates the resistance of coulomb counter to improve the accuracy
> of the coulomb counter.
> 
> Any comments are welcome. Thanks.

Thanks, queued to power-supply's for-next branch. I changed the
comment for struct sc27xx_fgu_data, so that it states calib_resist
being in uOhm instead of mOhm.

-- Sebastian

> Changes from v2:
>  - Add reviewed tag from Rob.
>  - Rebased.
> 
> Changes from v1:
>  - Fix the order of values in resistance-temp-table property's description.
>  - Add an unit suffix for FGU resistance property.
> 
> Baolin Wang (4):
>   dt-bindings: power: Introduce one property to describe the battery
>     resistance with temperature changes
>   power: supply: core: Add battery internal resistance temperature
>     table support
>   dt-bindings: power: sc27xx: Add a new property to describe the real
>     resistance of coulomb counter chip
>   power: supply: sc27xx: Calibrate the resistance of coulomb counter
> 
> Yuanjiang Yu (1):
>   power: supply: sc27xx: Optimize the battery resistance with measuring
>     temperature
> 
>  .../devicetree/bindings/power/supply/battery.txt   |    5 ++
>  .../devicetree/bindings/power/supply/sc27xx-fg.txt |    3 +
>  drivers/power/supply/power_supply_core.c           |   67 +++++++++++++++++++-
>  drivers/power/supply/sc27xx_fuel_gauge.c           |   49 +++++++++++++-
>  include/linux/power_supply.h                       |   10 +++
>  5 files changed, 130 insertions(+), 4 deletions(-)
> 
> -- 
> 1.7.9.5
>
Baolin Wang Dec. 19, 2019, 1:34 a.m. UTC | #2
Hi Sebastian,

On Thu, Dec 19, 2019 at 8:54 AM Sebastian Reichel
<sebastian.reichel@collabora.com> wrote:
>
> Hi,
>
> On Mon, Dec 09, 2019 at 11:56:20AM +0800, Baolin Wang wrote:
> > This patch set adds one battery resistance-temperature table to optimize
> > the real battery internal resistance in different tempertures, and
> > calibrates the resistance of coulomb counter to improve the accuracy
> > of the coulomb counter.
> >
> > Any comments are welcome. Thanks.
>
> Thanks, queued to power-supply's for-next branch. I changed the
> comment for struct sc27xx_fgu_data, so that it states calib_resist
> being in uOhm instead of mOhm.

OK. Thanks.