Re: [PATCH] power: supply: max17042_battery: Add support for the TTE_NOW prop

From: Sebastian Reichel
Date: Wed Apr 29 2020 - 14:28:34 EST


Hi,

On Mon, Mar 30, 2020 at 01:33:55AM -0400, Geordan Neukum wrote:
> The max170{42,47,50,55} family of fuel gauges all provide time-to-empty
> estimation. As such, let's export this as a property.
>
> Signed-off-by: Geordan Neukum <gneukum1@xxxxxxxxx>
> ---

Thanks, queued.

-- Sebastian

> drivers/power/supply/max17042_battery.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/power/supply/max17042_battery.c b/drivers/power/supply/max17042_battery.c
> index 69ec4295d55d..f284547913d6 100644
> --- a/drivers/power/supply/max17042_battery.c
> +++ b/drivers/power/supply/max17042_battery.c
> @@ -87,6 +87,7 @@ static enum power_supply_property max17042_battery_props[] = {
> POWER_SUPPLY_PROP_SCOPE,
> POWER_SUPPLY_PROP_CURRENT_NOW,
> POWER_SUPPLY_PROP_CURRENT_AVG,
> + POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW,
> };
>
> static int max17042_get_temperature(struct max17042_chip *chip, int *temp)
> @@ -411,6 +412,13 @@ static int max17042_get_property(struct power_supply *psy,
> return -EINVAL;
> }
> break;
> + case POWER_SUPPLY_PROP_TIME_TO_EMPTY_NOW:
> + ret = regmap_read(map, MAX17042_TTE, &data);
> + if (ret < 0)
> + return ret;
> +
> + val->intval = data * 5625 / 1000;
> + break;
> default:
> return -EINVAL;
> }
> --
> 2.18.1
>

Attachment: signature.asc
Description: PGP signature