[PATCH] watchdog: Add support for the Avionic Design Xanthos watchdog timer.

[Wim Van Sebroeck]

Hi Thierry,

> > I'm still puzzled. I have seen a number of watchdog devices that have the option to either reboot the system or generate an interrupt.
> > They can be set via toggling the correct bits of some configuration register.
> > 
> > How does this hardware works? Is there a datasheet available somewhere?
> 
> It works exactly the same way. The bit defined by ADX_WDT_CONTROL_nRESET
> controls this behaviour. If set, the watchdog will only generate an
> interrupt, if cleared (the default) it will reboot the system.
> 
> There's currently no real datasheet available. The register definitions at
> the top of adx_wdt.c are pretty much all of the documentation we currently
> have, I'm afraid.

I looked at the register definitions and then searched in your code where
the ADX_WDT_CONTROL_nRESET bit of the watchdog device was initialized.
I didn't find it.

So to get this driver in line with how the other drivers work, we should still
do the following:
1) initialize the device at startup so that the ADX_WDT_CONTROL_nRESET is set correctly
(and the device should also be disabled if it isn't done yet).
2) we can get rid of the interrupt handling because we will use the reboot behaviour.

Can you adapt your code?

Thanks in advance,
Wim.