Add driver for Synopsys Designware Watchdog timer.

[kjopek_gmail.com]

Summary

This driver supports some arm and arm64 boards equipped with "snps,dw-wdt"-compatible watchdog device. Tested on RK3399-based board (RockPro64). Once started watchdog device cannot be stopped. Interrupt handler has mode to kick watchdog even when software does not do it properly. This can be controlled via sysctl: dev.dwwdt.prevent_restart. Also - driver handles system shutdown and prevents from restart when system is asked to reboot.

Bibliography:

• RK3399 TRM Part 1 (it lacks details on clock configuration)

Other implementations:

• OpenBSD driver (it uses watchdog device to reset system): https://github.com/openbsd/src/blob/master/sys/dev/fdt/dwdog.c
• Linux driver (there is no 1:1 compatibility right now, but Linux folks have the same problem with clock configuration): https://github.com/torvalds/linux/blob/master/drivers/watchdog/dw_wdt.c

Test Plan

Tested on RockPro64

[kjopek_gmail.com]

sys/arm64/rockchip/clk/rk3399_cru.c

1214 ↗

(On Diff #78173)

No clock without this change.

sys/dev/dwwdt/dwwdt.c

63

This is uncalibrated. No guarantee that we will hit any particular time period.

289

Clock returns 0 even with the change above to CRU.

321

We can't detach if watchdog was once started.

[kjopek_gmail.com]

Fix interrupt handling: we should clear interrupt bit only if we want to tap watchdog. Minor fixes to build system, remove clock configuration in RK3399 - this should be done in separated commit.

[kjopek_gmail.com]

Add calibration.

[manu]

sys/arm64/rockchip/clk/rk3399_cru.c

1214 ↗

(On Diff #80878)

I've commited r368766 which fixes this clock.
You should need to fallback on pclk_alive now.

[kjopek_gmail.com]

Rework code to remove workarounds, remove clock definition, be more precise on calibration.

[manu]

Overall looks good, just a few questions on code that I don't understand.

sys/dev/dwwdt/dwwdt.c

160

This is more a full reload than a stop no ?

196

Why do we need to reload the counter here if we reload it after ?

[kjopek_gmail.com]

sys/dev/dwwdt/dwwdt.c

160

This "reload" is to decrese interrupt rate when watchdog is stopped. As dwwdt cannot be really stopped once started, I had to provide kind of "emulation". Counter reload here is just for keeping interrupt rate at minimal possible rate.

196

We don't really need to reload counter here. It is just to save couple of lines of code and to prevent reboot while reconfiguring watchdog.

[kjopek_gmail.com]

Fixed calibration issue: we should add one tick, not substract.