Re: [PATCH v10 3/3] watchdog: make the device time out at open_deadline when open_timeout is used

From: Rasmus Villemoes
Date: Fri Jun 14 2019 - 04:46:11 EST


On 07/06/2019 20.38, Guenter Roeck wrote:
> On Wed, Jun 05, 2019 at 02:06:44PM +0000, Rasmus Villemoes wrote:
>> When the watchdog device is not open by userspace, the kernel takes
>> care of pinging it. When the open_timeout feature is in use, we should
>> ensure that the hardware fires close to open_timeout seconds after the
>> kernel has assumed responsibility for the device.
>>
>> To do this, simply reuse the logic that is already in place for
>> ensuring the same thing when userspace is responsible for regularly
>> pinging the device:
>>
>> - When watchdog_active(wdd), this patch doesn't change anything.
>>
>> - When !watchdoc_active(wdd), the "virtual timeout" should be taken to
>
> s/watchdoc_active/watchdog_active/
>
> otherwise
>
> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>

Thanks! Wim, can you fix up if/when applying, or do you prefer I resend?

Rasmus