| Message ID | c4d59ad206394e4f73ab7547c6cd99d95c032128.1598442485.git.viresh.kumar@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | viresh kumar |
| Headers | show |
| Series | opp: Allow opp-supported-hw to contain multiple versions | expand |
On Wed, Aug 26, 2020 at 05:20:29PM +0530, Viresh Kumar wrote: > The bindings allow multiple versions to be passed to "opp-supported-hw" > property, either of which can result in enabling of the OPP. > > Update code to allow that. > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> This is exactly what I was looking for! And it works just fine in my tests: Tested-by: Stephan Gerhold <stephan@gerhold.net> Thank you! Stephan > --- > drivers/opp/of.c | 47 +++++++++++++++++++++++++++++++++-------------- > 1 file changed, 33 insertions(+), 14 deletions(-) > > diff --git a/drivers/opp/of.c b/drivers/opp/of.c > index e39ddcc779af..5dac8bffd68c 100644 > --- a/drivers/opp/of.c > +++ b/drivers/opp/of.c > @@ -434,9 +434,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_of_find_icc_paths); > static bool _opp_is_supported(struct device *dev, struct opp_table *opp_table, > struct device_node *np) > { > - unsigned int count = opp_table->supported_hw_count; > - u32 version; > - int ret; > + unsigned int levels = opp_table->supported_hw_count; > + int count, versions, ret, i, j; > + u32 val; > > if (!opp_table->supported_hw) { > /* > @@ -451,21 +451,40 @@ static bool _opp_is_supported(struct device *dev, struct opp_table *opp_table, > return true; > } > > - while (count--) { > - ret = of_property_read_u32_index(np, "opp-supported-hw", count, > - &version); > - if (ret) { > - dev_warn(dev, "%s: failed to read opp-supported-hw property at index %d: %d\n", > - __func__, count, ret); > - return false; > + count = of_property_count_u32_elems(np, "opp-supported-hw"); > + if (count <= 0 || count % levels) { > + dev_err(dev, "%s: Invalid opp-supported-hw property (%d)\n", > + __func__, count); > + return false; > + } > + > + versions = count / levels; > + > + /* All levels in at least one of the versions should match */ > + for (i = 0; i < versions; i++) { > + bool supported = true; > + > + for (j = 0; j < levels; j++) { > + ret = of_property_read_u32_index(np, "opp-supported-hw", > + i * levels + j, &val); > + if (ret) { > + dev_warn(dev, "%s: failed to read opp-supported-hw property at index %d: %d\n", > + __func__, i * levels + j, ret); > + return false; > + } > + > + /* Check if the level is supported */ > + if (!(val & opp_table->supported_hw[j])) { > + supported = false; > + break; > + } > } > > - /* Both of these are bitwise masks of the versions */ > - if (!(version & opp_table->supported_hw[count])) > - return false; > + if (supported) > + return true; > } > > - return true; > + return false; > } > > static int opp_parse_supplies(struct dev_pm_opp *opp, struct device *dev, > -- > 2.25.0.rc1.19.g042ed3e048af >
26.08.2020 14:50, Viresh Kumar пишет: > The bindings allow multiple versions to be passed to "opp-supported-hw" > property, either of which can result in enabling of the OPP. > > Update code to allow that. > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > --- > drivers/opp/of.c | 47 +++++++++++++++++++++++++++++++++-------------- > 1 file changed, 33 insertions(+), 14 deletions(-) I checked that CPU DVFS works on Nexus 7 after applying this series just like it worked before. Thanks! Tested-by: Dmitry Osipenko <digetx@gmail.com>
diff --git a/drivers/opp/of.c b/drivers/opp/of.c index e39ddcc779af..5dac8bffd68c 100644 --- a/drivers/opp/of.c +++ b/drivers/opp/of.c @@ -434,9 +434,9 @@ EXPORT_SYMBOL_GPL(dev_pm_opp_of_find_icc_paths); static bool _opp_is_supported(struct device *dev, struct opp_table *opp_table, struct device_node *np) { - unsigned int count = opp_table->supported_hw_count; - u32 version; - int ret; + unsigned int levels = opp_table->supported_hw_count; + int count, versions, ret, i, j; + u32 val; if (!opp_table->supported_hw) { /* @@ -451,21 +451,40 @@ static bool _opp_is_supported(struct device *dev, struct opp_table *opp_table, return true; } - while (count--) { - ret = of_property_read_u32_index(np, "opp-supported-hw", count, - &version); - if (ret) { - dev_warn(dev, "%s: failed to read opp-supported-hw property at index %d: %d\n", - __func__, count, ret); - return false; + count = of_property_count_u32_elems(np, "opp-supported-hw"); + if (count <= 0 || count % levels) { + dev_err(dev, "%s: Invalid opp-supported-hw property (%d)\n", + __func__, count); + return false; + } + + versions = count / levels; + + /* All levels in at least one of the versions should match */ + for (i = 0; i < versions; i++) { + bool supported = true; + + for (j = 0; j < levels; j++) { + ret = of_property_read_u32_index(np, "opp-supported-hw", + i * levels + j, &val); + if (ret) { + dev_warn(dev, "%s: failed to read opp-supported-hw property at index %d: %d\n", + __func__, i * levels + j, ret); + return false; + } + + /* Check if the level is supported */ + if (!(val & opp_table->supported_hw[j])) { + supported = false; + break; + } } - /* Both of these are bitwise masks of the versions */ - if (!(version & opp_table->supported_hw[count])) - return false; + if (supported) + return true; } - return true; + return false; } static int opp_parse_supplies(struct dev_pm_opp *opp, struct device *dev,
The bindings allow multiple versions to be passed to "opp-supported-hw" property, either of which can result in enabling of the OPP. Update code to allow that. Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- drivers/opp/of.c | 47 +++++++++++++++++++++++++++++++++-------------- 1 file changed, 33 insertions(+), 14 deletions(-)