Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c8b1499e authored by Rafael J. Wysocki's avatar Rafael J. Wysocki Committed by Greg Kroah-Hartman
Browse files

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: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tested-by: default avatarHagar Hemdan <hagarhem@amazon.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent f331413e

drivers/cpufreq/intel_pstate.c

+0 −14
Original line number Diff line number Diff line
@@ -442,20 +442,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");