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��������+%������w��{.n����z�{