Skip to content

Commit 000987a

Browse files
georgecherian81Wim Van Sebroeck
authored andcommitted
watchdog: sbsa_wdog: Make sure the timeout programming is within the limits
Make sure to honour the max_hw_heartbeat_ms while programming the timeout value to WOR. Clamp the timeout passed to sbsa_gwdt_set_timeout() to make sure the programmed value is within the permissible range. Fixes: abd3ac7 ("watchdog: sbsa: Support architecture version 1") Signed-off-by: George Cherian <[email protected]> Reviewed-by: Guenter Roeck <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Guenter Roeck <[email protected]> Signed-off-by: Wim Van Sebroeck <[email protected]>
1 parent c76675f commit 000987a

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

drivers/watchdog/sbsa_gwdt.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,7 @@ static int sbsa_gwdt_set_timeout(struct watchdog_device *wdd,
150150
struct sbsa_gwdt *gwdt = watchdog_get_drvdata(wdd);
151151

152152
wdd->timeout = timeout;
153+
timeout = clamp_t(unsigned int, timeout, 1, wdd->max_hw_heartbeat_ms / 1000);
153154

154155
if (action)
155156
sbsa_gwdt_reg_write(gwdt->clk * timeout, gwdt);

0 commit comments

Comments
 (0)