diff mbox series

[v2,4/5] OPP: Update the bandwidth on OPP frequency changes

Message ID 20190423132823.7915-5-georgi.djakov@linaro.org
State New
Headers show
Series None | expand

Commit Message

Georgi Djakov April 23, 2019, 1:28 p.m. UTC
If the OPP bandwidth values are populated, we want to switch also the
interconnect bandwidth in addition to frequency and voltage.

Signed-off-by: Georgi Djakov <georgi.djakov@linaro.org>

---
 drivers/opp/core.c | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

Comments

Viresh Kumar April 24, 2019, 5:55 a.m. UTC | #1
On 23-04-19, 16:28, Georgi Djakov wrote:
> If the OPP bandwidth values are populated, we want to switch also the

> interconnect bandwidth in addition to frequency and voltage.

> 

> Signed-off-by: Georgi Djakov <georgi.djakov@linaro.org>

> ---

>  drivers/opp/core.c | 9 ++++++++-

>  1 file changed, 8 insertions(+), 1 deletion(-)

> 

> diff --git a/drivers/opp/core.c b/drivers/opp/core.c

> index 97ee39ecdebd..91d1c2abfb3e 100644

> --- a/drivers/opp/core.c

> +++ b/drivers/opp/core.c

> @@ -707,7 +707,7 @@ int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq)

>  	unsigned long freq, old_freq;

>  	struct dev_pm_opp *old_opp, *opp;

>  	struct clk *clk;

> -	int ret;

> +	int ret, i;

>  

>  	if (unlikely(!target_freq)) {

>  		dev_err(dev, "%s: Invalid target frequency %lu\n", __func__,

> @@ -780,6 +780,13 @@ int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq)

>  		ret = _generic_set_opp_clk_only(dev, clk, freq);

>  	}

>  

> +	if (!ret && !IS_ERR_OR_NULL(opp_table->paths)) {


Can paths ever have a error value ? I believe only checking for NULL
is sufficient ?

> +		for (i = 0; i < opp_table->path_count; i++) {

> +			icc_set_bw(opp_table->paths[i], opp->bandwidth[i].avg,

> +				   opp->bandwidth[i].peak);

> +		}

> +	}

> +


I will set the path after required_opps are set.

>  	/* Scaling down? Configure required OPPs after frequency */

>  	if (!ret && freq < old_freq) {

>  		ret = _set_required_opps(dev, opp_table, opp);


-- 
viresh
diff mbox series

Patch

diff --git a/drivers/opp/core.c b/drivers/opp/core.c
index 97ee39ecdebd..91d1c2abfb3e 100644
--- a/drivers/opp/core.c
+++ b/drivers/opp/core.c
@@ -707,7 +707,7 @@  int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq)
 	unsigned long freq, old_freq;
 	struct dev_pm_opp *old_opp, *opp;
 	struct clk *clk;
-	int ret;
+	int ret, i;
 
 	if (unlikely(!target_freq)) {
 		dev_err(dev, "%s: Invalid target frequency %lu\n", __func__,
@@ -780,6 +780,13 @@  int dev_pm_opp_set_rate(struct device *dev, unsigned long target_freq)
 		ret = _generic_set_opp_clk_only(dev, clk, freq);
 	}
 
+	if (!ret && !IS_ERR_OR_NULL(opp_table->paths)) {
+		for (i = 0; i < opp_table->path_count; i++) {
+			icc_set_bw(opp_table->paths[i], opp->bandwidth[i].avg,
+				   opp->bandwidth[i].peak);
+		}
+	}
+
 	/* Scaling down? Configure required OPPs after frequency */
 	if (!ret && freq < old_freq) {
 		ret = _set_required_opps(dev, opp_table, opp);