On 31 July 2018 at 18:43, Angus Ainslie (Purism) <[email protected]> wrote: > The part has the capability of reading the current battery voltage.
Instead of "part": BQ25890-family of chargers? Chip? Anyway, Reviewed-by: Krzysztof Kozlowski <[email protected]> Best regards, Krzysztof > > Signed-off-by: Angus Ainslie (Purism) <[email protected]> > --- > drivers/power/supply/bq25890_charger.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/power/supply/bq25890_charger.c > b/drivers/power/supply/bq25890_charger.c > index 23b39da07e56..1aa7872ddeb0 100644 > --- a/drivers/power/supply/bq25890_charger.c > +++ b/drivers/power/supply/bq25890_charger.c > @@ -461,6 +461,15 @@ static int bq25890_power_supply_get_property(struct > power_supply *psy, > val->intval = bq25890_find_val(bq->init_data.iterm, > TBL_ITERM); > break; > > + case POWER_SUPPLY_PROP_VOLTAGE_NOW: > + ret = bq25890_field_read(bq, F_SYSV); /* read measured value > */ > + if (ret < 0) > + return ret; > + > + /* converted_val = 2.304V + ADC_val * 20mV (table 10.3.15) */ > + val->intval = 2304000 + ret * 20000; > + break; > + > default: > return -EINVAL; > } > @@ -669,6 +678,7 @@ static enum power_supply_property > bq25890_power_supply_props[] = { > POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE, > POWER_SUPPLY_PROP_CONSTANT_CHARGE_VOLTAGE_MAX, > POWER_SUPPLY_PROP_CHARGE_TERM_CURRENT, > + POWER_SUPPLY_PROP_VOLTAGE_NOW, > }; > > static char *bq25890_charger_supplied_to[] = { > -- > 2.17.1 >
