Re: [PATCH] watchdog: gpio_wdt: use __udelay() to avoid recursion

2022-09-27 Thread Stefan Roese
On 27.09.22 09:45, Rasmus Villemoes wrote: The udelay() function in lib/time.c contains a WATCHDOG_RESET() call. The only reason this doesn't lead to a catastrophic infinite recursion is due to the rate-limiting in wdt-uclass.c: if (time_after_eq(now, priv->next_reset)) {

Re: [PATCH] watchdog: gpio_wdt: use __udelay() to avoid recursion

2022-09-27 Thread Pali Rohár
On Tuesday 27 September 2022 10:00:00 Stefan Roese wrote: > The max6370_wdt.c driver has the same problem AFAICT. Pali, could you > please take a look a this? Done, here is patch: https://patchwork.ozlabs.org/project/uboot/patch/20220927101919.6999-1-p...@kernel.org/

Re: [PATCH] watchdog: gpio_wdt: use __udelay() to avoid recursion

2022-09-27 Thread Stefan Roese
Hi Rasmus, (add Pali to Cc) On 27.09.22 09:45, Rasmus Villemoes wrote: The udelay() function in lib/time.c contains a WATCHDOG_RESET() call. The only reason this doesn't lead to a catastrophic infinite recursion is due to the rate-limiting in wdt-uclass.c: if (time_after_eq(now

[PATCH] watchdog: gpio_wdt: use __udelay() to avoid recursion

2022-09-27 Thread Rasmus Villemoes
The udelay() function in lib/time.c contains a WATCHDOG_RESET() call. The only reason this doesn't lead to a catastrophic infinite recursion is due to the rate-limiting in wdt-uclass.c: if (time_after_eq(now, priv->next_reset)) { priv->next_reset = now + pri