diff options
author | Wolfram Sang <wsa+renesas@sang-engineering.com> | 2016-03-02 17:33:36 -0500 |
---|---|---|
committer | Wim Van Sebroeck <wim@iguana.be> | 2016-03-16 16:09:08 -0400 |
commit | ccc8208d0855a5b02a4fc49c141b6f1dbf282304 (patch) | |
tree | fe95e7e1685fe94fc97697801e759eccb03eac67 | |
parent | 882dec1ff125e9020157d27672ec11c2fdfb82c2 (diff) |
watchdog: atlas7_wdt: test clock rate to avoid division by 0
The clk API may return 0 on clk_get_rate, so we should check the result before
using it as a divisor.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
-rw-r--r-- | drivers/watchdog/atlas7_wdt.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/watchdog/atlas7_wdt.c b/drivers/watchdog/atlas7_wdt.c index df6d9242a319..ed80734befae 100644 --- a/drivers/watchdog/atlas7_wdt.c +++ b/drivers/watchdog/atlas7_wdt.c | |||
@@ -154,6 +154,11 @@ static int atlas7_wdt_probe(struct platform_device *pdev) | |||
154 | writel(0, wdt->base + ATLAS7_WDT_CNT_CTRL); | 154 | writel(0, wdt->base + ATLAS7_WDT_CNT_CTRL); |
155 | 155 | ||
156 | wdt->tick_rate = clk_get_rate(clk); | 156 | wdt->tick_rate = clk_get_rate(clk); |
157 | if (!wdt->tick_rate) { | ||
158 | ret = -EINVAL; | ||
159 | goto err1; | ||
160 | } | ||
161 | |||
157 | wdt->clk = clk; | 162 | wdt->clk = clk; |
158 | atlas7_wdd.min_timeout = 1; | 163 | atlas7_wdd.min_timeout = 1; |
159 | atlas7_wdd.max_timeout = UINT_MAX / wdt->tick_rate; | 164 | atlas7_wdd.max_timeout = UINT_MAX / wdt->tick_rate; |