[PATCH v2] ata: Fix typos in the comment

Yan Zhen posted 1 patch 2 months ago
There is a newer version of this series
drivers/ata/ahci.c           | 2 +-
drivers/ata/ahci_imx.c       | 2 +-
drivers/ata/ahci_xgene.c     | 2 +-
drivers/ata/libata-acpi.c    | 4 ++--
drivers/ata/pata_it8213.c    | 2 +-
drivers/ata/pata_octeon_cf.c | 2 +-
drivers/ata/pata_oldpiix.c   | 2 +-
drivers/ata/pata_radisys.c   | 2 +-
8 files changed, 9 insertions(+), 9 deletions(-)
[PATCH v2] ata: Fix typos in the comment
Posted by Yan Zhen 2 months ago
Correctly spelled comments make it easier for the reader to understand
the code.

Fix typos:
'multipe' ==> 'multiple',
'Paremeters' ==> 'Parameters',
'recieved' ==> 'received',
'realted' ==> 'related',
'evaulated' ==> 'evaluated',
'programing' ==> 'programming',
'coninue' ==> 'continue',
'Decriptor' ==> 'Descriptor',
'Lable' ==> 'Label',
'transmiting' ==> 'transmitting',
'correspondance' ==> 'correspondence',
'claculation' ==> 'calculation',
'everone' ==> 'everyone',
'contruct' ==> 'construct'.

Signed-off-by: Yan Zhen <yanzhen@vivo.com>
---

Change in v2:
-In the subject, replace "typo" with "typos".

 drivers/ata/ahci.c           | 2 +-
 drivers/ata/ahci_imx.c       | 2 +-
 drivers/ata/ahci_xgene.c     | 2 +-
 drivers/ata/libata-acpi.c    | 4 ++--
 drivers/ata/pata_it8213.c    | 2 +-
 drivers/ata/pata_octeon_cf.c | 2 +-
 drivers/ata/pata_oldpiix.c   | 2 +-
 drivers/ata/pata_radisys.c   | 2 +-
 8 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
