Hi Thomas,
On Mon, Apr 21, 2025 at 08:13:32PM +0200, Thomas Antoine via B4 Relay wrote:
> From: Thomas Antoine <t.antoine@xxxxxxxxxxxx>
>
> From the datasheet of the MAX17201/17205, the LSB should be
> "5.0μVh/RSENSE". The current computation sets it at 0.5mAh=5.0μVh/10mOhm
> which does not take into account the value of rsense which can be
> different from 10mOhm.
>
> Change the computation to fit the specs.
>
> Signed-off-by: Thomas Antoine <t.antoine@xxxxxxxxxxxx>
> ---
> drivers/power/supply/max1720x_battery.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/power/supply/max1720x_battery.c b/drivers/power/supply/max1720x_battery.c
> index ea3912fd1de8bfd0d029c16f276316d06e1b105c..cca5f8b5071fb731f9b60420239ea03d46cb1bf3 100644
> --- a/drivers/power/supply/max1720x_battery.c
> +++ b/drivers/power/supply/max1720x_battery.c
> @@ -288,9 +288,10 @@ static int max172xx_voltage_to_ps(unsigned int reg)
> return reg * 1250; /* in uV */
> }
>
> -static int max172xx_capacity_to_ps(unsigned int reg)
> +static int max172xx_capacity_to_ps(unsigned int reg,
> + struct max1720x_device_info *info)
> {
> - return reg * 500; /* in uAh */
> + return reg * (500000 / info->rsense); /* in uAh */
> }
>
> /*
>
> --
> 2.49.0
>
>
thanks for finding this.
Reviewed-by: Dimitri Fedrau <dimitri.fedrau@xxxxxxxxxxxx>
Best regards,
Dimitri Fedrau
