diff options
| author | Arnd Bergmann <arnd@arndb.de> | 2016-01-25 16:45:49 +0100 |
|---|---|---|
| committer | Sebastian Reichel <sre@kernel.org> | 2016-02-15 06:01:51 +0100 |
| commit | 6a65b2b2af716854986b6d7d17ce096a949be863 (patch) | |
| tree | f5b7bbe9e6286a15c293bc703b54d68acc5942bb /drivers/power | |
| parent | 3f652c969dd223a6ff1df6e9a7052b0424a14167 (diff) | |
| download | linux-6a65b2b2af716854986b6d7d17ce096a949be863.tar.gz | |
power: collie_battery: hide unused variable
The 'wakeup_enabled' variable in this driver is only accessed by the optional power management functions and we get a warning when they are disabled: drivers/power/collie_battery.c:29:12: error: 'wakeup_enabled' defined but not used [-Werror=unused-variable] This moves the definition next to the users inside the same #ifdef, which avoids the warning. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Acked-by: Pavel Machek <pavel@ucw.cz> Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/power')
| -rw-r--r-- | drivers/power/collie_battery.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/power/collie_battery.c b/drivers/power/collie_battery.c index 8a971b3dbe58..3a0bc608d4b5 100644 --- a/drivers/power/collie_battery.c +++ b/drivers/power/collie_battery.c @@ -26,7 +26,6 @@ static DEFINE_MUTEX(bat_lock); /* protects gpio pins */ static struct work_struct bat_work; static struct ucb1x00 *ucb; -static int wakeup_enabled; struct collie_bat { int status; @@ -291,6 +290,8 @@ static struct gpio collie_batt_gpios[] = { }; #ifdef CONFIG_PM +static int wakeup_enabled; + static int collie_bat_suspend(struct ucb1x00_dev *dev) { /* flush all pending status updates */ |