Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-11-03 Thread boris brezillon
On 03/11/2013 17:22, Guenter Roeck wrote: On 11/03/2013 06:42 AM, boris brezillon wrote: Hello Guenter, Are you okay with the 3 fixes/improvements provided by this patch, and the explanation I gave regarding the reason for these changes ? If so, I will submit a new series splitting the patch a

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-11-03 Thread Guenter Roeck
On 11/03/2013 06:42 AM, boris brezillon wrote: Hello Guenter, Are you okay with the 3 fixes/improvements provided by this patch, and the explanation I gave regarding the reason for these changes ? If so, I will submit a new series splitting the patch and including your suggestions. Yes, sorr

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-11-03 Thread boris brezillon
Hello Guenter, Are you okay with the 3 fixes/improvements provided by this patch, and the explanation I gave regarding the reason for these changes ? If so, I will submit a new series splitting the patch and including your suggestions. Best Regards, Boris On 29/10/2013 18:22, boris brezillo

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-31 Thread Jean-Christophe PLAGNIOL-VILLARD
On 14:27 Tue 29 Oct , Guenter Roeck wrote: > On Tue, Oct 29, 2013 at 06:22:47PM +0100, boris brezillon wrote: > > On 29/10/2013 17:43, Guenter Roeck wrote: > > >On Tue, Oct 29, 2013 at 05:22:50PM +0100, boris brezillon wrote: > > >>On 29/10/2013 16:45, Guenter Roeck wrote: > > >>>On Tue, Oct 29

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread b.brezillon
On Tue, 29 Oct 2013 14:27:38 -0700, Guenter Roeck wrote: > On Tue, Oct 29, 2013 at 06:22:47PM +0100, boris brezillon wrote: >> On 29/10/2013 17:43, Guenter Roeck wrote: >> >On Tue, Oct 29, 2013 at 05:22:50PM +0100, boris brezillon wrote: >> >>On 29/10/2013 16:45, Guenter Roeck wrote: >> >>>On Tue,

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread Guenter Roeck
On Tue, Oct 29, 2013 at 06:22:47PM +0100, boris brezillon wrote: > On 29/10/2013 17:43, Guenter Roeck wrote: > >On Tue, Oct 29, 2013 at 05:22:50PM +0100, boris brezillon wrote: > >>On 29/10/2013 16:45, Guenter Roeck wrote: > >>>On Tue, Oct 29, 2013 at 11:37:33AM +0100, Boris BREZILLON wrote: >

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread boris brezillon
On 29/10/2013 17:43, Guenter Roeck wrote: On Tue, Oct 29, 2013 at 05:22:50PM +0100, boris brezillon wrote: On 29/10/2013 16:45, Guenter Roeck wrote: On Tue, Oct 29, 2013 at 11:37:33AM +0100, Boris BREZILLON wrote: Fix the secs_to_ticks macro in case 0 is passed as an argument. Rework the hear

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread Guenter Roeck
On Tue, Oct 29, 2013 at 05:22:50PM +0100, boris brezillon wrote: > On 29/10/2013 16:45, Guenter Roeck wrote: > >On Tue, Oct 29, 2013 at 11:37:33AM +0100, Boris BREZILLON wrote: > >>Fix the secs_to_ticks macro in case 0 is passed as an argument. > >> > >>Rework the heartbeat calculation to increase

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread boris brezillon
On 29/10/2013 16:45, Guenter Roeck wrote: On Tue, Oct 29, 2013 at 11:37:33AM +0100, Boris BREZILLON wrote: Fix the secs_to_ticks macro in case 0 is passed as an argument. Rework the heartbeat calculation to increase the security margin of the watchdog reset timer. Use the min_heartbeat value i

Re: [PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread Guenter Roeck
On Tue, Oct 29, 2013 at 11:37:33AM +0100, Boris BREZILLON wrote: > Fix the secs_to_ticks macro in case 0 is passed as an argument. > > Rework the heartbeat calculation to increase the security margin of the > watchdog reset timer. > > Use the min_heartbeat value instead of the calculated heartbea

[PATCH] watchdog: at91sam9_wdt: various fixes

2013-10-29 Thread Boris BREZILLON
Fix the secs_to_ticks macro in case 0 is passed as an argument. Rework the heartbeat calculation to increase the security margin of the watchdog reset timer. Use the min_heartbeat value instead of the calculated heartbeat value for the first watchdog reset. Signed-off-by: Boris BREZILLON --- d