2022-08-08 16:22:37

by Shaomin Deng

Subject: [PATCH] watchdog: Fix typo in comments

Delete the repeated word "we" in comments.

Signed-off-by: shaomin Deng <[email protected]>
---
drivers/watchdog/pc87413_wdt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/watchdog/pc87413_wdt.c b/drivers/watchdog/pc87413_wdt.c
index 9f9a340427fc..14f37b3b903f 100644
--- a/drivers/watchdog/pc87413_wdt.c
+++ b/drivers/watchdog/pc87413_wdt.c
@@ -333,7 +333,7 @@ static int pc87413_status(void)
* @ppos: pointer to the position to write. No seeks allowed
*
* A write to a watchdog device is defined as a keepalive signal. Any
- * write of data will do, as we we don't define content meaning.
+ * write of data will do, as we don't define content meaning.
*/

static ssize_t pc87413_write(struct file *file, const char __user *data,
--
2.35.1

2022-08-08 16:39:20

by Guenter Roeck

Subject: Re: [PATCH] watchdog: Fix typo in comments

On 8/8/22 08:44, shaomin Deng wrote:
> Delete the repeated word "we" in comments.
>
> Signed-off-by: shaomin Deng <[email protected]>

Please at least use a proper subject indicating the affected driver if you really
have to make those cosmetic changes.

Guenter

> ---
> drivers/watchdog/pc87413_wdt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/watchdog/pc87413_wdt.c b/drivers/watchdog/pc87413_wdt.c
> index 9f9a340427fc..14f37b3b903f 100644
> --- a/drivers/watchdog/pc87413_wdt.c
> +++ b/drivers/watchdog/pc87413_wdt.c
> @@ -333,7 +333,7 @@ static int pc87413_status(void)
> * @ppos: pointer to the position to write. No seeks allowed
> *
> * A write to a watchdog device is defined as a keepalive signal. Any
> - * write of data will do, as we we don't define content meaning.
> + * write of data will do, as we don't define content meaning.
> */
>
> static ssize_t pc87413_write(struct file *file, const char __user *data,