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

Message ID	20200901200758.543851-4-seanga2@gmail.com
State	Superseded
Delegated to:	Andes
Headers	show Return-Path: <u-boot-bounces@lists.denx.de> From: Sean Anderson <seanga2@gmail.com> To: u-boot@lists.denx.de Cc: Bin Meng <bmeng.cn@gmail.com>, Heinrich Schuchardt <xypron.glpk@gmx.de>, Jagan Teki <jagan@amarulasolutions.com>, Simon Glass <sjg@chromium.org>, Marek Vasut <marex@denx.de>, Sean Anderson <seanga2@gmail.com> Subject: [PATCH v3 3/7] wdt: dw: Fix clock rate being off by 1000 Date: Tue, 1 Sep 2020 16:07:54 -0400 Message-Id: <20200901200758.543851-4-seanga2@gmail.com> In-Reply-To: <20200901200758.543851-1-seanga2@gmail.com> References: <20200901200758.543851-1-seanga2@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de>
Series	wdt: Add support for watchdogs on Kendryte K210 \| expand [v3,0/7] wdt: Add support for watchdogs on Kendryte K210 [v3,1/7] wdt: dw: Switch to using fls for log2 [v3,2/7] wdt: dw: Switch to if(CONFIG()) instead of using #if [v3,3/7] wdt: dw: Fix clock rate being off by 1000 [v3,4/7] wdt: dw: Enable the clock before using it [v3,5/7] wdt: dw: Free the clock on error [v3,6/7] riscv: Add watchdog bindings for the k210 [v3,7/7] riscv: Enable watchdog for the k210

Message ID

20200901200758.543851-4-seanga2@gmail.com

State

Superseded

Delegated to:

Andes

Headers

From: Sean Anderson <seanga2@gmail.com>
To: u-boot@lists.denx.de
Cc: Bin Meng <bmeng.cn@gmail.com>, Heinrich Schuchardt <xypron.glpk@gmx.de>,
 Jagan Teki <jagan@amarulasolutions.com>, Simon Glass <sjg@chromium.org>,
 Marek Vasut <marex@denx.de>, Sean Anderson <seanga2@gmail.com>
Subject: [PATCH v3 3/7] wdt: dw: Fix clock rate being off by 1000
Date: Tue,  1 Sep 2020 16:07:54 -0400
Message-Id: <20200901200758.543851-4-seanga2@gmail.com>
In-Reply-To: <20200901200758.543851-1-seanga2@gmail.com>
References: <20200901200758.543851-1-seanga2@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: list
Errors-To: u-boot-bounces@lists.denx.de
Sender: "U-Boot" <u-boot-bounces@lists.denx.de>

Series

wdt: Add support for watchdogs on Kendryte K210 | expand

Commit Message

Sean Anderson Sept. 1, 2020, 8:07 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>
Reviewed-by: Simon Glass <sjg@chromium.org>
---

(no changes since v1)

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

diff --git a/drivers/watchdog/designware_wdt.c b/drivers/watchdog/designware_wdt.c
index 49cf861d46..41866fa01b 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

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

Commit Message

Patch