From: Hans Verkuil <hverkuil@xs4all.nl>
To: Jilin Yuan <yuanjilin@cdjrlc.com>, mchehab@kernel.org
Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] i2c/cx25840: fix repeated words in comments
Date: Wed, 24 Aug 2022 17:01:20 +0200 [thread overview]
Message-ID: <42384aba-d519-1a98-80dc-00a997e7243f@xs4all.nl> (raw)
In-Reply-To: <20220824131619.54027-1-yuanjilin@cdjrlc.com>
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@cdjrlc.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,
next prev parent reply other threads:[~2022-08-24 15:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-24 13:16 Jilin Yuan
2022-08-24 15:01 ` Hans Verkuil [this message]
-- strict thread matches above, loose matches on Subject: below --
2022-08-21 15:15 wangjianli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=42384aba-d519-1a98-80dc-00a997e7243f@xs4all.nl \
--to=hverkuil@xs4all.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=yuanjilin@cdjrlc.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®