]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
cpufreq/amd-pstate-ut: Adjust variable scope
authorMario Limonciello <mario.limonciello@amd.com>
Fri, 14 Feb 2025 22:34:23 +0000 (16:34 -0600)
committerMario Limonciello <mario.limonciello@amd.com>
Thu, 6 Mar 2025 19:01:25 +0000 (13:01 -0600)
In amd_pstate_ut_check_freq() and amd_pstate_ut_check_perf() the cpudata
variable is only needed in the scope of the for loop. Move it there.

Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>
Reviewed-by: Dhananjay Ugwekar <dhananjay.ugwekar@amd.com>
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
drivers/cpufreq/amd-pstate-ut.c

index 2ab3017d7a0bbd7f05c8caaed4d1507b6916a1c3..edc1475989e3da2853c632733452485cbf8027d0 100644 (file)
@@ -113,11 +113,11 @@ static int amd_pstate_ut_check_perf(u32 index)
        u32 highest_perf = 0, nominal_perf = 0, lowest_nonlinear_perf = 0, lowest_perf = 0;
        u64 cap1 = 0;
        struct cppc_perf_caps cppc_perf;
-       struct amd_cpudata *cpudata = NULL;
        union perf_cached cur_perf;
 
        for_each_online_cpu(cpu) {
                struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL;
+               struct amd_cpudata *cpudata;
 
                policy = cpufreq_cpu_get(cpu);
                if (!policy)
@@ -186,10 +186,10 @@ static int amd_pstate_ut_check_perf(u32 index)
 static int amd_pstate_ut_check_freq(u32 index)
 {
        int cpu = 0;
-       struct amd_cpudata *cpudata = NULL;
 
        for_each_online_cpu(cpu) {
                struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL;
+               struct amd_cpudata *cpudata;
 
                policy = cpufreq_cpu_get(cpu);
                if (!policy)