[PATCH] drivers/scsi: fix repeated words in comments

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

 Delete the redundant word 'of'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/st.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/st.c b/drivers/scsi/st.c
index 56a093a90b92..e65f05615b7f 100644
--- a/drivers/scsi/st.c
+++ b/drivers/scsi/st.c
@@ -4660,7 +4660,7 @@ static DEVICE_ATTR_RO(read_cnt);
 /**
  * read_byte_cnt_show - return read byte count - tape drives
  * may use blocks less than 512 bytes this gives the raw byte count of
- * of data read from the tape drive.
+ * data read from the tape drive.
  * @dev: struct device
  * @attr: attribute structure
  * @buf: buffer to return formatted data in
-- 
2.36.1

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

Delete the redundant word 'the'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/dc395x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
index 670a836a6ba1..a7db373d5e2c 100644
--- a/drivers/scsi/dc395x.c
+++ b/drivers/scsi/dc395x.c
@@ -2051,7 +2051,7 @@ static void data_in_phase0(struct AdapterCtlBlk *acb, struct ScsiReqBlk *srb,
 		}
 		/*
 		 * KG: We should wait for the DMA FIFO to be empty ...
-		 * but: it would be better to wait first for the SCSI FIFO and then the
+		 * but: it would be better to wait first for the SCSI FIFO and then
 		 * the DMA FIFO to become empty? How do we know, that the device not already
 		 * sent data to the FIFO in a MsgIn phase, eg.?
 		 */
-- 
2.36.1

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

Delete the redundant word 'the'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/NCR5380.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/NCR5380.c b/drivers/scsi/NCR5380.c
index dece7d9eb4d3..36f8133ad7d8 100644
--- a/drivers/scsi/NCR5380.c
+++ b/drivers/scsi/NCR5380.c
@@ -857,7 +857,7 @@ static void NCR5380_dma_complete(struct Scsi_Host *instance)
  * Checking for bus reset by reading RST is futile because of interrupt
  * latency, but a bus reset will reset chip logic. Checking for parity error
  * is unnecessary because that interrupt is never enabled. A Loss of BSY
- * condition will clear DMA Mode. We can tell when this occurs because the
+ * condition will clear DMA Mode. We can tell when this occurs because
  * the Busy Monitor interrupt is enabled together with DMA Mode.
  */
 
-- 
2.36.1

Re: [PATCH] drivers/scsi: fix repeated words in comments

[Wei Liu]

On Thu, Sep 08, 2022 at 09:07:54PM +0800, wangjianli wrote:
> Delete the redundant word 'to'.
> 
> Signed-off-by: wangjianli <wangjianli@cdjrlc.com>

I changed the commit message a bit and pushed this patch to hyperv-next.
Thanks.

