Commit 205cb215 authored by Dhananjay Ugwekar's avatar Dhananjay Ugwekar Committed by Mario Limonciello
Browse files

cpufreq/amd-pstate: Remove the redundant verify() function 



Merge the two verify() callback functions and rename the
cpufreq_policy_data argument for better readability.

Signed-off-by: default avatarDhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
Reviewed-by: default avatarMario Limonciello <mario.limonciello@amd.com>
Reviewed-by: default avatarGautham R. Shenoy <gautham.shenoy@amd.com>
Link: https://lore.kernel.org/r/20241017053927.25285-2-Dhananjay.Ugwekar@amd.com


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

drivers/cpufreq/amd-pstate.c

+4 −11
Original line number Diff line number Diff line
@@ -527,10 +527,10 @@ static void amd_pstate_update(struct amd_cpudata *cpudata, u32 min_perf, 
	cpufreq_cpu_put(policy);

}



static int amd_pstate_verify(struct cpufreq_policy_data *policy)

static int amd_pstate_verify(struct cpufreq_policy_data *policy_data)

{

	cpufreq_verify_within_cpu_limits(policy);



	cpufreq_verify_within_cpu_limits(policy_data);

	pr_debug("policy_max =%d, policy_min=%d\n", policy_data->max, policy_data->min);

	return 0;

}
@@ -1638,13 +1638,6 @@ static int amd_pstate_epp_cpu_offline(struct cpufreq_policy *policy) 
	return 0;

}



static int amd_pstate_epp_verify_policy(struct cpufreq_policy_data *policy)

{

	cpufreq_verify_within_cpu_limits(policy);

	pr_debug("policy_max =%d, policy_min=%d\n", policy->max, policy->min);

	return 0;

}



static int amd_pstate_epp_suspend(struct cpufreq_policy *policy)

{

	struct amd_cpudata *cpudata = policy->driver_data;
@@ -1700,7 +1693,7 @@ static struct cpufreq_driver amd_pstate_driver = { 


static struct cpufreq_driver amd_pstate_epp_driver = {

	.flags		= CPUFREQ_CONST_LOOPS,

	.verify		= amd_pstate_epp_verify_policy,

	.verify		= amd_pstate_verify,

	.setpolicy	= amd_pstate_epp_set_policy,

	.init		= amd_pstate_epp_cpu_init,

	.exit		= amd_pstate_epp_cpu_exit,