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

From: boris brezillon
Date: Tue Oct 29 2013 - 09:51:14 EST


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.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/