GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

sb/amd/amd8111/acpi.c: Remove set but unused variable `dword` 

Removing `-Wno-unused-but-set-variable` from `CFLAGS` results in the error
below, when building for example the HP DL145 GL1.

	    CC         southbridge/amd/amd8111/acpi.ramstage.o
	src/southbridge/amd/amd8111/acpi.c: In function 'acpi_init':
	src/southbridge/amd/amd8111/acpi.c💯11: error: variable 'dword' set but not used [-Werror=unused-but-set-variable]

Removing the variable `dword` fixes this error.

The read is left in the code, as I do not know if it has an effect or
not.

Change-Id: I9957cef3a996c5974c275423c9de63ccf230974e
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/5315
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
Paul Menzel 2014-03-01 12:49:57 +01:00 committed by Patrick Georgi
parent dbc7bd9dce
commit 69813febbc
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/southbridge/amd/amd8111/acpi.c
View File

@ -97,7 +97,6 @@ static void acpi_init(struct device *dev)
{ {
uint8_t byte; uint8_t byte;
uint16_t pm10_bar; uint16_t pm10_bar;
uint32_t dword;
int on; int on;
#if 0 #if 0
@ -156,7 +155,7 @@ static void acpi_init(struct device *dev)
if(on) { if(on) {
pm10_bar = (pci_read_config16(dev, 0x58)&0xff00); pm10_bar = (pci_read_config16(dev, 0x58)&0xff00);
outl(((on<<1)+0x10) ,(pm10_bar + 0x10)); outl(((on<<1)+0x10) ,(pm10_bar + 0x10));
dword = inl(pm10_bar + 0x10); inl(pm10_bar + 0x10);
on = 8-on; on = 8-on;
printk(BIOS_DEBUG, "Throttling CPU %2d.%1.1d percent.\n", printk(BIOS_DEBUG, "Throttling CPU %2d.%1.1d percent.\n",
(on*12)+(on>>1),(on&1)*5); (on*12)+(on>>1),(on&1)*5);