[PATCH] sfc: Fix spelling mistake

Mohammed EL Kadiri posted 1 patch 1 week, 5 days ago
drivers/net/ethernet/sfc/mcdi_pcol.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] sfc: Fix spelling mistake
Posted by Mohammed EL Kadiri 1 week, 5 days ago
In the Solarflare MCDI protocol header, the word 'address' was
misspelled as 'adress' in a comment. Correcting this improves code
readability and documentation consistency within the networking
subsystem.

Signed-off-by: Mohammed EL Kadiri <med08elkadiri@gmail.com>
---
 drivers/net/ethernet/sfc/mcdi_pcol.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
index b9866e389e6d..a9a7076c00e4 100644
--- a/drivers/net/ethernet/sfc/mcdi_pcol.h
+++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
@@ -21387,7 +21387,7 @@
 #define          MC_CMD_PRIVILEGE_MASK_IN_GRP_ALL_MULTICAST 0x200 /* enum */
 #define          MC_CMD_PRIVILEGE_MASK_IN_GRP_PROMISCUOUS 0x400 /* enum */
 /* enum: Allows to set the TX packets' source MAC address to any arbitrary MAC
- * adress.
+ * address.
  */
 #define          MC_CMD_PRIVILEGE_MASK_IN_GRP_MAC_SPOOFING_TX 0x800
 /* enum: Privilege that allows a Function to change the MAC address configured
-- 
2.43.0
Re: [PATCH] sfc: Fix spelling mistake
Posted by Kohei Enju 1 week, 5 days ago
On 03/22 13:16, Mohammed EL Kadiri wrote:
> In the Solarflare MCDI protocol header, the word 'address' was
> misspelled as 'adress' in a comment. Correcting this improves code
> readability and documentation consistency within the networking
> subsystem.
> 
> Signed-off-by: Mohammed EL Kadiri <med08elkadiri@gmail.com>
> ---
>  drivers/net/ethernet/sfc/mcdi_pcol.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h b/drivers/net/ethernet/sfc/mcdi_pcol.h
> index b9866e389e6d..a9a7076c00e4 100644
> --- a/drivers/net/ethernet/sfc/mcdi_pcol.h
> +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h
> @@ -21387,7 +21387,7 @@
>  #define          MC_CMD_PRIVILEGE_MASK_IN_GRP_ALL_MULTICAST 0x200 /* enum */
>  #define          MC_CMD_PRIVILEGE_MASK_IN_GRP_PROMISCUOUS 0x400 /* enum */
>  /* enum: Allows to set the TX packets' source MAC address to any arbitrary MAC
> - * adress.
> + * address.
>   */

FWIW, I believe the same comment and typo exist in the siena variant as
well:
  drivers/net/ethernet/sfc/siena/mcdi_pcol.h
Maybe worth fixing there as well?