Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
IronPort-SDR: pZ9VW8cNmFpwhYkyw7M5MggMUriiCB4tvm/wYe1v/dTuut+2ukBdKO5nYlC9nDYOhQAIVdpEVL
 KcQhtaXzDlzRMt+I6pHS3qIpsYTQEGsIOm7UAIC9nEVGGWufG7LWRxqNk3+ptWzB8vSuNDsLNr
 xGDUNdCilahLl/XplTZ7LRTqJTbPXrzd5XkbimDx46zKIGzgaEDSehD1AtUrOZluBeStf54juK
 +CJ4wjiJhxJ6Nnjs+wy/JBRrxY8UiSIpHsWuGAOXGORHt5+mOeYO9PSNjw6odcNqjVv5Uh874f
 tzNOE/ATBWzXOtFSHLH2gLKK
IronPort-SDR: OLi9nhojBQdAALcJup4bhWOHIbMiG/h4+D4RxDfw+oJ7rl6iDzooZJZ9GgV6HxBcWbnbDIvv8+
 P5Ynt6ouR8VqL3gy+iw+Gq+5uk9+qqDHWhEko3eIzjhLnu+LNiNL+kMNtEZYUGu8TY+nbxltVy
 /u4hXtpuJuQ1KtoTQGtZ089LRbCgpKQ4n3dM2y9nOfwF/fW2LOgX/4TaRiwj/KJThjidYfXx3Y
 TrT2ym4ymm3RJVsK5g8t/dW6geW+QDbQWUzX2a9UJHm1/HvAbRZc1OX49LUlKipoD0ANQLDdxu
 BJQ=
WDCIronportException: Internal
Message-ID: <a4948117-dcb1-a27e-941a-811832e9a7c9@opensource.wdc.com>
Date:   Fri, 26 Aug 2022 07:27:20 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.12.0
Subject: Re: [PATCH] drivers/ata: fix repeated words in comments
Content-Language: en-US
To:     Jilin Yuan <yuanjilin@cdjrlc.com>, s.shtylyov@omp.r
Cc:     linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220825123644.9082-1-yuanjilin@cdjrlc.com>
From:   Damien Le Moal <damien.lemoal@opensource.wdc.com>
Organization: Western Digital Research
In-Reply-To: <20220825123644.9082-1-yuanjilin@cdjrlc.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 8/25/22 21:36, Jilin Yuan wrote:
>  Delete the redundant word 'in'.
>  Delete the redundant word 'to'.


> Signed-off-by: Jilin Yuan <yuanjilin@cdjrlc.com>
> ---
>  drivers/ata/libata-eh.c      | 2 +-
>  drivers/ata/pata_macio.c     | 2 +-
>  drivers/ata/sata_dwc_460ex.c | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/ata/libata-eh.c b/drivers/ata/libata-eh.c
> index 7c128c89b454..ca865a95cf24 100644
> --- a/drivers/ata/libata-eh.c
> +++ b/drivers/ata/libata-eh.c
> @@ -863,7 +863,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.

As already commented, this fix is not correct. The sentence should read:

	that EH kicks-in in a timely manner.

The patch title for this fix should be:

ata: libata-eh: fix repeated words in comment

>   *
>   *	LOCKING:
>   *	spin_lock_irqsave(host lock)
> diff --git a/drivers/ata/pata_macio.c b/drivers/ata/pata_macio.c
> index bfea2be2959a..076212fdb9d9 100644
> --- a/drivers/ata/pata_macio.c
> +++ b/drivers/ata/pata_macio.c
> @@ -666,7 +666,7 @@ static u8 pata_macio_bmdma_status(struct ata_port *ap)
>  	 * a multi-block transfer.
>  	 *
>  	 * - The dbdma fifo hasn't yet finished flushing to
> -	 * to system memory when the disk interrupt occurs.
> +	 * system memory when the disk interrupt occurs.
>  	 *

While at it, please remove the above useless extra "*" line.

>  	 */
>  
> diff --git a/drivers/ata/sata_dwc_460ex.c b/drivers/ata/sata_dwc_460ex.c
> index e3263e961045..a4c83a6e5631 100644
> --- a/drivers/ata/sata_dwc_460ex.c
> +++ b/drivers/ata/sata_dwc_460ex.c
> @@ -1087,7 +1087,7 @@ static struct scsi_host_template sata_dwc_sht = {
>  	/*
>  	 * test-only: Currently this driver doesn't handle NCQ
>  	 * correctly. We enable NCQ but set the queue depth to a
> -	 * max of 1. This will get fixed in in a future release.
> +	 * max of 1. This will get fixed in a future release.
>  	 */
>  	.sg_tablesize		= LIBATA_MAX_PRD,
>  	/* .can_queue		= ATA_MAX_QUEUE, */

Please move this second hunk to a different patch with the title:

ata: sata_dwc_460ex: fix repeated words in comment

-- 
Damien Le Moal
Western Digital Research