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

cpufreq/amd-pstate-ut: Adjust variable scope 



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: default avatarGautham R. Shenoy <gautham.shenoy@amd.com>
Reviewed-by: default avatarDhananjay Ugwekar <dhananjay.ugwekar@amd.com>
Signed-off-by: default avatarMario Limonciello <mario.limonciello@amd.com>
parent 2aac38ac

drivers/cpufreq/amd-pstate-ut.c

+2 −2
Original line number Diff line number Diff line
@@ -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)