Re: [PATCH v3 1/1] watchdog: Add common nowayout parameter to booke_wdt driver

From: Timothy Myers
Date: Tue Jul 07 2020 - 08:14:01 EST





From: linux-watchdog-owner@xxxxxxxxxxxxxxx <linux-watchdog-owner@xxxxxxxxxxxxxxx> on behalf of Guenter Roeck <linux@xxxxxxxxxxxx>
Sent: Monday, July 6, 2020 5:18 PM
To: Timothy Myers <timothy.myers@xxxxxxxxxx>; Wim Van Sebroeck <wim@xxxxxxxxxxxxxxxxxx>; linux-watchdog@xxxxxxxxxxxxxxx <linux-watchdog@xxxxxxxxxxxxxxx>
Cc: linux-kernel@xxxxxxxxxxxxxxx <linux-kernel@xxxxxxxxxxxxxxx>; David Boike <david.boike@xxxxxxxxxx>
Subject: Re: [PATCH v3 1/1] watchdog: Add common nowayout parameter to booke_wdt driver
 
On 7/6/20 9:33 AM, Timothy Myers wrote:
>> Add the common "nowayout" parameter to booke_wdt to make this behavior
>> selectable at runtime and to make the implementation more consistent with
>> many other watchdog drivers.
>>
>> Signed-off-by: Timothy Myers <timothy.myers@xxxxxxxxxx>
>> Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>
>>
>> Changes in:
>> v2:
>> Added Guenter Roeck's reviewed-by tag at his request
>> v3:
>> Fix whitespace formatting in message.
>> ---
>
>Change log goes here.
>
>I applied your patch to watchdog-next, but there is still something
>in it which prevents it from showing up at
>https://patchwork.kernel.org/project/linux-watchdog/list/, even though
>I do see it at https://lore.kernel.org/linux-watchdog/.
>
>Looking into the e-mail source, I think the problem may be due
>to some Outlook specific formatting in the raw message:
>
>Add the common "nowayout" parameter to booke_wdt to make this behavior=0A=
>selectable at runtime and to make the implementation more consistent with=
>=0A=
>many other watchdog drivers.=0A=
>=0A=
>
>and so on. If that is persistent, it may make sense to fix it, or
>you might wonder why some of your patches are being ignored.
>
>Thanks,
>Guenter
Thank you for applying the patch, and thank you again for your help with my mail. I'll look into the formatting.