Fix typos in network driver code comments
Subsystems (4)
v2 cover Fix typos in network driver code comments
Fix various minor typos and mispellings in 3 different driver subdirectories in drivers/net/ethernet
---
v2:
* Separate patch into patch-set of 3 separated by subdirectory
* Fix all typos in the subdirectory rather than just "Software"
* Sent to net-next rather than net
v1:https://lore.kernel.org/all/[email protected]/t/#u
Yicong Hui (3):
net/benet: Fix typos in driver code comments net/micrel: Fix typos in micrel driver code comments net/xen-netback: Fix mispelling of "Software" as "Softare"
drivers/net/ethernet/emulex/benet/be.h | 8 ++++---- drivers/net/ethernet/emulex/benet/be_cmds.c | 6 +++--- drivers/net/ethernet/emulex/benet/be_cmds.h | 6 +++--- drivers/net/ethernet/emulex/benet/be_ethtool.c | 6 +++--- drivers/net/ethernet/emulex/benet/be_hw.h | 6 +++--- drivers/net/ethernet/emulex/benet/be_main.c | 16 ++++++++-------- drivers/net/ethernet/micrel/ks8842.c | 4 ++-- drivers/net/ethernet/micrel/ks8851_common.c | 2 +- drivers/net/ethernet/micrel/ks8851_spi.c | 4 ++-- drivers/net/ethernet/micrel/ksz884x.c | 4 ++-- drivers/net/xen-netback/hash.c | 2 +- 11 files changed, 32 insertions(+), 32 deletions(-)
Signature · 1 line
2.52.0
PATCH v2 1/3 net/benet: Fix typos in driver code comments
Fix various typos and misspellings in code comments in the drivers/net/ethernet/emulex directory
--- drivers/net/ethernet/emulex/benet/be.h | 8 ++++---- drivers/net/ethernet/emulex/benet/be_cmds.c | 6 +++--- drivers/net/ethernet/emulex/benet/be_cmds.h | 6 +++--- drivers/net/ethernet/emulex/benet/be_ethtool.c | 6 +++--- drivers/net/ethernet/emulex/benet/be_hw.h | 6 +++--- drivers/net/ethernet/emulex/benet/be_main.c | 16 ++++++++-------- 6 files changed, 24 insertions(+), 24 deletions(-)
drivers/net/ethernet/emulex/benet/be.h +4 -4
@@ -672,7 +672,7 @@ struct be_adapter {struct be_error_recovery error_recovery; }; -/* Used for defered FW config cmds. Add fields to this struct as reqd */ +/* Used for deferred FW config cmds. Add fields to this struct as reqd */ struct be_cmd_work {struct work_struct work;struct be_adapter *adapter;
@@ -700,19 +700,19 @@ struct be_cmd_work {#define be_max_rxqs (adapter) (adapter->res.max_rx_qs)/* Max number of EQs available for the function (NIC + RoCE (if enabled)) */ #define be_max_func_eqs (adapter) (adapter->res.max_evt_qs) -/* Max number of EQs available avaialble only for NIC */ +/* Max number of EQs available only for NIC */ #define be_max_nic_eqs (adapter) (adapter->res.max_nic_evt_qs)#define be_if_cap_flags (adapter) (adapter->res.if_cap_flags)#define be_max_pf_pool_rss_tables (adapter) \ (adapter->pool_res.max_rss_tables) -/* Max irqs avaialble for NIC */ +/* Max irqs available for NIC */ #define be_max_irqs (adapter) \ (min_t(u16, be_max_nic_eqs(adapter), num_online_cpus()))/* Max irqs *needed* for RX queues */ static inline u16 be_max_rx_irqs (struct be_adapter *adapter) { -/* If no RSS, need atleast one irq for def-RXQ */ +/* If no RSS, need at least one irq for def-RXQ */ u16 num =max_t (u16,be_max_rss (adapter),1 );return min_t (u16, num,be_max_irqs (adapter));
drivers/net/ethernet/emulex/benet/be_cmds.c +3 -3
@@ -1941,7 +1941,7 @@ int be_cmd_modify_eqd(struct be_adapter *adapter, struct be_set_eqd *set_eqd,return 0 ; } -/* Uses sycnhronous mcc */ +/* Uses synchronous mcc */ int be_cmd_vlan_config (struct be_adapter *adapter,u32 if_id,u16 *vtag_array,u32 num,u32 domain) {
@@ -2035,7 +2035,7 @@ int be_cmd_rx_filter(struct be_adapter *adapter, u32 flags, u32 value)return __be_cmd_rx_filter (adapter, flags, value); } -/* Uses synchrounous mcc */ +/* Uses synchronous mcc */ int be_cmd_set_flow_control (struct be_adapter *adapter,u32 tx_fc,u32 rx_fc) {struct be_mcc_wrb *wrb;
@@ -2074,7 +2074,7 @@ int be_cmd_set_flow_control(struct be_adapter *adapter, u32 tx_fc, u32 rx_fc)return status; } -/* Uses sycn mcc */ +/* Uses sync mcc */ int be_cmd_get_flow_control (struct be_adapter *adapter,u32 *tx_fc,u32 *rx_fc) {struct be_mcc_wrb *wrb;
drivers/net/ethernet/emulex/benet/be_cmds.h +3 -3
@@ -1134,14 +1134,14 @@ struct be_cmd_resp_get_fw_version {u8 fw_on_flash_version_string[FW_VER_LEN ]; } __packed; -/******************** Set Flow Contrl *******************/ +/******************** Set Flow Control *******************/ struct be_cmd_req_set_flow_control {struct be_cmd_req_hdr hdr;u16 tx_flow_control;u16 rx_flow_control; } __packed; -/******************** Get Flow Contrl *******************/ +/******************** Get Flow Control *******************/ struct be_cmd_req_get_flow_control {struct be_cmd_req_hdr hdr;u32 rsvd;
@@ -2069,7 +2069,7 @@ struct be_cmd_resp_get_stats_v2 {struct be_hw_stats_v2 hw_stats; }; -/************** get fat capabilites *******************/ +/************** get fat capabilities *******************/ #define MAX_MODULES 27#define MAX_MODES 4#define MODE_UART 0
drivers/net/ethernet/emulex/benet/be_ethtool.c +3 -3
@@ -142,7 +142,7 @@ static const struct be_ethtool_stat et_rx_stats[] = {* to HW. */ {DRVSTAT_RX_INFO(rx_post_fail)}, - /* Recevied packets dropped due to skb allocation failure */ + /* Received packets dropped due to skb allocation failure */ {DRVSTAT_RX_INFO(rx_drops_no_skbs)}, /* Received packets dropped due to lack of available fetched buffers * posted by the driver.
@@ -189,7 +189,7 @@ static const struct be_ethtool_stat et_tx_stats[] = { {DRVSTAT_TX_INFO(tx_bytes)}, {DRVSTAT_TX_INFO(tx_pkts)}, {DRVSTAT_TX_INFO(tx_vxlan_offload_pkts)}, - /* Number of skbs queued for trasmission by the driver */ + /* Number of skbs queued for transmission by the driver */ {DRVSTAT_TX_INFO(tx_reqs)}, /* Number of times the TX queue was stopped due to lack * of spaces in the TXQ.
@@ -1222,7 +1222,7 @@ static void be_get_channels(struct net_device *netdev, ch->tx_count = adapter->num_tx_qs - ch->combined_count; ch->max_combined =be_max_qp_irqs (adapter); -/* The user must create atleast one combined channel */ +/* The user must create at least one combined channel */ ch->max_rx =be_max_rx_irqs (adapter) -1 ; ch->max_tx =be_max_tx_irqs (adapter) -1 ; }
drivers/net/ethernet/emulex/benet/be_hw.h +3 -3
@@ -16,7 +16,7 @@* The software must write this register twice to post any command. First, * it writes the register with hi=1 and the upper bits of the physical address * for the MAILBOX structure. Software must poll the ready bit until this -* is acknowledged. Then, sotware writes the register with hi=0 with the lower +* is acknowledged. Then, software writes the register with hi=0 with the lower * bits in the address. It must poll the ready bit until the command is * complete. Upon completion, the MAILBOX will contain a valid completion * queue entry.
@@ -27,7 +27,7 @@#define MPU_EP_CONTROL 0 -/********** MPU semphore: used for SH & BE *************/ +/********** MPU semaphore: used for SH & BE *************/ #define SLIPORT_SOFTRESET_OFFSET 0x5c/* CSR BAR offset */ #define SLIPORT_SEMAPHORE_OFFSET_BEx 0xac/* CSR BAR offset */ #define SLIPORT_SEMAPHORE_OFFSET_SH 0x94/* PCI-CFG offset */
@@ -39,7 +39,7 @@/* Soft Reset register masks */ #define SLIPORT_SOFTRESET_SR_MASK 0x00000080/* SR bit */ -/* MPU semphore POST stage values */ +/* MPU semaphore POST stage values */ #define POST_STAGE_AWAITING_HOST_RDY 0x1/* FW awaiting goahead from host */ #define POST_STAGE_HOST_RDY 0x2/* Host has given go-ahed to FW */ #define POST_STAGE_BE_RESET 0x3/* Host wants to reset chip */
drivers/net/ethernet/emulex/benet/be_main.c +8 -8
@@ -61,7 +61,7 @@ static const struct pci_device_id be_dev_ids[] = { };MODULE_DEVICE_TABLE (pci, be_dev_ids); -/* Workqueue used by all functions for defering cmd calls to the adapter */ +/* Workqueue used by all functions for deferring cmd calls to the adapter */ static struct workqueue_struct *be_wq;/* UE Status Low CSR */
@@ -1129,7 +1129,7 @@ static struct sk_buff *be_lancer_xmit_workarounds(struct be_adapter *adapter,struct iphdr *ip;/* For padded packets, BE HW modifies tot_len field in IP header -* incorrecly when VLAN tag is inserted by HW. +* incorrectly when VLAN tag is inserted by HW. * For padded packets, Lancer computes incorrect checksum. */ eth_hdr_len = ntohs(skb->protocol) == ETH_P_8021Q ?
@@ -2568,7 +2568,7 @@ static struct be_rx_compl_info *be_rx_compl_get(struct be_rx_obj *rxo) rxcp->vlanf =0 ; } -/* As the compl has been parsed, reset it; we wont touch it again */ +/* As the compl has been parsed, reset it; we won't touch it again */ compl->dw[offsetof(struct amap_eth_rx_compl_v1 , valid) /32 ] =0 ;queue_tail_inc (&rxo->cq);
@@ -2727,7 +2727,7 @@ static struct be_tx_compl_info *be_tx_compl_get(struct be_adapter *adapter,if (txcp->status) {if (lancer_chip (adapter)) {lancer_update_tx_err (txo, txcp->status); -/* Reset the adapter incase of TSO, +/* Reset the adapter in case of TSO, * SGE or Parity error */ if (txcp->status == LANCER_TX_COMP_LSO_ERR ||
@@ -3125,7 +3125,7 @@ static int be_rx_cqs_create(struct be_adapter *adapter) adapter->num_rss_qs =min (adapter->num_evt_qs, adapter->cfg_num_rx_irqs); -/* We'll use RSS only if atleast 2 RSS rings are supported. */ +/* We'll use RSS only if at least 2 RSS rings are supported. */ if (adapter->num_rss_qs <2 ) adapter->num_rss_qs =0 ;
@@ -3167,7 +3167,7 @@ static irqreturn_t be_intx(int irq, void *dev)/* IRQ is not expected when NAPI is scheduled as the EQ * will not be armed. * But, this can happen on Lancer INTx where it takes -* a while to de-assert INTx or in BE2 where occasionaly +* a while to de-assert INTx or in BE2 where occasionally * an interrupt may be raised even when EQ is unarmed. * If NAPI is already scheduled, then counting & notifying * events will orphan them.
@@ -4415,7 +4415,7 @@ static void be_setup_init(struct be_adapter *adapter)/* HW supports only MAX_PORT_RSS_TABLES RSS Policy Tables per port. * However, this HW limitation is not exposed to the host via any SLI cmd. * As a result, in the case of SRIOV and in particular multi-partition configs -* the driver needs to calcuate a proportional share of RSS Tables per PF-pool +* the driver needs to calculate a proportional share of RSS Tables per PF-pool * for distribution between the VFs. This self-imposed limit will determine the * no: of VFs for which RSS can be enabled. */
@@ -4519,7 +4519,7 @@ static int be_get_resources(struct be_adapter *adapter)if (status)return status; -/* If a deafault RXQ must be created, we'll use up one RSSQ*/ +/* If a default RXQ must be created, we'll use up one RSSQ*/ if (res.max_rss_qs && res.max_rss_qs == res.max_rx_qs && !(res.if_cap_flags &BE_IF_FLAGS_DEFQ_RSS )) res.max_rss_qs -=1 ;
Signature · 1 line
2.52.0
PATCH v2 2/3 net/micrel: Fix typos in micrel driver code comments
In reply to Yicong HuiFix various typos and misspellings in code comments in the drivers/net/ethernet/micrel directory
--- drivers/net/ethernet/micrel/ks8842.c | 4 ++-- drivers/net/ethernet/micrel/ks8851_common.c | 2 +- drivers/net/ethernet/micrel/ks8851_spi.c | 4 ++-- drivers/net/ethernet/micrel/ksz884x.c | 4 ++-- 4 files changed, 7 insertions(+), 7 deletions(-)
drivers/net/ethernet/micrel/ks8842.c +2 -2
@@ -242,7 +242,7 @@ static void ks8842_reset(struct ks8842_adapter *adapter)msleep (10 );iowrite16 (0 , adapter->hw_addr +REG_GRR ); }else { -/* The KS8842 goes haywire when doing softare reset +/* The KS8842 goes haywire when doing software reset * a work around in the timberdale IP is implemented to * do a hardware reset instead ks8842_write16(adapter, 3, 1, REG_GRR);
@@ -312,7 +312,7 @@ static void ks8842_reset_hw(struct ks8842_adapter *adapter)/* aggressive back off in half duplex */ ks8842_enable_bits (adapter,32 ,1 <<8 ,REG_SGCR1 ); -/* enable no excessive collison drop */ +/* enable no excessive collision drop */ ks8842_enable_bits (adapter,32 ,1 <<3 ,REG_SGCR2 );/* Enable port 1 force flow control / back pressure / transmit / recv */
drivers/net/ethernet/micrel/ks8851_common.c +1 -1
@@ -480,7 +480,7 @@ static int ks8851_net_open(struct net_device *dev)* ks8851_net_stop - close network device * @dev: The device being closed. * -* Called to close down a network device which has been active. Cancell any +* Called to close down a network device which has been active. Cancel any * work, shutdown the RX and TX process and then place the chip into a low * power state whilst it is not being used. */
drivers/net/ethernet/micrel/ks8851_spi.c +2 -2
@@ -39,7 +39,7 @@ static int msg_enable;* * The @lock ensures that the chip is protected when certain operations are * in progress. When the read or write packet transfer is in progress, most -* of the chip registers are not ccessible until the transfer is finished and +* of the chip registers are not accessible until the transfer is finished and * the DMA has been de-asserted. */ struct ks8851_net_spi {
@@ -298,7 +298,7 @@ static unsigned int calc_txlen(unsigned int len)/** * ks8851_tx_work - process tx packet(s) -* @work: The work strucutre what was scheduled. +* @work: The work structure what was scheduled. * * This is called when a number of packets have been scheduled for * transmission and need to be sent to the device.
drivers/net/ethernet/micrel/ksz884x.c +2 -2
@@ -1166,7 +1166,7 @@ struct ksz_port_info {* @tx_cfg: Cached transmit control settings. * @rx_cfg: Cached receive control settings. * @intr_mask: Current interrupt mask. -* @intr_set: Current interrup set. +* @intr_set: Current interrupt set. * @intr_blocked: Interrupt blocked. * @rx_desc_info: Receive descriptor information. * @tx_desc_info: Transmit descriptor information.
@@ -2096,7 +2096,7 @@ static void sw_dis_prio_rate(struct ksz_hw *hw, int port) }/** -* sw_init_prio_rate - initialize switch prioirty rate +* sw_init_prio_rate - initialize switch priority rate * @hw: The hardware instance. * * This routine initializes the priority rate function of the switch.
Signature · 1 line
2.52.0
PATCH v2 3/3 net/xen-netback: Fix mispelling of "Software" as "Softare"
In reply to Yicong HuiFix misspelling of "software" as "softare" in xen-netback code comment.
--- drivers/net/xen-netback/hash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/net/xen-netback/hash.c +1 -1
@@ -3,7 +3,7 @@* * This program is free software; you can redistribute it and/or * modify it under the terms of the GNU General Public License version 2 -* as published by the Free Softare Foundation; or, when distributed +* as published by the Free Software Foundation; or, when distributed * separately from the Linux kernel or incorporated into other * software packages, subject to the following license: *
Signature · 1 line
2.52.0
This series was applied to netdev/net-next.git (main) by Jakub Kicinski <[email protected]>:
Hello:
This series was applied to netdev/net-next.git (main) by Jakub Kicinski <[email protected]>:
Quoted 10 lines · On Sun, 18 Jan 2026 12:09:58 +0000 you wrote:
Fix various minor typos and mispellings in 3 different driver subdirectories in
drivers/net/ethernet---
v2:
* Separate patch into patch-set of 3 separated by subdirectory
* Fix all typos in the subdirectory rather than just "Software"
* Sent to net-next rather than net[...]
Here is the summary with links:
- [net-next,v2,1/3] net/benet: Fix typos in driver code comments
https://git.kernel.org/netdev/net-next/c/1cbc1fe0a434 - [net-next,v2,2/3] net/micrel: Fix typos in micrel driver code comments https://git.kernel.org/netdev/net-next/c/c2b733da93bf - [net-next,v2,3/3] net/xen-netback: Fix mispelling of "Software" as "Softare" https://git.kernel.org/netdev/net-next/c/f117205ad70b
You are awesome, thank you!
Signature · 2 lines
Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html