> ---
>  drivers/scsi/storvsc_drv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
> index 573f89eade3b..9f7c71a8c80e 100644
> --- a/drivers/scsi/storvsc_drv.c
> +++ b/drivers/scsi/storvsc_drv.c
> @@ -2059,7 +2059,7 @@ static int storvsc_probe(struct hv_device *device,
>  err_out2:
>  	/*
>  	 * Once we have connected with the host, we would need to
> -	 * to invoke storvsc_dev_remove() to rollback this state and
> +	 * invoke storvsc_dev_remove() to rollback this state and
>  	 * this call also frees up the stor_device; hence the jump around
>  	 * err_out1 label.
>  	 */
> -- 
> 2.36.1
> 

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

Delete the redundant word 'to'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/storvsc_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/storvsc_drv.c b/drivers/scsi/storvsc_drv.c
index 573f89eade3b..9f7c71a8c80e 100644
--- a/drivers/scsi/storvsc_drv.c
+++ b/drivers/scsi/storvsc_drv.c
@@ -2059,7 +2059,7 @@ static int storvsc_probe(struct hv_device *device,
 err_out2:
 	/*
 	 * Once we have connected with the host, we would need to
-	 * to invoke storvsc_dev_remove() to rollback this state and
+	 * invoke storvsc_dev_remove() to rollback this state and
 	 * this call also frees up the stor_device; hence the jump around
 	 * err_out1 label.
 	 */
-- 
2.36.1

Re: [PATCH] drivers/scsi: fix repeated words in comments

[Conor.Dooley]

On 08/09/2022 13:24, wangjianli wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Delete the redundant word 'in'.
> 
> Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
> ---
>   drivers/scsi/dc395x.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
> index 670a836a6ba1..aea82007dd61 100644
> --- a/drivers/scsi/dc395x.c
> +++ b/drivers/scsi/dc395x.c
> @@ -4226,7 +4226,7 @@ static void adapter_init_params(struct AdapterCtlBlk *acb)
>    * values that we have already stored in the adapter instance. There's
>    * some mention that a lot of these are deprecated, so we won't use
>    * them (we'll use the ones in the adapter instance) but we'll fill
> - * them in in case something else needs them.
> + * them in case something else needs them.

Hey wangjianli,
Thanks for your patch.

This is not a typo, is it? With your "fix", it does not make sense
anymore. If you are running a tool, you should probably validate
the outputs before sending patches.
Thanks,
Conor.

>    *
>    * The eeprom structure, irq and io ports in the adapter need to have
>    * been set before calling this function.
> --
> 2.36.1
> 

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

Delete the redundant word 'in'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/wd33c93.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/wd33c93.c b/drivers/scsi/wd33c93.c
index 3fe562047d85..a087e681805d 100644
--- a/drivers/scsi/wd33c93.c
+++ b/drivers/scsi/wd33c93.c
@@ -821,7 +821,7 @@ wd33c93_intr(struct Scsi_Host *instance)
 /* After starting a DMA transfer, the next interrupt
  * is guaranteed to be in response to completion of
  * the transfer. Since the Amiga DMA hardware runs in
- * in an open-ended fashion, it needs to be told when
+ * an open-ended fashion, it needs to be told when
  * to stop; do that here if D_DMA_RUNNING is true.
  * Also, we have to update 'this_residual' and 'ptr'
  * based on the contents of the TRANSFER_COUNT register,
-- 
2.36.1

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

Delete the redundant word 'in'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/dc395x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
index 670a836a6ba1..aea82007dd61 100644
--- a/drivers/scsi/dc395x.c
+++ b/drivers/scsi/dc395x.c
@@ -4226,7 +4226,7 @@ static void adapter_init_params(struct AdapterCtlBlk *acb)
  * values that we have already stored in the adapter instance. There's
  * some mention that a lot of these are deprecated, so we won't use
  * them (we'll use the ones in the adapter instance) but we'll fill
- * them in in case something else needs them.
+ * them in case something else needs them.
  *
  * The eeprom structure, irq and io ports in the adapter need to have
  * been set before calling this function.
-- 
2.36.1

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

 Delete the redundant word 'in'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/wd33c93.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/wd33c93.c b/drivers/scsi/wd33c93.c
index 3fe562047d85..a087e681805d 100644
--- a/drivers/scsi/wd33c93.c
+++ b/drivers/scsi/wd33c93.c
@@ -821,7 +821,7 @@ wd33c93_intr(struct Scsi_Host *instance)
 /* After starting a DMA transfer, the next interrupt
  * is guaranteed to be in response to completion of
  * the transfer. Since the Amiga DMA hardware runs in
- * in an open-ended fashion, it needs to be told when
+ * an open-ended fashion, it needs to be told when
  * to stop; do that here if D_DMA_RUNNING is true.
  * Also, we have to update 'this_residual' and 'ptr'
  * based on the contents of the TRANSFER_COUNT register,
-- 
2.36.1

[PATCH] drivers/scsi: fix repeated words in comments

[wangjianli]

 Delete the redundant word 'in'.

Signed-off-by: wangjianli <wangjianli@cdjrlc.com>
---
 drivers/scsi/dc395x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/dc395x.c b/drivers/scsi/dc395x.c
index 670a836a6ba1..aea82007dd61 100644
--- a/drivers/scsi/dc395x.c
+++ b/drivers/scsi/dc395x.c
@@ -4226,7 +4226,7 @@ static void adapter_init_params(struct AdapterCtlBlk *acb)
  * values that we have already stored in the adapter instance. There's
  * some mention that a lot of these are deprecated, so we won't use
  * them (we'll use the ones in the adapter instance) but we'll fill
- * them in in case something else needs them.
+ * them in case something else needs them.
  *
  * The eeprom structure, irq and io ports in the adapter need to have
  * been set before calling this function.
-- 
2.36.1