On Fri, Feb 25, 2022 at 6:53 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > Both rzg2l_wdt_probe() and rzg2l_wdt_start() calls reset_control_ > deassert() which results in a reset control imbalance. > > This patch fixes reset control imbalance by removing reset_control_ > deassert() from rzg2l_wdt_start() and replaces reset_control_assert with > reset_control_reset in rzg2l_wdt_stop() as watchdog module can be stopped > only by a module reset. This change will allow us to restart WDT after > stop() by configuring WDT timeout and enable registers. > > Fixes: 2cbc5cd0b55fa2 ("watchdog: Add Watchdog Timer driver for RZ/G2L") > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds