Re: [v2,1/4] watchdog: at91sam9_wdt: better watchdog support

2013-10-02 Thread Guenter Roeck
On Wed, Oct 02, 2013 at 09:53:54PM +0200, boris brezillon wrote: [ ... ] > + > + if (!wdt->heartbeat) { > + dev_err(wdt->wdd.parent, > + ": sorry, linux timer (%i Hz) cannot handle watchdog > timeout (%i ms)\n", > + HZ, ticks_to_ms(va

Re: [v2,1/4] watchdog: at91sam9_wdt: better watchdog support

2013-10-02 Thread boris brezillon
On 02/10/2013 21:34, Guenter Roeck wrote: On Wed, Oct 02, 2013 at 09:27:03PM +0200, boris brezillon wrote: Hello Guenter, Thanks for reviewing this patch. On 02/10/2013 18:12, Guenter Roeck wrote: On Fri, Jun 21, 2013 at 03:21:28PM -, Boris BREZILLON wrote: The at91sam9 watchdog timer ca

Re: [v2,1/4] watchdog: at91sam9_wdt: better watchdog support

2013-10-02 Thread Guenter Roeck
On Wed, Oct 02, 2013 at 09:27:03PM +0200, boris brezillon wrote: > Hello Guenter, > > Thanks for reviewing this patch. > > On 02/10/2013 18:12, Guenter Roeck wrote: > >On Fri, Jun 21, 2013 at 03:21:28PM -, Boris BREZILLON wrote: > >>The at91sam9 watchdog timer can only be configured once, and

Re: [v2,1/4] watchdog: at91sam9_wdt: better watchdog support

2013-10-02 Thread boris brezillon
Hello Guenter, Thanks for reviewing this patch. On 02/10/2013 18:12, Guenter Roeck wrote: On Fri, Jun 21, 2013 at 03:21:28PM -, Boris BREZILLON wrote: The at91sam9 watchdog timer can only be configured once, and the current implementation tries to configure it in a static way: - 2 seconds

Re: [v2,1/4] watchdog: at91sam9_wdt: better watchdog support

2013-10-02 Thread Guenter Roeck
On Fri, Jun 21, 2013 at 03:21:28PM -, Boris BREZILLON wrote: > The at91sam9 watchdog timer can only be configured once, and the current > implementation tries to configure it in a static way: > - 2 seconds timeout > - wdt restart every 500ms > > If the timer has already been configured with di