linux-kernel - Re: [PATCH] i2c/cx25840: fix repeated words in comments

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <42384aba-d519-1a98-80dc-00a997e7243f@xs4all.nl>
Date:   Wed, 24 Aug 2022 17:01:20 +0200
From:   Hans Verkuil <hverkuil@...all.nl>
To:     Jilin Yuan <yuanjilin@...rlc.com>, mchehab@...nel.org
Cc:     linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] i2c/cx25840: fix repeated words in comments

This is a duplicate of:

https://patchwork.linuxtv.org/project/linux-media/patch/20220821151552.58514-1-wangjianli@cdjrlc.com/

Please check for similar patches first in the future.

Why is seemingly everyone suddenly chasing errors like this?

I don't mind taking such patches since typos like this annoy me, but
having to figure out which patches are duplicates is getting old quickly...

I'll try to go through your patches and see which are duplicates, but
next time I'd appreciate it if you would check this first.

Regards,

	Hans

On 24/08/2022 15:16, Jilin Yuan wrote:
>  Delete the redundant word 'of'.
> 
> Signed-off-by: Jilin Yuan <yuanjilin@...rlc.com>
> ---
>  drivers/media/i2c/cx25840/cx25840-ir.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/cx25840/cx25840-ir.c b/drivers/media/i2c/cx25840/cx25840-ir.c
> index 9d7d1d149f1a..8cef9656c612 100644
> --- a/drivers/media/i2c/cx25840/cx25840-ir.c
> +++ b/drivers/media/i2c/cx25840/cx25840-ir.c
> @@ -196,7 +196,7 @@ static u32 clock_divider_to_resolution(u16 divider)
>  {
>  	/*
>  	 * Resolution is the duration of 1 tick of the readable portion of
> -	 * of the pulse width counter as read from the FIFO.  The two lsb's are
> +	 * the pulse width counter as read from the FIFO.  The two lsb's are
>  	 * not readable, hence the << 2.  This function returns ns.
>  	 */
>  	return DIV_ROUND_CLOSEST((1 << 2)  * ((u32) divider + 1) * 1000,