On Thu, May 24, 2018 at 09:36:25AM -0700, Ray Jui wrote: > > > On 5/24/2018 9:19 AM, Guenter Roeck wrote: > >On Wed, May 23, 2018 at 05:15:22PM -0700, Ray Jui wrote: > >>If the watchdog hardware is already enabled during the boot process, > >>when the Linux watchdog driver loads, it should reset the watchdog and > >>tell the watchdog framework. As a result, ping can be generated from > >>the watchdog framework, until the userspace watchdog daemon takes over > >>control > >> > >>Signed-off-by: Ray Jui <ray.jui@xxxxxxxxxxxx> > >>Reviewed-by: Vladimir Olovyannikov <vladimir.olovyannikov@xxxxxxxxxxxx> > >>Reviewed-by: Scott Branden <scott.branden@xxxxxxxxxxxx> > > > >Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx> > > > >I have one question, though: Is it really correct that both > >INT_ENABLE _and_ RESET_ENABLE have to be set to enable the watdog ? > >What if only RESET_ENABLE is set ? > > According to the SP805 TRM, INT_ENABLE needs to be set to high to enable the > counter and the interrupt. Counter will be stopped if INT_ENABLE is cleared. > So yes, INT_ENABLE needs to be set. > Excellent, thanks for the clarification. Guenter -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
