On 18/03/2022 20:38, Guenter Roeck wrote:
On 3/18/22 11:26, Jean-Jacques Hiblot wrote:
Hi,
I did not receive any feedback for those 2 patches of the v3 of this
series, so resending them. The rest of the patches (dt-stuff) have
Well, yes, you didn't send it to linux-watchdog@xxxxxxxxxxxxxxx, which
yes I see now that linux-watchdog@xxxxxxxxxxxxxxx is missing from the
send list.
I'll make sure to include it next time. Sorry about that.
I use as base for reviews, to collect review feedback, and as source
for applying the patches. On top of that, you didn't provide change logs
either, meaning I'd have to dig on the web to find older versions of
your patches (since they won't be in watchdog patchwork either) and
compare those. All that takes a lot of time, and you really should not
expect fast turnaround (if any - your v2 got completely lost for me).
Also, FWIW, after doing just that right now I still disagree that it
makes
sense to have a watchdog interrupt do nothing. That is just as good as no
watchdog.
if it's okay, I can add a call to emergency_restart() in the interrupt
handler
as it is done in other drivers (sama5d4.c, wdt.c).
Thanks,
Jean-Jacques
Guenter
