Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1383956 > unrolled thread

[PATCH 05/10] PM / OPP: Mark shared-opp for non-dt case

Started byViresh Kumar <viresh.kumar@linaro.org>
First post2016-04-21 11:00 +0200
Last post2016-04-23 00:30 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [PATCH 05/10] PM / OPP: Mark shared-opp for non-dt case Viresh Kumar <viresh.kumar@linaro.org> - 2016-04-21 11:00 +0200
    Re: [PATCH 05/10] PM / OPP: Mark shared-opp for non-dt case Stephen Boyd <sboyd@codeaurora.org> - 2016-04-23 00:30 +0200

#1383956 — [PATCH 05/10] PM / OPP: Mark shared-opp for non-dt case

FromViresh Kumar <viresh.kumar@linaro.org>
Date2016-04-21 11:00 +0200
Subject[PATCH 05/10] PM / OPP: Mark shared-opp for non-dt case
Message-ID<rqiCS-6mQ-11@gated-at.bofh.it>
opp core allows OPPs to be explicitly marked as shared from platform
code, in case of operating-point v1 bindings.

Though we do everything fine in that case, we don't set the flag in the
opp-table to indicate that the OPPs are shared. It works fine today as
the flag isn't used anywhere else in the core, but we should be doing
the right thing by marking it set.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
 drivers/base/power/opp/cpu.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/base/power/opp/cpu.c b/drivers/base/power/opp/cpu.c
index 491e8684bd5f..55cbf9bd8707 100644
--- a/drivers/base/power/opp/cpu.c
+++ b/drivers/base/power/opp/cpu.c
@@ -319,6 +319,9 @@ int dev_pm_opp_set_sharing_cpus(struct device *cpu_dev, cpumask_var_t cpumask)
 				__func__, cpu);
 			continue;
 		}
+
+		/* Mark opp-table as multiple CPUs are sharing it now */
+		opp_table->shared_opp = true;
 	}
 unlock:
 	mutex_unlock(&opp_table_lock);
-- 
2.7.1.410.g6faf27b

[toc] | [next] | [standalone]


#1385504

FromStephen Boyd <sboyd@codeaurora.org>
Date2016-04-23 00:30 +0200
Message-ID<rqRKi-Ux-3@gated-at.bofh.it>
In reply to#1383956
On 04/21, Viresh Kumar wrote:
> opp core allows OPPs to be explicitly marked as shared from platform
> code, in case of operating-point v1 bindings.
> 
> Though we do everything fine in that case, we don't set the flag in the
> opp-table to indicate that the OPPs are shared. It works fine today as
> the flag isn't used anywhere else in the core, but we should be doing
> the right thing by marking it set.
> 
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---

Reviewed-by: Stephen Boyd <sboyd@codeaurora.org>

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web