summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2017-03-28 00:11:53 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2017-03-28 23:12:14 +0200
commit694cb173475a048a05daebf27cc8fdb7865c158b (patch)
tree8c447a2f29340a0da3bd9149ea80f0c3dde6b40b
parentcpufreq: intel_pstate: Set HWP sampling interval once (diff)
downloadlinux-694cb173475a048a05daebf27cc8fdb7865c158b.tar.xz
linux-694cb173475a048a05daebf27cc8fdb7865c158b.zip
cpufreq: intel_pstate: Skip unnecessary PID resets on init
PID controller parameters only need to be initialized if the get_target_pstate_use_performance() P-state selection routine is going to be used. It is not necessary to initialize them otherwise, so don't do that. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/cpufreq/intel_pstate.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index b631ab02f170..ee61db93163c 100644
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -1911,12 +1911,12 @@ static int intel_pstate_init_cpu(unsigned int cpunum)
intel_pstate_disable_ee(cpunum);
intel_pstate_hwp_enable(cpu);
+ } else if (pstate_funcs.get_target_pstate == get_target_pstate_use_performance) {
+ intel_pstate_pid_reset(cpu);
}
intel_pstate_get_cpu_pstates(cpu);
- intel_pstate_pid_reset(cpu);
-
pr_debug("controlling: cpu %d\n", cpunum);
return 0;