[PATCH][next] net/mlx5: Fix spelling mistakes in mlx5_core_dbg message and comments

Colin Ian King posted 1 patch 7 months, 4 weeks ago
drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
[PATCH][next] net/mlx5: Fix spelling mistakes in mlx5_core_dbg message and comments
Posted by Colin Ian King 7 months, 4 weeks ago
There is a spelling mistake in a mlx5_core_dbg and two spelling mistakes
in comment blocks. Fix them.

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c b/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
index 2c5f850c31f6..40024cfa3099 100644
--- a/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
+++ b/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
@@ -148,7 +148,7 @@ int mlx5_set_msix_vec_count(struct mlx5_core_dev *dev, int function_id,
  * Free the IRQ and other resources such as rmap from the system.
  * BUT doesn't free or remove reference from mlx5.
  * This function is very important for the shutdown flow, where we need to
- * cleanup system resoruces but keep mlx5 objects alive,
+ * cleanup system resources but keep mlx5 objects alive,
  * see mlx5_irq_table_free_irqs().
  */
 static void mlx5_system_free_irq(struct mlx5_irq *irq)
@@ -588,7 +588,7 @@ static void irq_pool_free(struct mlx5_irq_pool *pool)
 	struct mlx5_irq *irq;
 	unsigned long index;
 
-	/* There are cases in which we are destrying the irq_table before
+	/* There are cases in which we are destroying the irq_table before
 	 * freeing all the IRQs, fast teardown for example. Hence, free the irqs
 	 * which might not have been freed.
 	 */
@@ -617,7 +617,7 @@ static int irq_pools_init(struct mlx5_core_dev *dev, int sf_vec, int pcif_vec,
 	if (!mlx5_sf_max_functions(dev))
 		return 0;
 	if (sf_vec < MLX5_IRQ_VEC_COMP_BASE_SF) {
-		mlx5_core_dbg(dev, "Not enught IRQs for SFs. SF may run at lower performance\n");
+		mlx5_core_dbg(dev, "Not enough IRQs for SFs. SF may run at lower performance\n");
 		return 0;
 	}
 
-- 
2.49.0
Re: [PATCH][next] net/mlx5: Fix spelling mistakes in mlx5_core_dbg message and comments
Posted by Mark Bloch 7 months, 4 weeks ago

On 18/04/2025 16:57, Colin Ian King wrote:
> There is a spelling mistake in a mlx5_core_dbg and two spelling mistakes
> in comment blocks. Fix them.
> 
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
>  drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c b/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
> index 2c5f850c31f6..40024cfa3099 100644
> --- a/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
> +++ b/drivers/net/ethernet/mellanox/mlx5/core/pci_irq.c
> @@ -148,7 +148,7 @@ int mlx5_set_msix_vec_count(struct mlx5_core_dev *dev, int function_id,
>   * Free the IRQ and other resources such as rmap from the system.
>   * BUT doesn't free or remove reference from mlx5.
>   * This function is very important for the shutdown flow, where we need to
> - * cleanup system resoruces but keep mlx5 objects alive,
> + * cleanup system resources but keep mlx5 objects alive,
>   * see mlx5_irq_table_free_irqs().
>   */
>  static void mlx5_system_free_irq(struct mlx5_irq *irq)
> @@ -588,7 +588,7 @@ static void irq_pool_free(struct mlx5_irq_pool *pool)
>  	struct mlx5_irq *irq;
>  	unsigned long index;
>  
> -	/* There are cases in which we are destrying the irq_table before
> +	/* There are cases in which we are destroying the irq_table before
>  	 * freeing all the IRQs, fast teardown for example. Hence, free the irqs
>  	 * which might not have been freed.
>  	 */
> @@ -617,7 +617,7 @@ static int irq_pools_init(struct mlx5_core_dev *dev, int sf_vec, int pcif_vec,
>  	if (!mlx5_sf_max_functions(dev))
>  		return 0;
>  	if (sf_vec < MLX5_IRQ_VEC_COMP_BASE_SF) {
> -		mlx5_core_dbg(dev, "Not enught IRQs for SFs. SF may run at lower performance\n");
> +		mlx5_core_dbg(dev, "Not enough IRQs for SFs. SF may run at lower performance\n");
>  		return 0;
>  	}
>  

Acked-by: Mark Bloch <mbloch@nvidia.com>

Thanks