cpufreq: intel_pstate: Drop ACPI _PSS states table patching
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 28 Dec 2022 21:26:04 +0000 (22:26 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 30 Dec 2022 18:10:02 +0000 (19:10 +0100)
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>
drivers/cpufreq/intel_pstate.c

index fd73d6d2b808482a08ee83da2039897ee12e71e0..cb4beec27555a0bdef8f1ddd49e9361ab8a79295 100644 (file)
@@ -452,20 +452,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");