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

From: Wim Van Sebroeck
Date: Tue Oct 29 2013 - 08:59:09 EST


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.

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/