]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
cpufreq/amd-pstate: Remove the redundant verify() function
authorDhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
Thu, 17 Oct 2024 05:39:28 +0000 (05:39 +0000)
committerMario Limonciello <mario.limonciello@amd.com>
Mon, 28 Oct 2024 19:54:35 +0000 (14:54 -0500)
Merge the two verify() callback functions and rename the
cpufreq_policy_data argument for better readability.

Signed-off-by: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>
Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>
Link: https://lore.kernel.org/r/20241017053927.25285-2-Dhananjay.Ugwekar@amd.com
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
drivers/cpufreq/amd-pstate.c

index 90868c8b214ec380ea27883ab1ad1f276e6ffc1e..ee2decad10afa5924725727b382df22b0e60942f 100644 (file)
@@ -527,10 +527,10 @@ cpufreq_policy_put:
        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,