1. Patchew
  2. linux
  3. View series

[PATCH] media: i2c: Fix repeated word in comments

Jules Maselbas posted 1 patch 3 years, 7 months ago 
drivers/media/i2c/mt9v111.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] media: i2c: Fix repeated word in comments
Posted by Jules Maselbas 3 years, 7 months ago 
Remove redundant word `the`.

CC: Jacopo Mondi <jacopo@jmondi.org>
CC: linux-media@vger.kernel.org
Signed-off-by: Jules Maselbas <jmaselbas@kalray.eu>
---
 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.17.1
Re: [PATCH] media: i2c: Fix repeated word in comments
Posted by Jacopo Mondi 3 years, 7 months ago 
Hi Jules

  thanks, but as reported in patchwork and lore
https://patchwork.linuxtv.org/project/linux-media/list/?series=&submitter=&state=*&q=mt9v111&archive=both&delegate=
https://lore.kernel.org/all/20220824135433.h2udwtnub7mlmtam@uno.localdomain/

This is the 4th patch to fix the same repeated word in a comment.

As I asked to Jilin, are you submitting this as part of a new
contributor program ?

Thanks
   j

On Fri, Aug 26, 2022 at 12:00:50PM +0200, Jules Maselbas wrote:
> Remove redundant word `the`.
>
> CC: Jacopo Mondi <jacopo@jmondi.org>
> CC: linux-media@vger.kernel.org
> Signed-off-by: Jules Maselbas <jmaselbas@kalray.eu>
> ---
>  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.17.1
>
Re: [PATCH] media: i2c: Fix repeated word in comments
Posted by Jules Maselbas 3 years, 7 months ago 
Hi Jacopo,

On Fri, Aug 26, 2022 at 12:24:47PM +0200, Jacopo Mondi wrote:
> Hi Jules
> 
>   thanks, but as reported in patchwork and lore
> https://patchwork.linuxtv.org/project/linux-media/list/?series=&submitter=&state=*&q=mt9v111&archive=both&delegate=
> https://lore.kernel.org/all/20220824135433.h2udwtnub7mlmtam@uno.localdomain/
> 
> This is the 4th patch to fix the same repeated word in a comment.
 
> As I asked to Jilin, are you submitting this as part of a new
> contributor program ?
No, I've search for more repeated word after I've found one (in dma-buf heaps).
I then saw that there were already some fixed by Jilin Yuan (and
probably others).

I was expecting that some patches were sent for this, sorry for the noise. 

Cheers
 
> Thanks
>    j
> 
> On Fri, Aug 26, 2022 at 12:00:50PM +0200, Jules Maselbas wrote:
> > Remove redundant word `the`.
> >
> > CC: Jacopo Mondi <jacopo@jmondi.org>
> > CC: linux-media@vger.kernel.org
> > Signed-off-by: Jules Maselbas <jmaselbas@kalray.eu>
> > ---
> >  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.17.1
> >
Re: [PATCH] media: i2c: Fix repeated word in comments
Posted by Hans Verkuil 3 years, 7 months ago 
I'm rejecting these patches in patchwork. All (or almost all) are duplicates.

Regards,

	Hans

On 8/26/22 12:24, Jacopo Mondi wrote:
> Hi Jules
> 
>   thanks, but as reported in patchwork and lore
> https://patchwork.linuxtv.org/project/linux-media/list/?series=&submitter=&state=*&q=mt9v111&archive=both&delegate=
> https://lore.kernel.org/all/20220824135433.h2udwtnub7mlmtam@uno.localdomain/
> 
> This is the 4th patch to fix the same repeated word in a comment.
> 
> As I asked to Jilin, are you submitting this as part of a new
> contributor program ?
> 
> Thanks
>    j
> 
> On Fri, Aug 26, 2022 at 12:00:50PM +0200, Jules Maselbas wrote:
>> Remove redundant word `the`.
>>
>> CC: Jacopo Mondi <jacopo@jmondi.org>
>> CC: linux-media@vger.kernel.org
>> Signed-off-by: Jules Maselbas <jmaselbas@kalray.eu>
>> ---
>>  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.17.1
>>
Re: [PATCH] media: i2c: Fix repeated word in comments
Posted by Jules Maselbas 3 years, 7 months ago 
On Fri, Aug 26, 2022 at 01:23:54PM +0200, Hans Verkuil wrote:
> I'm rejecting these patches in patchwork. All (or almost all) are duplicates.
I was expecting this, I didn't search for exiting patches. Sorry for the
noise.

Thanks
> Regards,
> 
> 	Hans
> 
> On 8/26/22 12:24, Jacopo Mondi wrote:
> > Hi Jules
> > 
> >   thanks, but as reported in patchwork and lore
> > https://patchwork.linuxtv.org/project/linux-media/list/?series=&submitter=&state=*&q=mt9v111&archive=both&delegate=
> > https://lore.kernel.org/all/20220824135433.h2udwtnub7mlmtam@uno.localdomain/
> > 
> > This is the 4th patch to fix the same repeated word in a comment.
> > 
> > As I asked to Jilin, are you submitting this as part of a new
> > contributor program ?
> > 
> > Thanks
> >    j
> > 
> > On Fri, Aug 26, 2022 at 12:00:50PM +0200, Jules Maselbas wrote:
> >> Remove redundant word `the`.
> >>
> >> CC: Jacopo Mondi <jacopo@jmondi.org>
> >> CC: linux-media@vger.kernel.org
> >> Signed-off-by: Jules Maselbas <jmaselbas@kalray.eu>
> >> ---
> >>  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.17.1
> >>

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.