Commit 50823972 authored by Mario Limonciello's avatar Mario Limonciello
Browse files

cpufreq/amd-pstate: Drop needless EPP initialization 



The EPP value doesn't need to be cached to the CPPC request in
amd_pstate_epp_update_limit() because it's passed as an argument
at the end to amd_pstate_set_epp() and stored at that time.

Reviewed-by: default avatarGautham R. Shenoy <gautham.shenoy@amd.com>
Reviewed-by: default avatarPerry Yuan <perry.yuan@amd.com>
Tested-by: default avatarDhananjay Ugwekar <dhananjay.ugwekar@amd.com>
Link: https://lore.kernel.org/r/20241012174519.897-4-mario.limonciello@amd.com


Signed-off-by: default avatarMario Limonciello <mario.limonciello@amd.com>
parent 047a2d0c

drivers/cpufreq/amd-pstate.c

+0 −6
Original line number Diff line number Diff line
@@ -1528,12 +1528,6 @@ static int amd_pstate_epp_update_limit(struct cpufreq_policy *policy) 
	if (cpudata->policy == CPUFREQ_POLICY_PERFORMANCE)

		epp = 0;



	/* Set initial EPP value */

	if (cpu_feature_enabled(X86_FEATURE_CPPC)) {

		value &= ~GENMASK_ULL(31, 24);

		value |= (u64)epp << 24;

	}



	WRITE_ONCE(cpudata->cppc_req_cached, value);

	return amd_pstate_set_epp(cpudata, epp);

}