diff options
| author | Jiri Slaby <jirislaby@gmail.com> | 2009-06-29 20:00:39 +0400 |
|---|---|---|
| committer | Wim Van Sebroeck <wim@iguana.be> | 2009-07-09 22:29:16 +0400 |
| commit | db5d2d8a5dfe0ae3e83ac618fd953ecc621adcdf (patch) | |
| tree | 2124d7b3ac8dbb72dc0784e7f40b5d08bec60f9e /drivers/watchdog/w83697ug_wdt.c | |
| parent | b86a6c6c7b0bfc26b3e8d4f48e16ee0b13716385 (diff) | |
| download | linux-db5d2d8a5dfe0ae3e83ac618fd953ecc621adcdf.tar.xz | |
[WATCHDOG] w83697ug, fix lock imbalance
Don't forget to unlock io_lock when w83697ug_select_wd_register fails in
wdt_ctrl.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Diffstat (limited to 'drivers/watchdog/w83697ug_wdt.c')
| -rw-r--r-- | drivers/watchdog/w83697ug_wdt.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/watchdog/w83697ug_wdt.c b/drivers/watchdog/w83697ug_wdt.c index 883b5f79673a..a6c12dec91a1 100644 --- a/drivers/watchdog/w83697ug_wdt.c +++ b/drivers/watchdog/w83697ug_wdt.c @@ -149,8 +149,10 @@ static void wdt_ctrl(int timeout) { spin_lock(&io_lock); - if (w83697ug_select_wd_register() < 0) + if (w83697ug_select_wd_register() < 0) { + spin_unlock(&io_lock); return; + } outb_p(0xF4, WDT_EFER); /* Select CRF4 */ outb_p(timeout, WDT_EFDR); /* Write Timeout counter to CRF4 */ |