Re: [PATCH v2 04/22] watchdog: ziirave_wdt: Don't bail out on unexpected timeout value

From: Guenter Roeck
Date: Thu Aug 15 2019 - 14:12:17 EST


On Mon, Aug 12, 2019 at 01:08:48PM -0700, Andrey Smirnov wrote:
> Reprogramming bootloader on watchdog MCU will result in reported
> default timeout value of "0". That in turn will be unnecessarily
> rejected by the driver as invalid device (-ENODEV). Simplify probe to
> read stored timeout value, set it to a sane default if it is bogus,
> and then program that value unconditionally.
>
> Signed-off-by: Andrey Smirnov <andrew.smirnov@xxxxxxxxx>
> Cc: Chris Healy <cphealy@xxxxxxxxx>
> Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
> Cc: Rick Ramstetter <rick@xxxxxxxxxxxxxxx>
> Cc: linux-watchdog@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx

Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>