index 45f63b09828a..2d3d3d67b4d9 100644
--- a/drivers/ata/ahci.c
+++ b/drivers/ata/ahci.c
@@ -1676,7 +1676,7 @@ static int ahci_init_msi(struct pci_dev *pdev, unsigned int n_ports,
 	/*
 	 * If number of MSIs is less than number of ports then Sharing Last
 	 * Message mode could be enforced. In this case assume that advantage
-	 * of multipe MSIs is negated and use single MSI mode instead.
+	 * of multiple MSIs is negated and use single MSI mode instead.
 	 */
 	if (n_ports > 1) {
 		nvec = pci_alloc_irq_vectors(pdev, n_ports, INT_MAX,
diff --git a/drivers/ata/ahci_imx.c b/drivers/ata/ahci_imx.c
index 6f955e9105e8..357aba1d5929 100644
--- a/drivers/ata/ahci_imx.c
+++ b/drivers/ata/ahci_imx.c
@@ -511,7 +511,7 @@ static int imx_sata_enable(struct ahci_host_priv *hpriv)
 
 	if (imxpriv->type == AHCI_IMX6Q || imxpriv->type == AHCI_IMX6QP) {
 		/*
-		 * set PHY Paremeters, two steps to configure the GPR13,
+		 * set PHY Parameters, two steps to configure the GPR13,
 		 * one write for rest of parameters, mask of first write
 		 * is 0x07ffffff, and the other one write for setting
 		 * the mpll_clk_en.
diff --git a/drivers/ata/ahci_xgene.c b/drivers/ata/ahci_xgene.c
index 81a1d838c0fc..7e6b4db8a1c2 100644
--- a/drivers/ata/ahci_xgene.c
+++ b/drivers/ata/ahci_xgene.c
@@ -534,7 +534,7 @@ static int xgene_ahci_softreset(struct ata_link *link, unsigned int *class,
 
 /**
  * xgene_ahci_handle_broken_edge_irq - Handle the broken irq.
- * @host: Host that recieved the irq
+ * @host: Host that received the irq
  * @irq_masked: HOST_IRQ_STAT value
  *
  * For hardware with broken edge trigger latch
diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c
index d36e71f475ab..b7f0bf795521 100644
--- a/drivers/ata/libata-acpi.c
+++ b/drivers/ata/libata-acpi.c
@@ -86,7 +86,7 @@ static void ata_acpi_detach_device(struct ata_port *ap, struct ata_device *dev)
  * @dev: ATA device ACPI event occurred (can be NULL)
  * @event: ACPI event which occurred
  *
- * All ACPI bay / device realted events end up in this function.  If
+ * All ACPI bay / device related events end up in this function.  If
  * the event is port-wide @dev is NULL.  If the event is specific to a
  * device, @dev points to it.
  *
@@ -832,7 +832,7 @@ void ata_acpi_on_resume(struct ata_port *ap)
 				dev->flags |= ATA_DFLAG_ACPI_PENDING;
 		}
 	} else {
-		/* SATA _GTF needs to be evaulated after _SDD and
+		/* SATA _GTF needs to be evaluated after _SDD and
 		 * there's no reason to evaluate IDE _GTF early
 		 * without _STM.  Clear cache and schedule _GTF.
 		 */
diff --git a/drivers/ata/pata_it8213.c b/drivers/ata/pata_it8213.c
index b7ac56103c8a..9cbe2132ce59 100644
--- a/drivers/ata/pata_it8213.c
+++ b/drivers/ata/pata_it8213.c
@@ -81,7 +81,7 @@ static void it8213_set_piomode (struct ata_port *ap, struct ata_device *adev)
 	int control = 0;
 
 	/*
-	 *	See Intel Document 298600-004 for the timing programing rules
+	 *	See Intel Document 298600-004 for the timing programming rules
 	 *	for PIIX/ICH. The 8213 is a clone so very similar
 	 */
 
diff --git a/drivers/ata/pata_octeon_cf.c b/drivers/ata/pata_octeon_cf.c
index 0bb9607e7348..dce24806a052 100644
--- a/drivers/ata/pata_octeon_cf.c
+++ b/drivers/ata/pata_octeon_cf.c
@@ -183,7 +183,7 @@ static void octeon_cf_set_piomode(struct ata_port *ap, struct ata_device *dev)
 	reg_tim.s.ale = 0;
 	/* Not used */
 	reg_tim.s.page = 0;
-	/* Time after IORDY to coninue to assert the data */
+	/* Time after IORDY to continue to assert the data */
 	reg_tim.s.wait = 0;
 	/* Time to wait to complete the cycle. */
 	reg_tim.s.pause = pause;
diff --git a/drivers/ata/pata_oldpiix.c b/drivers/ata/pata_oldpiix.c
index dca82d92b004..3d01b7000e41 100644
--- a/drivers/ata/pata_oldpiix.c
+++ b/drivers/ata/pata_oldpiix.c
@@ -70,7 +70,7 @@ static void oldpiix_set_piomode (struct ata_port *ap, struct ata_device *adev)
 	int control = 0;
 
 	/*
-	 *	See Intel Document 298600-004 for the timing programing rules
+	 *	See Intel Document 298600-004 for the timing programming rules
 	 *	for PIIX/ICH. Note that the early PIIX does not have the slave
 	 *	timing port at 0x44.
 	 */
diff --git a/drivers/ata/pata_radisys.c b/drivers/ata/pata_radisys.c
index 84b001097093..40ef8072c159 100644
--- a/drivers/ata/pata_radisys.c
+++ b/drivers/ata/pata_radisys.c
@@ -45,7 +45,7 @@ static void radisys_set_piomode (struct ata_port *ap, struct ata_device *adev)
 	int control = 0;
 
 	/*
-	 *	See Intel Document 298600-004 for the timing programing rules
+	 *	See Intel Document 298600-004 for the timing programming rules
 	 *	for PIIX/ICH. Note that the early PIIX does not have the slave
 	 *	timing port at 0x44. The Radisys is a relative of the PIIX
 	 *	but not the same so be careful.
-- 
2.34.1
Re: [PATCH v2] ata: Fix typos in the comment
Posted by yanzhen 2 months ago
在 2024/9/27 9:33, Yan Zhen 写道:
> Correctly spelled comments make it easier for the reader to understand
> the code.
>
> Fix typos:
> 'multipe' ==> 'multiple',
> 'Paremeters' ==> 'Parameters',
> 'recieved' ==> 'received',
> 'realted' ==> 'related',
> 'evaulated' ==> 'evaluated',
> 'programing' ==> 'programming',
> 'coninue' ==> 'continue',
> 'Decriptor' ==> 'Descriptor',
> 'Lable' ==> 'Label',
> 'transmiting' ==> 'transmitting',
> 'correspondance' ==> 'correspondence',
> 'claculation' ==> 'calculation',
> 'everone' ==> 'everyone',
> 'contruct' ==> 'construct'.

Very sorry. A mistake was made in the typos description, please ignore 
this patch. I will submit v3.

> Signed-off-by: Yan Zhen <yanzhen@vivo.com>
> ---
>
> Change in v2:
> -In the subject, replace "typo" with "typos".
>
>   drivers/ata/ahci.c           | 2 +-
>   drivers/ata/ahci_imx.c       | 2 +-
>   drivers/ata/ahci_xgene.c     | 2 +-
>   drivers/ata/libata-acpi.c    | 4 ++--
>   drivers/ata/pata_it8213.c    | 2 +-
>   drivers/ata/pata_octeon_cf.c | 2 +-
>   drivers/ata/pata_oldpiix.c   | 2 +-
>   drivers/ata/pata_radisys.c   | 2 +-
>   8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
> index 45f63b09828a..2d3d3d67b4d9 100644
> --- a/drivers/ata/ahci.c
> +++ b/drivers/ata/ahci.c
> @@ -1676,7 +1676,7 @@ static int ahci_init_msi(struct pci_dev *pdev, unsigned int n_ports,
>   	/*
>   	 * If number of MSIs is less than number of ports then Sharing Last
>   	 * Message mode could be enforced. In this case assume that advantage
> -	 * of multipe MSIs is negated and use single MSI mode instead.
> +	 * of multiple MSIs is negated and use single MSI mode instead.
>   	 */
>   	if (n_ports > 1) {
>   		nvec = pci_alloc_irq_vectors(pdev, n_ports, INT_MAX,
> diff --git a/drivers/ata/ahci_imx.c b/drivers/ata/ahci_imx.c
> index 6f955e9105e8..357aba1d5929 100644
> --- a/drivers/ata/ahci_imx.c
> +++ b/drivers/ata/ahci_imx.c
> @@ -511,7 +511,7 @@ static int imx_sata_enable(struct ahci_host_priv *hpriv)
>   
>   	if (imxpriv->type == AHCI_IMX6Q || imxpriv->type == AHCI_IMX6QP) {
>   		/*
> -		 * set PHY Paremeters, two steps to configure the GPR13,
> +		 * set PHY Parameters, two steps to configure the GPR13,
>   		 * one write for rest of parameters, mask of first write
>   		 * is 0x07ffffff, and the other one write for setting
>   		 * the mpll_clk_en.
> diff --git a/drivers/ata/ahci_xgene.c b/drivers/ata/ahci_xgene.c
> index 81a1d838c0fc..7e6b4db8a1c2 100644
> --- a/drivers/ata/ahci_xgene.c
> +++ b/drivers/ata/ahci_xgene.c
> @@ -534,7 +534,7 @@ static int xgene_ahci_softreset(struct ata_link *link, unsigned int *class,
>   
>   /**
>    * xgene_ahci_handle_broken_edge_irq - Handle the broken irq.
> - * @host: Host that recieved the irq
> + * @host: Host that received the irq
>    * @irq_masked: HOST_IRQ_STAT value
>    *
>    * For hardware with broken edge trigger latch
> diff --git a/drivers/ata/libata-acpi.c b/drivers/ata/libata-acpi.c
> index d36e71f475ab..b7f0bf795521 100644
> --- a/drivers/ata/libata-acpi.c
> +++ b/drivers/ata/libata-acpi.c
> @@ -86,7 +86,7 @@ static void ata_acpi_detach_device(struct ata_port *ap, struct ata_device *dev)
>    * @dev: ATA device ACPI event occurred (can be NULL)
>    * @event: ACPI event which occurred
>    *
> - * All ACPI bay / device realted events end up in this function.  If
> + * All ACPI bay / device related events end up in this function.  If
>    * the event is port-wide @dev is NULL.  If the event is specific to a
>    * device, @dev points to it.
>    *
> @@ -832,7 +832,7 @@ void ata_acpi_on_resume(struct ata_port *ap)
>   				dev->flags |= ATA_DFLAG_ACPI_PENDING;
>   		}
>   	} else {
> -		/* SATA _GTF needs to be evaulated after _SDD and
> +		/* SATA _GTF needs to be evaluated after _SDD and
>   		 * there's no reason to evaluate IDE _GTF early
>   		 * without _STM.  Clear cache and schedule _GTF.
>   		 */
> diff --git a/drivers/ata/pata_it8213.c b/drivers/ata/pata_it8213.c
> index b7ac56103c8a..9cbe2132ce59 100644
> --- a/drivers/ata/pata_it8213.c
> +++ b/drivers/ata/pata_it8213.c
> @@ -81,7 +81,7 @@ static void it8213_set_piomode (struct ata_port *ap, struct ata_device *adev)
>   	int control = 0;
>   
>   	/*
> -	 *	See Intel Document 298600-004 for the timing programing rules
> +	 *	See Intel Document 298600-004 for the timing programming rules
>   	 *	for PIIX/ICH. The 8213 is a clone so very similar
>   	 */
>   
> diff --git a/drivers/ata/pata_octeon_cf.c b/drivers/ata/pata_octeon_cf.c
> index 0bb9607e7348..dce24806a052 100644
> --- a/drivers/ata/pata_octeon_cf.c
> +++ b/drivers/ata/pata_octeon_cf.c
> @@ -183,7 +183,7 @@ static void octeon_cf_set_piomode(struct ata_port *ap, struct ata_device *dev)
>   	reg_tim.s.ale = 0;
>   	/* Not used */
>   	reg_tim.s.page = 0;
> -	/* Time after IORDY to coninue to assert the data */
> +	/* Time after IORDY to continue to assert the data */
>   	reg_tim.s.wait = 0;
>   	/* Time to wait to complete the cycle. */
>   	reg_tim.s.pause = pause;
> diff --git a/drivers/ata/pata_oldpiix.c b/drivers/ata/pata_oldpiix.c
> index dca82d92b004..3d01b7000e41 100644
> --- a/drivers/ata/pata_oldpiix.c
> +++ b/drivers/ata/pata_oldpiix.c
> @@ -70,7 +70,7 @@ static void oldpiix_set_piomode (struct ata_port *ap, struct ata_device *adev)
>   	int control = 0;
>   
>   	/*
> -	 *	See Intel Document 298600-004 for the timing programing rules
> +	 *	See Intel Document 298600-004 for the timing programming rules
>   	 *	for PIIX/ICH. Note that the early PIIX does not have the slave
>   	 *	timing port at 0x44.
>   	 */
> diff --git a/drivers/ata/pata_radisys.c b/drivers/ata/pata_radisys.c
> index 84b001097093..40ef8072c159 100644
> --- a/drivers/ata/pata_radisys.c
> +++ b/drivers/ata/pata_radisys.c
> @@ -45,7 +45,7 @@ static void radisys_set_piomode (struct ata_port *ap, struct ata_device *adev)
>   	int control = 0;
>   
>   	/*
> -	 *	See Intel Document 298600-004 for the timing programing rules
> +	 *	See Intel Document 298600-004 for the timing programming rules
>   	 *	for PIIX/ICH. Note that the early PIIX does not have the slave
>   	 *	timing port at 0x44. The Radisys is a relative of the PIIX
>   	 *	but not the same so be careful.