diff mbox series

[net-nex] mlx4: Address spelling errors

Message ID 20240205-mlx5-codespell-v1-1-63b86dffbb61@kernel.org (mailing list archive)
State Accepted
Commit 6cc9c6fbc79f64a0c5ebdaec77684888665d4ab3
Delegated to: Netdev Maintainers
Headers show
Series [net-nex] mlx4: Address spelling errors | expand

Checks

Context Check Description
netdev/series_format success Single patches do not need cover letters
netdev/tree_selection success Clearly marked for net
netdev/ynl success Generated files up to date; no warnings/errors; no diff in generated;
netdev/fixes_present fail Series targets non-next tree, but doesn't contain any Fixes tags
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 1064 this patch: 1064
netdev/build_tools success No tools touched, skip
netdev/cc_maintainers success CCed 6 of 6 maintainers
netdev/build_clang success Errors and warnings before: 1081 this patch: 1081
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 1081 this patch: 1081
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 150 lines checked
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0
netdev/contest success net-next-2024-02-05--18-00 (tests: 720)

Commit Message

Simon Horman Feb. 5, 2024, 11:51 a.m. UTC
Address spelling errors flagged by codespell.

This patch follows-up on an earlier patch by Colin Ian King,
which addressed a spelling error in a user-visible log message [1].
This patch includes that change.

[1] https://lore.kernel.org/netdev/20231209225135.4055334-1-colin.i.king@gmail.com/

This patch is intended to cover all files under
drivers/net/ethernet/mellanox/mlx4

Signed-off-by: Simon Horman <horms@kernel.org>
---
 drivers/net/ethernet/mellanox/mlx4/cmd.c        | 7 ++++---
 drivers/net/ethernet/mellanox/mlx4/cq.c         | 4 ++--
 drivers/net/ethernet/mellanox/mlx4/en_clock.c   | 4 ++--
 drivers/net/ethernet/mellanox/mlx4/en_netdev.c  | 5 +++--
 drivers/net/ethernet/mellanox/mlx4/en_rx.c      | 2 +-
 drivers/net/ethernet/mellanox/mlx4/en_tx.c      | 2 +-
 drivers/net/ethernet/mellanox/mlx4/eq.c         | 2 +-
 drivers/net/ethernet/mellanox/mlx4/fw_qos.h     | 8 ++++----
 drivers/net/ethernet/mellanox/mlx4/main.c       | 4 ++--
 drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h | 2 +-
 drivers/net/ethernet/mellanox/mlx4/port.c       | 2 +-
 11 files changed, 22 insertions(+), 20 deletions(-)

Comments

