]> Pileus Git - ~andy/linux/commitdiff
intel_pstate: Correct calculation of min pstate value
authorDirk Brandewie <dirk.j.brandewie@intel.com>
Mon, 21 Oct 2013 16:20:33 +0000 (09:20 -0700)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 21 Oct 2013 23:16:39 +0000 (01:16 +0200)
The minimum pstate is supposed to be a percentage of the maximum P
state available.  Calculate min using max pstate and not the
current max which may have been limited by the user

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

index 8b8677f927004dc4cd4a20b7a65f060a83e0bf0a..eb3fdc755000e0005f43474bb5ae967dd6ab8aa8 100644 (file)
@@ -366,12 +366,13 @@ static int intel_pstate_turbo_pstate(void)
 static void intel_pstate_get_min_max(struct cpudata *cpu, int *min, int *max)
 {
        int max_perf = cpu->pstate.turbo_pstate;
+       int max_perf_adj;
        int min_perf;
        if (limits.no_turbo)
                max_perf = cpu->pstate.max_pstate;
 
-       max_perf = fp_toint(mul_fp(int_tofp(max_perf), limits.max_perf));
-       *max = clamp_t(int, max_perf,
+       max_perf_adj = fp_toint(mul_fp(int_tofp(max_perf), limits.max_perf));
+       *max = clamp_t(int, max_perf_adj,
                        cpu->pstate.min_pstate, cpu->pstate.turbo_pstate);
 
        min_perf = fp_toint(mul_fp(int_tofp(max_perf), limits.min_perf));