Messages in this thread | | | Date | Tue, 29 Oct 2013 14:25:38 +0100 | From | boris brezillon <> | Subject | Re: [PATCH v5 0/4] watchdog: at91sam9_wdt: handle already configured wdt |
| |
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 kernel might trigger a watchdog reset before >> the new kernel >> is able to load the watchdog driver) >> 3) when initializing the watchdog driver, the timer should be configured >> with the min_heartbeat value >> instead of the standard heartbeat value, because we don't for how >> long the timer has been running. >> >> I'll send a new patch fixing those issues. >> I hope it won't bother you :-(. > No problem :-). Just sent me the fixes when they are ready.
Already done : https://lkml.org/lkml/2013/10/29/207.
Thanks.
Best Regards,
Boris
> > Kind regards, > Wim. >
| |