Tariq Toukan Feb. 5, 2024, 12:20 p.m. UTC | #1
On 05/02/2024 13:51, Simon Horman wrote:
> Address spelling errors flagged by codespell.
> 
> This patch follows-up on an earlier patch by Colin Ian King,
> which addressed a spelling error in a user-visible log message [1].
> This patch includes that change.
> 
> [1] https://lore.kernel.org/netdev/20231209225135.4055334-1-colin.i.king@gmail.com/
> 
> This patch is intended to cover all files under
> drivers/net/ethernet/mellanox/mlx4
> 
> Signed-off-by: Simon Horman <horms@kernel.org>
> ---
>   drivers/net/ethernet/mellanox/mlx4/cmd.c        | 7 ++++---
>   drivers/net/ethernet/mellanox/mlx4/cq.c         | 4 ++--
>   drivers/net/ethernet/mellanox/mlx4/en_clock.c   | 4 ++--
>   drivers/net/ethernet/mellanox/mlx4/en_netdev.c  | 5 +++--
>   drivers/net/ethernet/mellanox/mlx4/en_rx.c      | 2 +-
>   drivers/net/ethernet/mellanox/mlx4/en_tx.c      | 2 +-
>   drivers/net/ethernet/mellanox/mlx4/eq.c         | 2 +-
>   drivers/net/ethernet/mellanox/mlx4/fw_qos.h     | 8 ++++----
>   drivers/net/ethernet/mellanox/mlx4/main.c       | 4 ++--
>   drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h | 2 +-
>   drivers/net/ethernet/mellanox/mlx4/port.c       | 2 +-
>   11 files changed, 22 insertions(+), 20 deletions(-)
> 
> diff --git a/drivers/net/ethernet/mellanox/mlx4/cmd.c b/drivers/net/ethernet/mellanox/mlx4/cmd.c
> index f5b1f8c7834f..7f20813456e2 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/cmd.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/cmd.c
> @@ -2199,8 +2199,9 @@ static void mlx4_master_do_cmd(struct mlx4_dev *dev, int slave, u8 cmd,
>   	if (cmd != MLX4_COMM_CMD_RESET) {
>   		mlx4_warn(dev, "Turn on internal error to force reset, slave=%d, cmd=0x%x\n",
>   			  slave, cmd);
> -		/* Turn on internal error letting slave reset itself immeditaly,
> -		 * otherwise it might take till timeout on command is passed
> +		/* Turn on internal error letting slave reset itself
> +		 * immediately, otherwise it might take till timeout on
> +		 * command is passed
>   		 */
>   		reply |= ((u32)COMM_CHAN_EVENT_INTERNAL_ERR);
>   	}
> @@ -2954,7 +2955,7 @@ static bool mlx4_valid_vf_state_change(struct mlx4_dev *dev, int port,
>   	dummy_admin.default_vlan = vlan;
>   
>   	/* VF wants to move to other VST state which is valid with current
> -	 * rate limit. Either differnt default vlan in VST or other
> +	 * rate limit. Either different default vlan in VST or other
>   	 * supported QoS priority. Otherwise we don't allow this change when
>   	 * the TX rate is still configured.
>   	 */
> diff --git a/drivers/net/ethernet/mellanox/mlx4/cq.c b/drivers/net/ethernet/mellanox/mlx4/cq.c
> index 4d4f9cf9facb..e130e7259275 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/cq.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/cq.c
> @@ -115,7 +115,7 @@ void mlx4_cq_completion(struct mlx4_dev *dev, u32 cqn)
>   		return;
>   	}
>   
> -	/* Acessing the CQ outside of rcu_read_lock is safe, because
> +	/* Accessing the CQ outside of rcu_read_lock is safe, because
>   	 * the CQ is freed only after interrupt handling is completed.
>   	 */
>   	++cq->arm_sn;
> @@ -137,7 +137,7 @@ void mlx4_cq_event(struct mlx4_dev *dev, u32 cqn, int event_type)
>   		return;
>   	}
>   
> -	/* Acessing the CQ outside of rcu_read_lock is safe, because
> +	/* Accessing the CQ outside of rcu_read_lock is safe, because
>   	 * the CQ is freed only after interrupt handling is completed.
>   	 */
>   	cq->event(cq, event_type);
> diff --git a/drivers/net/ethernet/mellanox/mlx4/en_clock.c b/drivers/net/ethernet/mellanox/mlx4/en_clock.c
> index 9e3b76182088..cd754cd76bde 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/en_clock.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/en_clock.c
> @@ -96,8 +96,8 @@ void mlx4_en_remove_timestamp(struct mlx4_en_dev *mdev)
>   
>   #define MLX4_EN_WRAP_AROUND_SEC	10UL
>   /* By scheduling the overflow check every 5 seconds, we have a reasonably
> - * good chance we wont miss a wrap around.
> - * TOTO: Use a timer instead of a work queue to increase the guarantee.
> + * good chance we won't miss a wrap around.
> + * TODO: Use a timer instead of a work queue to increase the guarantee.
>    */
>   #define MLX4_EN_OVERFLOW_PERIOD (MLX4_EN_WRAP_AROUND_SEC * HZ / 2)
>   
> diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
> index 33bbcced8105..d7da62cda821 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
> @@ -1072,7 +1072,8 @@ static void mlx4_en_do_multicast(struct mlx4_en_priv *priv,
>   				    1, MLX4_MCAST_CONFIG);
>   
>   		/* Update multicast list - we cache all addresses so they won't
> -		 * change while HW is updated holding the command semaphor */
> +		 * change while HW is updated holding the command semaphore
> +		 */
>   		netif_addr_lock_bh(dev);
>   		mlx4_en_cache_mclist(dev);
>   		netif_addr_unlock_bh(dev);
> @@ -1817,7 +1818,7 @@ int mlx4_en_start_port(struct net_device *dev)
>   	    mlx4_en_set_rss_steer_rules(priv))
>   		mlx4_warn(mdev, "Failed setting steering rules\n");
>   
> -	/* Attach rx QP to bradcast address */
> +	/* Attach rx QP to broadcast address */
>   	eth_broadcast_addr(&mc_list[10]);
>   	mc_list[5] = priv->port; /* needed for B0 steering support */
>   	if (mlx4_multicast_attach(mdev->dev, priv->rss_map.indir_qp, mc_list,
> diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
> index a09b6e05337d..eac49657bd07 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
> @@ -762,7 +762,7 @@ int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud
>   		/* Drop packet on bad receive or bad checksum */
>   		if (unlikely((cqe->owner_sr_opcode & MLX4_CQE_OPCODE_MASK) ==
>   						MLX4_CQE_OPCODE_ERROR)) {
> -			en_err(priv, "CQE completed in error - vendor syndrom:%d syndrom:%d\n",
> +			en_err(priv, "CQE completed in error - vendor syndrome:%d syndrome:%d\n",
>   			       ((struct mlx4_err_cqe *)cqe)->vendor_err_syndrome,
>   			       ((struct mlx4_err_cqe *)cqe)->syndrome);
>   			goto next;
> diff --git a/drivers/net/ethernet/mellanox/mlx4/en_tx.c b/drivers/net/ethernet/mellanox/mlx4/en_tx.c
> index 65cb63f6c465..1ddb11cb25f9 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/en_tx.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/en_tx.c
> @@ -992,7 +992,7 @@ netdev_tx_t mlx4_en_xmit(struct sk_buff *skb, struct net_device *dev)
>   		tx_info->ts_requested = 1;
>   	}
>   
> -	/* Prepare ctrl segement apart opcode+ownership, which depends on
> +	/* Prepare ctrl segment apart opcode+ownership, which depends on
>   	 * whether LSO is used */
>   	tx_desc->ctrl.srcrb_flags = priv->ctrl_flags;
>   	if (likely(skb->ip_summed == CHECKSUM_PARTIAL)) {
> diff --git a/drivers/net/ethernet/mellanox/mlx4/eq.c b/drivers/net/ethernet/mellanox/mlx4/eq.c
> index 6598b10a9ff4..9572a45f6143 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/eq.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/eq.c
> @@ -210,7 +210,7 @@ static void slave_event(struct mlx4_dev *dev, u8 slave, struct mlx4_eqe *eqe)
>   
>   	memcpy(s_eqe, eqe, sizeof(struct mlx4_eqe) - 1);
>   	s_eqe->slave_id = slave;
> -	/* ensure all information is written before setting the ownersip bit */
> +	/* ensure all information is written before setting the ownership bit */
>   	dma_wmb();
>   	s_eqe->owner = !!(slave_eq->prod & SLAVE_EVENT_EQ_SIZE) ? 0x0 : 0x80;
>   	++slave_eq->prod;
> diff --git a/drivers/net/ethernet/mellanox/mlx4/fw_qos.h b/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
> index 954b86faac29..40ca29bb928c 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
> +++ b/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
> @@ -44,7 +44,7 @@
>   /* Default supported priorities for VPP allocation */
>   #define MLX4_DEFAULT_QOS_PRIO (0)
>   
> -/* Derived from FW feature definition, 0 is the default vport fo all QPs */
> +/* Derived from FW feature definition, 0 is the default vport for all QPs */
>   #define MLX4_VPP_DEFAULT_VPORT (0)
>   
>   struct mlx4_vport_qos_param {
> @@ -98,7 +98,7 @@ int mlx4_SET_PORT_SCHEDULER(struct mlx4_dev *dev, u8 port, u8 *tc_tx_bw,
>   int mlx4_ALLOCATE_VPP_get(struct mlx4_dev *dev, u8 port,
>   			  u16 *available_vpp, u8 *vpp_p_up);
>   /**
> - * mlx4_ALLOCATE_VPP_set - Distribution of VPPs among differnt priorities.
> + * mlx4_ALLOCATE_VPP_set - Distribution of VPPs among different priorities.
>    * The total number of VPPs assigned to all for a port must not exceed
>    * the value reported by available_vpp in mlx4_ALLOCATE_VPP_get.
>    * VPP allocation is allowed only after the port type has been set,
> @@ -113,7 +113,7 @@ int mlx4_ALLOCATE_VPP_get(struct mlx4_dev *dev, u8 port,
>   int mlx4_ALLOCATE_VPP_set(struct mlx4_dev *dev, u8 port, u8 *vpp_p_up);
>   
>   /**
> - * mlx4_SET_VPORT_QOS_get - Query QoS proporties of a Vport.
> + * mlx4_SET_VPORT_QOS_get - Query QoS properties of a Vport.
>    * Each priority allowed for the Vport is assigned with a share of the BW,
>    * and a BW limitation. This commands query the current QoS values.
>    *
> @@ -128,7 +128,7 @@ int mlx4_SET_VPORT_QOS_get(struct mlx4_dev *dev, u8 port, u8 vport,
>   			   struct mlx4_vport_qos_param *out_param);
>   
>   /**
> - * mlx4_SET_VPORT_QOS_set - Set QoS proporties of a Vport.
> + * mlx4_SET_VPORT_QOS_set - Set QoS properties of a Vport.
>    * QoS parameters can be modified at any time, but must be initialized
>    * before any QP is associated with the VPort.
>    *
> diff --git a/drivers/net/ethernet/mellanox/mlx4/main.c b/drivers/net/ethernet/mellanox/mlx4/main.c
> index 2581226836b5..7b02ff61126d 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/main.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/main.c
> @@ -129,7 +129,7 @@ static const struct mlx4_profile default_profile = {
>   	.num_cq		= 1 << 16,
>   	.num_mcg	= 1 << 13,
>   	.num_mpt	= 1 << 19,
> -	.num_mtt	= 1 << 20, /* It is really num mtt segements */
> +	.num_mtt	= 1 << 20, /* It is really num mtt segments */
>   };
>   
>   static const struct mlx4_profile low_mem_profile = {
> @@ -1508,7 +1508,7 @@ static int mlx4_port_map_set(struct mlx4_dev *dev, struct mlx4_port_map *v2p)
>   			priv->v2p.port1 = port1;
>   			priv->v2p.port2 = port2;
>   		} else {
> -			mlx4_err(dev, "Failed to change port mape: %d\n", err);
> +			mlx4_err(dev, "Failed to change port map: %d\n", err);
>   		}
>   	}
>   
> diff --git a/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h b/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
> index e9cd4bb6f83d..d3d9ec042d2c 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
> +++ b/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
> @@ -112,7 +112,7 @@ struct mlx4_en_stat_out_flow_control_mbox {
>   	__be64 tx_pause_duration;
>   	/* Number of transmitter transitions from XOFF state to XON state */
>   	__be64 tx_pause_transition;
> -	/* Reserverd */
> +	/* Reserved */
>   	__be64 reserved[2];
>   };
>   
> diff --git a/drivers/net/ethernet/mellanox/mlx4/port.c b/drivers/net/ethernet/mellanox/mlx4/port.c
> index 256a06b3c096..4e43f4a7d246 100644
> --- a/drivers/net/ethernet/mellanox/mlx4/port.c
> +++ b/drivers/net/ethernet/mellanox/mlx4/port.c
> @@ -2118,7 +2118,7 @@ static void mlx4_qsfp_eeprom_params_set(u8 *i2c_addr, u8 *page_num, u16 *offset)
>    * @data: output buffer to put the requested data into.
>    *
>    * Reads cable module eeprom data, puts the outcome data into
> - * data pointer paramer.
> + * data pointer parameter.
>    * Returns num of read bytes on success or a negative error
>    * code.
>    */
> 
> 

Thanks for your patch.
Reviewed-by: Tariq Toukan <tariqt@nvidia.com>

I'm ignoring the branch name typo in the mail subject :)
[PATCH net-nex]
Randy Dunlap Feb. 5, 2024, 9:28 p.m. UTC | #2
Hi Simon,

On 2/5/24 03:51, Simon Horman wrote:
> Address spelling errors flagged by codespell.
> 
> This patch follows-up on an earlier patch by Colin Ian King,
> which addressed a spelling error in a user-visible log message [1].
> This patch includes that change.
> 
> [1] https://lore.kernel.org/netdev/20231209225135.4055334-1-colin.i.king@gmail.com/
> 
> This patch is intended to cover all files under
> drivers/net/ethernet/mellanox/mlx4
> 
> Signed-off-by: Simon Horman <horms@kernel.org>
> ---
>  drivers/net/ethernet/mellanox/mlx4/cmd.c        | 7 ++++---
>  drivers/net/ethernet/mellanox/mlx4/cq.c         | 4 ++--
>  drivers/net/ethernet/mellanox/mlx4/en_clock.c   | 4 ++--
>  drivers/net/ethernet/mellanox/mlx4/en_netdev.c  | 5 +++--
>  drivers/net/ethernet/mellanox/mlx4/en_rx.c      | 2 +-
>  drivers/net/ethernet/mellanox/mlx4/en_tx.c      | 2 +-
>  drivers/net/ethernet/mellanox/mlx4/eq.c         | 2 +-
>  drivers/net/ethernet/mellanox/mlx4/fw_qos.h     | 8 ++++----
>  drivers/net/ethernet/mellanox/mlx4/main.c       | 4 ++--
>  drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h | 2 +-
>  drivers/net/ethernet/mellanox/mlx4/port.c       | 2 +-
>  11 files changed, 22 insertions(+), 20 deletions(-)
> 

Reviewed-by: Randy Dunlap <rdunlap@infradead.org>

Thanks.
patchwork-bot+netdevbpf@kernel.org Feb. 7, 2024, 3 a.m. UTC | #3
Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Mon, 05 Feb 2024 11:51:57 +0000 you wrote:
> Address spelling errors flagged by codespell.
> 
> This patch follows-up on an earlier patch by Colin Ian King,
> which addressed a spelling error in a user-visible log message [1].
> This patch includes that change.
> 
> [1] https://lore.kernel.org/netdev/20231209225135.4055334-1-colin.i.king@gmail.com/
> 
> [...]

Here is the summary with links:
  - [net-nex] mlx4: Address spelling errors
    https://git.kernel.org/netdev/net-next/c/6cc9c6fbc79f

You are awesome, thank you!
diff mbox series

Patch

diff --git a/drivers/net/ethernet/mellanox/mlx4/cmd.c b/drivers/net/ethernet/mellanox/mlx4/cmd.c
index f5b1f8c7834f..7f20813456e2 100644
--- a/drivers/net/ethernet/mellanox/mlx4/cmd.c
+++ b/drivers/net/ethernet/mellanox/mlx4/cmd.c
@@ -2199,8 +2199,9 @@  static void mlx4_master_do_cmd(struct mlx4_dev *dev, int slave, u8 cmd,
 	if (cmd != MLX4_COMM_CMD_RESET) {
 		mlx4_warn(dev, "Turn on internal error to force reset, slave=%d, cmd=0x%x\n",
 			  slave, cmd);
-		/* Turn on internal error letting slave reset itself immeditaly,
-		 * otherwise it might take till timeout on command is passed
+		/* Turn on internal error letting slave reset itself
+		 * immediately, otherwise it might take till timeout on
+		 * command is passed
 		 */
 		reply |= ((u32)COMM_CHAN_EVENT_INTERNAL_ERR);
 	}
@@ -2954,7 +2955,7 @@  static bool mlx4_valid_vf_state_change(struct mlx4_dev *dev, int port,
 	dummy_admin.default_vlan = vlan;
 
 	/* VF wants to move to other VST state which is valid with current
-	 * rate limit. Either differnt default vlan in VST or other
+	 * rate limit. Either different default vlan in VST or other
 	 * supported QoS priority. Otherwise we don't allow this change when
 	 * the TX rate is still configured.
 	 */
diff --git a/drivers/net/ethernet/mellanox/mlx4/cq.c b/drivers/net/ethernet/mellanox/mlx4/cq.c
index 4d4f9cf9facb..e130e7259275 100644
--- a/drivers/net/ethernet/mellanox/mlx4/cq.c
+++ b/drivers/net/ethernet/mellanox/mlx4/cq.c
@@ -115,7 +115,7 @@  void mlx4_cq_completion(struct mlx4_dev *dev, u32 cqn)
 		return;
 	}
 
-	/* Acessing the CQ outside of rcu_read_lock is safe, because
+	/* Accessing the CQ outside of rcu_read_lock is safe, because
 	 * the CQ is freed only after interrupt handling is completed.
 	 */
 	++cq->arm_sn;
@@ -137,7 +137,7 @@  void mlx4_cq_event(struct mlx4_dev *dev, u32 cqn, int event_type)
 		return;
 	}
 
-	/* Acessing the CQ outside of rcu_read_lock is safe, because
+	/* Accessing the CQ outside of rcu_read_lock is safe, because
 	 * the CQ is freed only after interrupt handling is completed.
 	 */
 	cq->event(cq, event_type);
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_clock.c b/drivers/net/ethernet/mellanox/mlx4/en_clock.c
index 9e3b76182088..cd754cd76bde 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_clock.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_clock.c
@@ -96,8 +96,8 @@  void mlx4_en_remove_timestamp(struct mlx4_en_dev *mdev)
 
 #define MLX4_EN_WRAP_AROUND_SEC	10UL
 /* By scheduling the overflow check every 5 seconds, we have a reasonably
- * good chance we wont miss a wrap around.
- * TOTO: Use a timer instead of a work queue to increase the guarantee.
+ * good chance we won't miss a wrap around.
+ * TODO: Use a timer instead of a work queue to increase the guarantee.
  */
 #define MLX4_EN_OVERFLOW_PERIOD (MLX4_EN_WRAP_AROUND_SEC * HZ / 2)
 
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
index 33bbcced8105..d7da62cda821 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_netdev.c
@@ -1072,7 +1072,8 @@  static void mlx4_en_do_multicast(struct mlx4_en_priv *priv,
 				    1, MLX4_MCAST_CONFIG);
 
 		/* Update multicast list - we cache all addresses so they won't
-		 * change while HW is updated holding the command semaphor */
+		 * change while HW is updated holding the command semaphore
+		 */
 		netif_addr_lock_bh(dev);
 		mlx4_en_cache_mclist(dev);
 		netif_addr_unlock_bh(dev);
@@ -1817,7 +1818,7 @@  int mlx4_en_start_port(struct net_device *dev)
 	    mlx4_en_set_rss_steer_rules(priv))
 		mlx4_warn(mdev, "Failed setting steering rules\n");
 
-	/* Attach rx QP to bradcast address */
+	/* Attach rx QP to broadcast address */
 	eth_broadcast_addr(&mc_list[10]);
 	mc_list[5] = priv->port; /* needed for B0 steering support */
 	if (mlx4_multicast_attach(mdev->dev, priv->rss_map.indir_qp, mc_list,
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_rx.c b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
index a09b6e05337d..eac49657bd07 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_rx.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_rx.c
@@ -762,7 +762,7 @@  int mlx4_en_process_rx_cq(struct net_device *dev, struct mlx4_en_cq *cq, int bud
 		/* Drop packet on bad receive or bad checksum */
 		if (unlikely((cqe->owner_sr_opcode & MLX4_CQE_OPCODE_MASK) ==
 						MLX4_CQE_OPCODE_ERROR)) {
-			en_err(priv, "CQE completed in error - vendor syndrom:%d syndrom:%d\n",
+			en_err(priv, "CQE completed in error - vendor syndrome:%d syndrome:%d\n",
 			       ((struct mlx4_err_cqe *)cqe)->vendor_err_syndrome,
 			       ((struct mlx4_err_cqe *)cqe)->syndrome);
 			goto next;
diff --git a/drivers/net/ethernet/mellanox/mlx4/en_tx.c b/drivers/net/ethernet/mellanox/mlx4/en_tx.c
index 65cb63f6c465..1ddb11cb25f9 100644
--- a/drivers/net/ethernet/mellanox/mlx4/en_tx.c
+++ b/drivers/net/ethernet/mellanox/mlx4/en_tx.c
@@ -992,7 +992,7 @@  netdev_tx_t mlx4_en_xmit(struct sk_buff *skb, struct net_device *dev)
 		tx_info->ts_requested = 1;
 	}
 
-	/* Prepare ctrl segement apart opcode+ownership, which depends on
+	/* Prepare ctrl segment apart opcode+ownership, which depends on
 	 * whether LSO is used */
 	tx_desc->ctrl.srcrb_flags = priv->ctrl_flags;
 	if (likely(skb->ip_summed == CHECKSUM_PARTIAL)) {
diff --git a/drivers/net/ethernet/mellanox/mlx4/eq.c b/drivers/net/ethernet/mellanox/mlx4/eq.c
index 6598b10a9ff4..9572a45f6143 100644
--- a/drivers/net/ethernet/mellanox/mlx4/eq.c
+++ b/drivers/net/ethernet/mellanox/mlx4/eq.c
@@ -210,7 +210,7 @@  static void slave_event(struct mlx4_dev *dev, u8 slave, struct mlx4_eqe *eqe)
 
 	memcpy(s_eqe, eqe, sizeof(struct mlx4_eqe) - 1);
 	s_eqe->slave_id = slave;
-	/* ensure all information is written before setting the ownersip bit */
+	/* ensure all information is written before setting the ownership bit */
 	dma_wmb();
 	s_eqe->owner = !!(slave_eq->prod & SLAVE_EVENT_EQ_SIZE) ? 0x0 : 0x80;
 	++slave_eq->prod;
diff --git a/drivers/net/ethernet/mellanox/mlx4/fw_qos.h b/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
index 954b86faac29..40ca29bb928c 100644
--- a/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
+++ b/drivers/net/ethernet/mellanox/mlx4/fw_qos.h
@@ -44,7 +44,7 @@ 
 /* Default supported priorities for VPP allocation */
 #define MLX4_DEFAULT_QOS_PRIO (0)
 
-/* Derived from FW feature definition, 0 is the default vport fo all QPs */
+/* Derived from FW feature definition, 0 is the default vport for all QPs */
 #define MLX4_VPP_DEFAULT_VPORT (0)
 
 struct mlx4_vport_qos_param {
@@ -98,7 +98,7 @@  int mlx4_SET_PORT_SCHEDULER(struct mlx4_dev *dev, u8 port, u8 *tc_tx_bw,
 int mlx4_ALLOCATE_VPP_get(struct mlx4_dev *dev, u8 port,
 			  u16 *available_vpp, u8 *vpp_p_up);
 /**
- * mlx4_ALLOCATE_VPP_set - Distribution of VPPs among differnt priorities.
+ * mlx4_ALLOCATE_VPP_set - Distribution of VPPs among different priorities.
  * The total number of VPPs assigned to all for a port must not exceed
  * the value reported by available_vpp in mlx4_ALLOCATE_VPP_get.
  * VPP allocation is allowed only after the port type has been set,
@@ -113,7 +113,7 @@  int mlx4_ALLOCATE_VPP_get(struct mlx4_dev *dev, u8 port,
 int mlx4_ALLOCATE_VPP_set(struct mlx4_dev *dev, u8 port, u8 *vpp_p_up);
 
 /**
- * mlx4_SET_VPORT_QOS_get - Query QoS proporties of a Vport.
+ * mlx4_SET_VPORT_QOS_get - Query QoS properties of a Vport.
  * Each priority allowed for the Vport is assigned with a share of the BW,
  * and a BW limitation. This commands query the current QoS values.
  *
@@ -128,7 +128,7 @@  int mlx4_SET_VPORT_QOS_get(struct mlx4_dev *dev, u8 port, u8 vport,
 			   struct mlx4_vport_qos_param *out_param);
 
 /**
- * mlx4_SET_VPORT_QOS_set - Set QoS proporties of a Vport.
+ * mlx4_SET_VPORT_QOS_set - Set QoS properties of a Vport.
  * QoS parameters can be modified at any time, but must be initialized
  * before any QP is associated with the VPort.
  *
diff --git a/drivers/net/ethernet/mellanox/mlx4/main.c b/drivers/net/ethernet/mellanox/mlx4/main.c
index 2581226836b5..7b02ff61126d 100644
--- a/drivers/net/ethernet/mellanox/mlx4/main.c
+++ b/drivers/net/ethernet/mellanox/mlx4/main.c
@@ -129,7 +129,7 @@  static const struct mlx4_profile default_profile = {
 	.num_cq		= 1 << 16,
 	.num_mcg	= 1 << 13,
 	.num_mpt	= 1 << 19,
-	.num_mtt	= 1 << 20, /* It is really num mtt segements */
+	.num_mtt	= 1 << 20, /* It is really num mtt segments */
 };
 
 static const struct mlx4_profile low_mem_profile = {
@@ -1508,7 +1508,7 @@  static int mlx4_port_map_set(struct mlx4_dev *dev, struct mlx4_port_map *v2p)
 			priv->v2p.port1 = port1;
 			priv->v2p.port2 = port2;
 		} else {
-			mlx4_err(dev, "Failed to change port mape: %d\n", err);
+			mlx4_err(dev, "Failed to change port map: %d\n", err);
 		}
 	}
 
diff --git a/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h b/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
index e9cd4bb6f83d..d3d9ec042d2c 100644
--- a/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
+++ b/drivers/net/ethernet/mellanox/mlx4/mlx4_stats.h
@@ -112,7 +112,7 @@  struct mlx4_en_stat_out_flow_control_mbox {
 	__be64 tx_pause_duration;
 	/* Number of transmitter transitions from XOFF state to XON state */
 	__be64 tx_pause_transition;
-	/* Reserverd */
+	/* Reserved */
 	__be64 reserved[2];
 };
 
diff --git a/drivers/net/ethernet/mellanox/mlx4/port.c b/drivers/net/ethernet/mellanox/mlx4/port.c
index 256a06b3c096..4e43f4a7d246 100644
--- a/drivers/net/ethernet/mellanox/mlx4/port.c
+++ b/drivers/net/ethernet/mellanox/mlx4/port.c
@@ -2118,7 +2118,7 @@  static void mlx4_qsfp_eeprom_params_set(u8 *i2c_addr, u8 *page_num, u16 *offset)
  * @data: output buffer to put the requested data into.
  *
  * Reads cable module eeprom data, puts the outcome data into
- * data pointer paramer.
+ * data pointer parameter.
  * Returns num of read bytes on success or a negative error
  * code.
  */