Debug message was showing timeout value which was passed to start
function but there is a checking if this value can be setup.
The patch is moving this debug printf function below checking.

Signed-off-by: Michal Simek <michal.si...@xilinx.com>
---

 drivers/watchdog/cdns_wdt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/watchdog/cdns_wdt.c b/drivers/watchdog/cdns_wdt.c
index c43f7e8096ca..25cb34c493d2 100644
--- a/drivers/watchdog/cdns_wdt.c
+++ b/drivers/watchdog/cdns_wdt.c
@@ -143,13 +143,13 @@ static int cdns_wdt_start(struct udevice *dev, u64 
timeout, ulong flags)
                return -1;
        }
 
-       debug("%s: CLK_FREQ %ld, timeout %lld\n", __func__, clk_f, timeout);
-
        if ((timeout < CDNS_WDT_MIN_TIMEOUT) ||
            (timeout > CDNS_WDT_MAX_TIMEOUT)) {
                timeout = priv->timeout;
        }
 
+       debug("%s: CLK_FREQ %ld, timeout %lld\n", __func__, clk_f, timeout);
+
        if (clk_f <= CDNS_WDT_CLK_75MHZ) {
                prescaler = CDNS_WDT_PRESCALE_512;
                ctrl_clksel = CDNS_WDT_PRESCALE_SELECT_512;
-- 
2.17.0

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to