Re: [PATCH v2 2/2] drivers/watchdog: ASPEED reference dev tree properties for config

Guenter Roeck
2017-06-28T17:52:27Z

On Wed, Jun 28, 2017 at 11:09:15AM -0500, Christopher Bostic wrote:
>
> >>
> >>Why would there even be a need configure for no system reset? The external
> >>mode 'wdt ext: External signal enable after timeout' can be sent off chip
> >>even if system reset is disabled. The watchdog is doing something in this
> >>case and would require wdt instantiation.
> >>
> >Wouldn't that be "aspeed,external-signal" ?
> Yes that optional parameter would need to be specified in that situation, as
> well as 'aspeed,no-system-reset'.
>

Ok, makes sense (after looking into the datasheet). The missing detail
is that there is a separate configuration bit for "Reset system after
timeout", which is set in addition to reset_{sys,soc,arm}.

Thanks,
Guenter