Skip to content

Commit e8a0e30

Browse files
committed
cpufreq: intel_pstate: Drop ACPI _PSS states table patching
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 <[email protected]>
1 parent 99387b0 commit e8a0e30

File tree

1 file changed

+0
-14
lines changed

1 file changed

+0
-14
lines changed

drivers/cpufreq/intel_pstate.c

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -452,20 +452,6 @@ static void intel_pstate_init_acpi_perf_limits(struct cpufreq_policy *policy)
452452
(u32) cpu->acpi_perf_data.states[i].control);
453453
}
454454

455-
/*
456-
* The _PSS table doesn't contain whole turbo frequency range.
457-
* This just contains +1 MHZ above the max non turbo frequency,
458-
* with control value corresponding to max turbo ratio. But
459-
* when cpufreq set policy is called, it will call with this
460-
* max frequency, which will cause a reduced performance as
461-
* this driver uses real max turbo frequency as the max
462-
* frequency. So correct this frequency in _PSS table to
463-
* correct max turbo frequency based on the turbo state.
464-
* Also need to convert to MHz as _PSS freq is in MHz.
465-
*/
466-
if (!global.turbo_disabled)
467-
cpu->acpi_perf_data.states[0].core_frequency =
468-
policy->cpuinfo.max_freq / 1000;
469455
cpu->valid_pss_table = true;
470456
pr_debug("_PPC limits will be enforced\n");
471457

0 commit comments

Comments
 (0)