cpufreq: intel_pstate: Eliminate intel_pstate_get_base_pstate()
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 15 Feb 2019 12:16:40 +0000 (13:16 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 18 Feb 2019 10:34:32 +0000 (11:34 +0100)
There is only one caller of intel_pstate_get_base_pstate() and it is
more straightforward to carry out the computation directly in the
caller, so do that and drop intel_pstate_get_base_pstate().

No intentional changes of behavior.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/intel_pstate.c

index 3a4d15a48c1f489aeb836d5a519392a0555879ab..9a6cb3a1be50bbf50e6e66572e9a86c55248d670 100644 (file)
@@ -1445,12 +1445,6 @@ static int knl_get_turbo_pstate(void)
        return ret;
 }
 
-static int intel_pstate_get_base_pstate(struct cpudata *cpu)
-{
-       return global.no_turbo || global.turbo_disabled ?
-                       cpu->pstate.max_pstate : cpu->pstate.turbo_pstate;
-}
-
 static void intel_pstate_set_pstate(struct cpudata *cpu, int pstate)
 {
        trace_cpu_frequency(pstate * cpu->pstate.scaling, cpu->cpu);
@@ -1973,7 +1967,8 @@ static void intel_pstate_update_perf_limits(struct cpufreq_policy *policy,
        if (hwp_active) {
                intel_pstate_get_hwp_max(cpu->cpu, &turbo_max, &max_state);
        } else {
-               max_state = intel_pstate_get_base_pstate(cpu);
+               max_state = global.no_turbo || global.turbo_disabled ?
+                       cpu->pstate.max_pstate : cpu->pstate.turbo_pstate;
                turbo_max = cpu->pstate.turbo_pstate;
        }