diff mbox series

[4/7] wdt: dw: Enable the clock before using it

Message ID 20200805191414.146609-4-seanga2@gmail.com
State Superseded
Delegated to: Andes
Headers show
Series [1/7] wdt: dw: Switch to using fls for log2 | expand

Commit Message

Sean Anderson Aug. 5, 2020, 7:14 p.m. UTC 
The watchdog won't work if the clock isn't enabled.

Fixes: cf89ef8d10f240554541c20b2e1bdcdd58d1d7e6
Signed-off-by: Sean Anderson <seanga2@gmail.com>
---

 drivers/watchdog/designware_wdt.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Simon Glass Aug. 16, 2020, 3:39 a.m. UTC | #1 
On Wed, 5 Aug 2020 at 13:14, Sean Anderson <seanga2@gmail.com> wrote:
>
> The watchdog won't work if the clock isn't enabled.
>
> Fixes: cf89ef8d10f240554541c20b2e1bdcdd58d1d7e6
> Signed-off-by: Sean Anderson <seanga2@gmail.com>
> ---
>
>  drivers/watchdog/designware_wdt.c | 4 ++++
>  1 file changed, 4 insertions(+)
>

Reviewed-by: Simon Glass <sjg@chromium.org>
diff mbox series

Patch

diff --git a/drivers/watchdog/designware_wdt.c b/drivers/watchdog/designware_wdt.c
index c7a0c896b8..70c863dc9c 100644
--- a/drivers/watchdog/designware_wdt.c
+++ b/drivers/watchdog/designware_wdt.c
@@ -129,6 +129,10 @@  static int designware_wdt_probe(struct udevice *dev)
 	if (ret)
 		return ret;
 
+	ret = clk_enable(&clk);
+	if (ret)
+		return ret;
+
 	priv->clk_khz = clk_get_rate(&clk) / 1000;
 	if (!priv->clk_khz)
 		return -EINVAL;