Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt

2013-10-29 Thread boris brezillon
On 29/10/2013 13:58, Wim Van Sebroeck wrote: Hi Boris, I'm sorry for the inconvenience, but I found some bugs in my patch series: 1) the secs_to_ticks returns an erronous value when 0 is passed as an argument 2) the calculated heartbeat is too small for some use cases (i.e. kexecing a new

Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt

2013-10-29 Thread boris brezillon
Hi Wim, I'm sorry for the inconvenience, but I found some bugs in my patch series: 1) the secs_to_ticks returns an erronous value when 0 is passed as an argument 2) the calculated heartbeat is too small for some use cases (i.e. kexecing a new kernel might trigger a watchdog reset before t

Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt

2013-10-29 Thread Wim Van Sebroeck
Hi Boris, > I'm sorry for the inconvenience, but I found some bugs in my patch series: > > 1) the secs_to_ticks returns an erronous value when 0 is passed as an > argument > 2) the calculated heartbeat is too small for some use cases > (i.e. kexecing a new kernel might trigger a watchdog res

Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt

2013-10-29 Thread Wim Van Sebroeck
Hi Boris, > Hello, > > This patch series is a porposal to enhance the sam9 watchdog timer support. > > 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 ti

[PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt

2013-10-04 Thread Boris BREZILLON
Hello, This patch series is a porposal to enhance the sam9 watchdog timer support. 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