2022-08-24 13:57:52

by Jilin Yuan

[permalink] [raw]
Subject: [PATCH] media/i2c: fix repeated words in comments

Delete the redundant word 'the'.

Signed-off-by: Jilin Yuan <[email protected]>
---
drivers/media/i2c/mt9v111.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/i2c/mt9v111.c b/drivers/media/i2c/mt9v111.c
index 2dc4a0f24ce8..7beca0b70b72 100644
--- a/drivers/media/i2c/mt9v111.c
+++ b/drivers/media/i2c/mt9v111.c
@@ -633,7 +633,7 @@ static int mt9v111_hw_config(struct mt9v111_dev *mt9v111)

/*
* Set pixel integration time to the whole frame time.
- * This value controls the the shutter delay when running with AE
+ * This value controls the shutter delay when running with AE
* disabled. If longer than frame time, it affects the output
* frame rate.
*/
--
2.36.1


2022-08-24 14:15:23

by jacopo mondi

[permalink] [raw]
Subject: Re: [PATCH] media/i2c: fix repeated words in comments

Hi Jilin

On Wed, Aug 24, 2022 at 09:14:59PM +0800, Jilin Yuan wrote:
> Delete the redundant word 'the'.
>
> Signed-off-by: Jilin Yuan <[email protected]>

Already addressed by
https://patchwork.kernel.org/project/linux-media/patch/[email protected]/

I wonder what's happening... This is the 3rd if not 4th identical
patch I receive for this issue.

Are you sending this patch as part of a new contributors program like
outreachy or similar ? If that's the case I would like to ask mentors
to require contributors to check patchwork/lore before sending a
patch. I'm reading a lot of "thanks but it's already fixed" replies on
the media list these days..

> ---
> drivers/media/i2c/mt9v111.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/i2c/mt9v111.c b/drivers/media/i2c/mt9v111.c
> index 2dc4a0f24ce8..7beca0b70b72 100644
> --- a/drivers/media/i2c/mt9v111.c
> +++ b/drivers/media/i2c/mt9v111.c
> @@ -633,7 +633,7 @@ static int mt9v111_hw_config(struct mt9v111_dev *mt9v111)
>
> /*
> * Set pixel integration time to the whole frame time.
> - * This value controls the the shutter delay when running with AE
> + * This value controls the shutter delay when running with AE
> * disabled. If longer than frame time, it affects the output
> * frame rate.
> */
> --
> 2.36.1
>