diff mbox series

[3/7] wdt: dw: Fix clock rate being off by 1000

Message ID 20200805191414.146609-3-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 clock subsystem returns clock rates in Hz. We need to divide by 1000 so
the rate is in kHz.

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

 drivers/watchdog/designware_wdt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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 clock subsystem returns clock rates in Hz. We need to divide by 1000 so
> the rate is in kHz.
>
> Fixes: cf89ef8d10f240554541c20b2e1bdcdd58d1d7e6
> Signed-off-by: Sean Anderson <seanga2@gmail.com>
> ---
>
>  drivers/watchdog/designware_wdt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

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 1d46382f67..c7a0c896b8 100644
--- a/drivers/watchdog/designware_wdt.c
+++ b/drivers/watchdog/designware_wdt.c
@@ -129,7 +129,7 @@  static int designware_wdt_probe(struct udevice *dev)
 	if (ret)
 		return ret;
 
-	priv->clk_khz = clk_get_rate(&clk);
+	priv->clk_khz = clk_get_rate(&clk) / 1000;
 	if (!priv->clk_khz)
 		return -EINVAL;
 #else