diff options
| author | Jacob Garber <jgarber1@ualberta.ca> | 2019-06-24 13:24:16 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2019-06-28 19:23:21 +0000 |
| commit | 70f6d8261427104d246847e4289d6bae07db851e (patch) | |
| tree | ab274a850736bccb221417aafa18bd899cce545a | |
| parent | 8a443b9aded5a2e78871484b80c09a94bf54708d (diff) | |
| download | coreboot-70f6d8261427104d246847e4289d6bae07db851e.tar.xz | |
cpu/amd/family_10h-family_15h: Remove dead assignment
Pstate_num is initialized later when it is used as a loop index, so this
duplicate assignment can be removed.
Change-Id: I71429bd3306139a823ed39e751d779e4d874f657
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Found-by: scan-build 8.0.0
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33745
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | src/cpu/amd/family_10h-family_15h/powernow_acpi.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cpu/amd/family_10h-family_15h/powernow_acpi.c b/src/cpu/amd/family_10h-family_15h/powernow_acpi.c index e648b71750..be936f66e3 100644 --- a/src/cpu/amd/family_10h-family_15h/powernow_acpi.c +++ b/src/cpu/amd/family_10h-family_15h/powernow_acpi.c @@ -259,8 +259,6 @@ void amd_generate_powernow(u32 pcontrol_blk, u8 plen, u8 onlyBSP) else if (mctGetLogicalCPUID(0) & AMD_FAM15_ALL) boost_count = (dtemp >> 2) & 0x7; - Pstate_num = 0; - /* See if the CPUID(0x80000007) returned EDX[7]==1b */ cpuid1 = cpuid(0x80000007); if ((cpuid1.edx & 0x80) != 0x80) { |