[coreboot-gerrit] New patch to review for coreboot: f08cf42 cpu/amd/model_fxx/powernow_acpi.c: Remove set but unused variable `Start_vid`
Paul Menzel (paulepanter@users.sourceforge.net)
gerrit at coreboot.org
Sun Nov 10 23:52:17 CET 2013
Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4036
-gerrit
commit f08cf42e5476a62529d98c543c5709c7e0d21a36
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Tue Oct 29 20:41:50 2013 +0100
cpu/amd/model_fxx/powernow_acpi.c: Remove set but unused variable `Start_vid`
Removing `-Wno-unused-but-set-variable` from `CFLAGS` the build for the
ASRock 939A785GMH fails with the following error.
CC cpu/amd/model_fxx/powernow_acpi.ramstage.o
src/cpu/amd/model_fxx/powernow_acpi.c: In function 'pstates_algorithm':
src/cpu/amd/model_fxx/powernow_acpi.c:761:25: error: variable 'Start_vid' set but not used [-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors
When adding support for PSS object generation for AMD pre Family Fh CPUs
(199c694f) the function `pstates_algorithm` was copied and adapted, but
`Start_vid` is not needed anymore as a static table is used. So remove
the variable `Start_vid`.
Change-Id: I3002951d168cade6461941c16d78373c47792e13
CC: Rudolf Marek <r.marek at assembler.cz>
CC: Jonathan Kollasch <jakllsch at kollasch.net>
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
src/cpu/amd/model_fxx/powernow_acpi.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/cpu/amd/model_fxx/powernow_acpi.c b/src/cpu/amd/model_fxx/powernow_acpi.c
index af1e24b..eca736c 100644
--- a/src/cpu/amd/model_fxx/powernow_acpi.c
+++ b/src/cpu/amd/model_fxx/powernow_acpi.c
@@ -758,7 +758,7 @@ static int pstates_algorithm(u32 pcontrol_blk, u8 plen, u8 onlyBSP)
u16 Pstate_feq[MAXP+1];
u8 Pstate_vid[MAXP+1];
u32 Pstate_power[MAXP+1];
- u8 Max_fid, Start_fid, Start_vid, Max_vid;
+ u8 Max_fid, Start_fid, Max_vid;
struct cpuid_result cpuid1;
/* See if the CPUID(0x80000007) returned EDX[2:1]==11b */
@@ -777,7 +777,6 @@ static int pstates_algorithm(u32 pcontrol_blk, u8 plen, u8 onlyBSP)
Max_fid = (msr.lo & 0x3F0000) >> 16;
Max_vid = (msr.hi & 0x3F0000) >> 16;
Start_fid = (msr.lo & 0x3F00) >> 8;
- Start_vid = (msr.hi & 0x3F00) >> 8;
cmp_cap =
(pci_read_config16(dev_find_slot(0, PCI_DEVFN(0x18, 3)), 0xE8) &
More information about the coreboot-gerrit
mailing list