RE: [PATCH 3/4] power: Add support for DA9150 Fuel-Gauge

From: Opensource [Adam Thomson]
Date: Mon Jun 22 2015 - 06:20:50 EST


On June 19, 2015 17:52, Paul Bolle wrote:

> On Thu, 2015-06-18 at 17:06 +0100, Adam Thomson wrote:
> > --- /dev/null
> > +++ b/drivers/power/da9150-fg.c
>
> > +/* Register external function to give battery temperature */
> > +void da9150_fg_register_temp_cb(struct power_supply *psy,
> da9150_read_temp_t cb,
> > + void *cb_context)
> > +{
> > + struct da9150_fg *fg = dev_get_drvdata(psy->dev.parent);
> > +
> > + fg->read_bat_temp = cb;
> > + fg->bat_temp_context = cb_context;
> > +}
> > +EXPORT_SYMBOL_GPL(da9150_fg_register_temp_cb);
>
> This series doesn't add a user of this export. Actually, it doesn't even
> add a local caller of this function. Is a patch that adds a user of this
> function queued somewhere?

The device relies on the battery providing temperature through a dedicated
pin, but that isn't always the case. I added this function as a means to handle
the scenarios where the battery being used didn't provide such a feature so
this allows for an alternative method of giving the battery temperature. At
present there is no patch lined up to use this.
N‹§²æ¸›yú²X¬¶ÇvØ–)Þ{.nlj·¥Š{±‘êX§>Wš±êÛxPjضŸÿÃ-»+ƒùdš_