Commit 1421de79 authored by Nathan Fontenot's avatar Nathan Fontenot Committed by Shuah Khan
Browse files

cpupower: Remove unused pscur variable. 



The pscur variable is set but not uused, just remove it.

This may have previsously been set to validate the MSR_AMD_PSTATE_STATUS
MSR. With the addition of the CPUPOWER_CAP_AMD_HW_PSTATE cap flag this
is no longer needed since the cpuid bit to enable this cap flag also
validates that the MSR_AMD_PSTATE_STATUS MSR is present.

Signed-off-by: default avatarNathan Fontenot <nathan.fontenot@amd.com>
Reviewed-by: default avatarRobert Richter <rrichter@amd.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent a0255a76

tools/power/cpupower/utils/helpers/amd.c

+1 −8
Original line number Diff line number Diff line
@@ -90,7 +90,7 @@ static int get_cof(int family, union core_pstate pstate) 
int decode_pstates(unsigned int cpu, unsigned int cpu_family,

		   int boost_states, unsigned long *pstates, int *no)

{

	int i, psmax, pscur;

	int i, psmax;

	union core_pstate pstate;

	unsigned long long val;
@@ -104,13 +104,6 @@ int decode_pstates(unsigned int cpu, unsigned int cpu_family, 
		return -1;



	psmax = (val >> 4) & 0x7;



	if (read_msr(cpu, MSR_AMD_PSTATE_STATUS, &val))

		return -1;



	pscur = val & 0x7;



	pscur += boost_states;

	psmax += boost_states;

	for (i = 0; i <= psmax; i++) {

		if (i >= MAX_HW_PSTATES) {