| Message ID | 20250215005244.1212285-11-superm1@kernel.org (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | amd-pstate cleanups | expand |
On Fri, Feb 14, 2025 at 06:52:37PM -0600, Mario Limonciello wrote: > From: Mario Limonciello <mario.limonciello@amd.com> > > The cpudata variable is only needed in the scope of the for loop. Move it > there. Makes sense. Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com> > > Signed-off-by: Mario Limonciello <mario.limonciello@amd.com> > --- > v2: > * new patch > > drivers/cpufreq/amd-pstate-ut.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/amd-pstate-ut.c b/drivers/cpufreq/amd-pstate-ut.c > index b888a5877ad93..9db20ac357042 100644 > --- a/drivers/cpufreq/amd-pstate-ut.c > +++ b/drivers/cpufreq/amd-pstate-ut.c > @@ -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_possible_cpu(cpu) { > struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL; > + struct amd_cpudata *cpudata; > > policy = cpufreq_cpu_get(cpu); > if (!policy) > -- > 2.43.0 >
diff --git a/drivers/cpufreq/amd-pstate-ut.c b/drivers/cpufreq/amd-pstate-ut.c index b888a5877ad93..9db20ac357042 100644 --- a/drivers/cpufreq/amd-pstate-ut.c +++ b/drivers/cpufreq/amd-pstate-ut.c @@ -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_possible_cpu(cpu) { struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL; + struct amd_cpudata *cpudata; policy = cpufreq_cpu_get(cpu); if (!policy)