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

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The patch blindly removes a valid occurrence of the word 'in', creating a grammatical error in the comment.
--

commit 82b3223c0ee5c7c14e26327dc44e83c5226cc9b8
Author: Hemanth Selam <hemanth.selam@gmail.com>

ata: fix repeated words in comments

This commit drops repeated words in comments reported by checkpatch.pl.
It only touches comments and makes no code changes.

> diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
> index c154103d892c3..bdf25c535c07c 100644
> --- a/drivers/ata/libata-eh.c
> +++ b/drivers/ata/libata-eh.c
> @@ -923,7 +923,7 @@ void ata_eh_fastdrain_timerfn(struct timer_list *t)
>   *
>   *	Set ATA_PFLAG_EH_PENDING and activate fast drain if @fastdrain
>   *	is non-zero and EH wasn't pending before.  Fast drain ensures
> - *	that EH kicks in in timely manner.
> + *	that EH kicks in timely manner.

[Severity: Low]
Does this accidentally remove a valid occurrence of the word "in"?

The original text uses "in" twice for two different grammatical purposes: the
first is part of the phrasal verb "kick in", and the second is the preposition
for "in [a] timely manner".

Removing the second one leaves the sentence missing a preposition.

Would it be better to change this to "EH kicks in in a timely manner" or 
"EH kicks in promptly" instead of just dropping the word?

[ ... ]

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260904123207.36203-1-hemanth.selam@gmail.com?part=2