diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2014-03-01 12:49:57 +0100 |
|---|---|---|
| committer | Patrick Georgi <patrick@georgi-clan.de> | 2014-04-09 13:31:31 +0200 |
| commit | 69813febbc91571f9126f980b97fd7b37a45f125 (patch) | |
| tree | 3f47de983be7efabf909c45097a7e590b763203f | |
| parent | dbc7bd9dce2d188dd48de7acd5754507ff0c87e5 (diff) | |
| download | coreboot-staging-69813febbc91571f9126f980b97fd7b37a45f125.zip coreboot-staging-69813febbc91571f9126f980b97fd7b37a45f125.tar.gz | |
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:100: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>
| -rw-r--r-- | src/southbridge/amd/amd8111/acpi.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/southbridge/amd/amd8111/acpi.c b/src/southbridge/amd/amd8111/acpi.c index b734846..bfa5224 100644 --- a/src/southbridge/amd/amd8111/acpi.c +++ b/src/southbridge/amd/amd8111/acpi.c @@ -97,7 +97,6 @@ static void acpi_init(struct device *dev) { uint8_t byte; uint16_t pm10_bar; - uint32_t dword; int on; #if 0 @@ -156,7 +155,7 @@ static void acpi_init(struct device *dev) if(on) { pm10_bar = (pci_read_config16(dev, 0x58)&0xff00); outl(((on<<1)+0x10) ,(pm10_bar + 0x10)); - dword = inl(pm10_bar + 0x10); + inl(pm10_bar + 0x10); on = 8-on; printk(BIOS_DEBUG, "Throttling CPU %2d.%1.1d percent.\n", (on*12)+(on>>1),(on&1)*5); |
