diff options
| author | Vladimir Zapolskiy <[email protected]> | 2016-12-21 03:18:16 +0200 |
|---|---|---|
| committer | Guenter Roeck <[email protected]> | 2017-02-24 14:00:23 -0800 |
| commit | 027d89d96b6833f37091d858b9dcf92cd9b19aa7 (patch) | |
| tree | e808bda336f93d2bf47d99c7f131d891e30783c1 | |
| parent | f3519a662f5f8abf6411a238b1af08a0101878c7 (diff) | |
watchdog: sa11x0/pxa: fix error path of driver initialization
The change corrects release of captured resources on error path, namely
the clock is disabled and put if misc device registration fails and not
enabled clock is not disabled now.
Fixes: 6924089c488e ("watchdog: sa11x0/pxa: get rid of get_clock_tick_rate")
Signed-off-by: Vladimir Zapolskiy <[email protected]>
Acked-by: Robert Jarzmik <[email protected]>
Reviewed-by: Guenter Roeck <[email protected]>
Acked-by: Russell King <[email protected]>
Signed-off-by: Guenter Roeck <[email protected]>
| -rw-r--r-- | drivers/watchdog/sa1100_wdt.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/drivers/watchdog/sa1100_wdt.c b/drivers/watchdog/sa1100_wdt.c index 8965e3f536c3..d3be4f831db5 100644 --- a/drivers/watchdog/sa1100_wdt.c +++ b/drivers/watchdog/sa1100_wdt.c @@ -188,12 +188,14 @@ static int __init sa1100dog_init(void) pre_margin = oscr_freq * margin; ret = misc_register(&sa1100dog_miscdev); - if (ret == 0) + if (ret == 0) { pr_info("SA1100/PXA2xx Watchdog Timer: timer margin %d sec\n", margin); - return ret; -err: + return 0; + } + clk_disable_unprepare(clk); +err: clk_put(clk); return ret; } |