Re: [PATCH 2/3] media: fix repeated words in comments

[Laurent Pinchart]

On Thu, Sep 10, 2026 at 01:30:11PM +0200, Hans Verkuil wrote:
> On 04/09/2026 15:36, Laurent Pinchart wrote:
> > On Fri, Sep 04, 2026 at 05:04:43PM +0530, Hemanth Selam wrote:
> >> Drop words accidentally written twice, reported by checkpatch.pl as a
> >> possible repeated word.  Only touches comments, no code changes.
> >>
> >> Assisted-by: Cursor:claude-opus-5
> > 
> > This is not a good use of reviewer time. NAK.
> 
> Actually, I'll take this patch.
> 
> Laurent, feel free to delegate patches fixing typos/grammar to me. I hate
> typos etc. in code, they actually make the code harder to read. Patches
> fixing that take very little time to review, so I'm fine doing that.

I don't welcome people who use LLMs to generate noise with barely any
value, just for the sake of having their name in the kernel git log. We
have enough important patches to review. This kind of submission should
be discouraged. It would be different if the typo fixes were part of a
human-written series that adds real value.

> >> Signed-off-by: Hemanth Selam <hemanth.selam@xxxxxxxxx>
> >> ---
> >>  drivers/media/i2c/adv7175.c                   | 2 +-
> >>  drivers/media/platform/renesas/vsp1/vsp1_dl.c | 2 +-
> >>  drivers/media/usb/gspca/ov519.c               | 2 +-
> >>  3 files changed, 3 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/drivers/media/i2c/adv7175.c b/drivers/media/i2c/adv7175.c
> >> index f1caab8e2abd..b6f0e1b24383 100644
> >> --- a/drivers/media/i2c/adv7175.c
> >> +++ b/drivers/media/i2c/adv7175.c
> >> @@ -209,7 +209,7 @@ static int adv7175_s_std_output(struct v4l2_subdev *sd, v4l2_std_id std)
> >>  		/* This is an attempt to convert
> >>  		 * SECAM->PAL (typically it does not work
> >>  		 * due to genlock: when decoder is in SECAM
> >> -		 * and encoder in in PAL the subcarrier can
> >> +		 * and encoder in PAL the subcarrier can
> >>  		 * not be synchronized with horizontal
> >>  		 * quency) */
> >>  		adv7175_write_block(sd, init_pal, sizeof(init_pal));
> >> diff --git a/drivers/media/platform/renesas/vsp1/vsp1_dl.c b/drivers/media/platform/renesas/vsp1/vsp1_dl.c
> >> index 6430f2ec8b32..8257759421f3 100644
> >> --- a/drivers/media/platform/renesas/vsp1/vsp1_dl.c
> >> +++ b/drivers/media/platform/renesas/vsp1/vsp1_dl.c
> >> @@ -649,7 +649,7 @@ static void __vsp1_dl_list_put(struct vsp1_dl_list *dl)
> >>  	dl->post_cmd = NULL;
> >>  
> >>  	/*
> >> -	 * body0 is reused as as an optimisation as presently every display list
> >> +	 * body0 is reused as an optimisation as presently every display list
> >>  	 * has at least one body, thus we reinitialise the entries list.
> >>  	 */
> >>  	dl->body0->num_entries = 0;
> >> diff --git a/drivers/media/usb/gspca/ov519.c b/drivers/media/usb/gspca/ov519.c
> >> index bffa94e76da5..57219a738c73 100644
> >> --- a/drivers/media/usb/gspca/ov519.c
> >> +++ b/drivers/media/usb/gspca/ov519.c
> >> @@ -13,7 +13,7 @@
> >>   * Copyright (c) 1999-2006 Mark W. McClelland
> >>   * Support for OV519, OV8610 Copyright (c) 2003 Joerg Heckenbach
> >>   * Many improvements by Bret Wallach <bwallac1@xxxxxxxxxx>
> >> - * Color fixes by by Orion Sky Lawlor <olawlor@xxxxxxx> (2/26/2000)
> >> + * Color fixes by Orion Sky Lawlor <olawlor@xxxxxxx> (2/26/2000)
> >>   * OV7620 fixes by Charl P. Botha <cpbotha@xxxxxxxx>
> >>   * Changes by Claudio Matsuoka <claudio@xxxxxxxxxxxxx>
> >>   *

-- 
Regards,

Laurent Pinchart