Re: [PATCH 2/3] media: fix repeated words in comments
From: Hans Verkuil
Date: Thu Sep 10 2026 - 07:40:03 EST
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.
Regards,
Hans
>
>> 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>
>> *
>