Message ID | 1628645918-12890-1-git-send-email-tdas@codeaurora.org |
---|---|
State | New |
Headers | show |
Series | [v1] cpufreq: qcom-hw: Set dvfs_possible_from_any_cpu cpufreq driver flag | expand |
On 11-08-21, 07:08, Taniya Das wrote: > As remote cpufreq updates are supported on QCOM platforms, set > dvfs_possible_from_any_cpu cpufreq driver flag. > > Signed-off-by: Taniya Das <tdas@codeaurora.org> > --- > drivers/cpufreq/qcom-cpufreq-hw.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/qcom-cpufreq-hw.c b/drivers/cpufreq/qcom-cpufreq-hw.c > index f86859b..20455aa 100644 > --- a/drivers/cpufreq/qcom-cpufreq-hw.c > +++ b/drivers/cpufreq/qcom-cpufreq-hw.c > @@ -161,6 +161,7 @@ static int qcom_cpufreq_hw_read_lut(struct device *cpu_dev, > return ret; > } else { > policy->fast_switch_possible = true; > + policy->dvfs_possible_from_any_cpu = true; > icc_scaling_enabled = false; > } Why don't you set it always ? > > -- > Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member > of the Code Aurora Forum, hosted by the Linux Foundation.
Thanks Viresh for the review. On 8/11/2021 9:27 AM, Viresh Kumar wrote: > On 11-08-21, 07:08, Taniya Das wrote: >> As remote cpufreq updates are supported on QCOM platforms, set >> dvfs_possible_from_any_cpu cpufreq driver flag. >> >> Signed-off-by: Taniya Das <tdas@codeaurora.org> >> --- >> drivers/cpufreq/qcom-cpufreq-hw.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/cpufreq/qcom-cpufreq-hw.c b/drivers/cpufreq/qcom-cpufreq-hw.c >> index f86859b..20455aa 100644 >> --- a/drivers/cpufreq/qcom-cpufreq-hw.c >> +++ b/drivers/cpufreq/qcom-cpufreq-hw.c >> @@ -161,6 +161,7 @@ static int qcom_cpufreq_hw_read_lut(struct device *cpu_dev, >> return ret; >> } else { >> policy->fast_switch_possible = true; >> + policy->dvfs_possible_from_any_cpu = true; >> icc_scaling_enabled = false; >> } > > Why don't you set it always ? > Sure, will move it outside. >> >> -- >> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member >> of the Code Aurora Forum, hosted by the Linux Foundation. > -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation. --
diff --git a/drivers/cpufreq/qcom-cpufreq-hw.c b/drivers/cpufreq/qcom-cpufreq-hw.c index f86859b..20455aa 100644 --- a/drivers/cpufreq/qcom-cpufreq-hw.c +++ b/drivers/cpufreq/qcom-cpufreq-hw.c @@ -161,6 +161,7 @@ static int qcom_cpufreq_hw_read_lut(struct device *cpu_dev, return ret; } else { policy->fast_switch_possible = true; + policy->dvfs_possible_from_any_cpu = true; icc_scaling_enabled = false; }
As remote cpufreq updates are supported on QCOM platforms, set dvfs_possible_from_any_cpu cpufreq driver flag. Signed-off-by: Taniya Das <tdas@codeaurora.org> --- drivers/cpufreq/qcom-cpufreq-hw.c | 1 + 1 file changed, 1 insertion(+) -- Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member of the Code Aurora Forum, hosted by the Linux Foundation.