cpufreq: intel_pstate: Drop ACPI _PSS states table patching

commit e8a0e30b742f76ebd0f3b196973df4bf65d8fbbb upstream.

After making acpi_processor_get_platform_limit() use the "no limit"
value for its frequency QoS request when _PPC returns 0, it is not
necessary to replace the frequency corresponding to the first _PSS
return package entry with the maximum turbo frequency of the given
CPU in intel_pstate_init_acpi_perf_limits() any more, so drop the
code doing that along with the comment explaining it.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Tested-by: Hagar Hemdan <hagarhem@amazon.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c
index 2f5063db2f1fc6a366c6b07b56c7d60b08915e49..736cb2cfcbb082226c8d3eee2c4fa5fe8b4340bd 100644 (file)
--- a/drivers/cpufreq/intel_pstate.c
+++ b/drivers/cpufreq/intel_pstate.c
@@ -448,20 +448,6 @@ static void intel_pstate_init_acpi_perf_limits(struct cpufreq_policy *policy)
                         (u32) cpu->acpi_perf_data.states[i].control);
        }
 
-       /*
-        * The _PSS table doesn't contain whole turbo frequency range.
-        * This just contains +1 MHZ above the max non turbo frequency,
-        * with control value corresponding to max turbo ratio. But
-        * when cpufreq set policy is called, it will call with this
-        * max frequency, which will cause a reduced performance as
-        * this driver uses real max turbo frequency as the max
-        * frequency. So correct this frequency in _PSS table to
-        * correct max turbo frequency based on the turbo state.
-        * Also need to convert to MHz as _PSS freq is in MHz.
-        */
-       if (!global.turbo_disabled)
-               cpu->acpi_perf_data.states[0].core_frequency =
-                                       policy->cpuinfo.max_freq / 1000;
        cpu->valid_pss_table = true;
        pr_debug("_PPC limits will be enforced\n");