On 04-12-18, 14:59, Nick Fan wrote: > Add API to get voltage for multiple supplies from opp table
And who needs to use this new API ? It would be better to add the user in the same series to make sure this really gets used. > Signed-off-by: Nick Fan <[email protected]> > --- > drivers/opp/core.c | 28 ++++++++++++++++++++++++++++ > include/linux/pm_opp.h | 3 +++ > 2 files changed, 31 insertions(+) > > diff --git a/drivers/opp/core.c b/drivers/opp/core.c > index 2c2df4e..ee73546 100644 > --- a/drivers/opp/core.c > +++ b/drivers/opp/core.c > @@ -113,6 +113,34 @@ unsigned long dev_pm_opp_get_voltage(struct dev_pm_opp > *opp) > EXPORT_SYMBOL_GPL(dev_pm_opp_get_voltage); > > /** > + * dev_pm_opp_get_voltage_supply() - Gets the voltage corresponding to an opp > + * with index > + * @opp: opp for which voltage has to be returned for > + * @index: index to specify the returned supplies > + * > + * Return: voltage in micro volt corresponding to the opp with index, else > + * return 0 > + * > + * This is useful for devices with multiple power supplies. > + */ > +unsigned long dev_pm_opp_get_voltage_supply(struct dev_pm_opp *opp, > + unsigned int index) How will the users of this API get the index ? > +{ > + if (IS_ERR_OR_NULL(opp)) { > + pr_err("%s: Invalid parameters\n", __func__); > + return 0; > + } > + > + if (index >= opp->opp_table->regulator_count) { > + pr_err("%s: Invalid supply index: %u\n", __func__, index); > + return 0; > + } > + > + return opp->supplies[index].u_volt; -- viresh
