On 07/09/2026 05:46, Hemanth Selam wrote: > Fix typos in comments, reported by scripts/checkpatch.pl using the > misspelling list in scripts/spelling.txt. Only touches comments, no code > changes. > > Assisted-by: Cursor:claude-opus-5 > Signed-off-by: Hemanth Selam <[email protected]> ... > diff --git a/drivers/net/ethernet/sfc/mcdi_pcol.h > b/drivers/net/ethernet/sfc/mcdi_pcol.h
mcdi_pcol.h (and the Siena version) are generated from firmware definitions, so any changes you make would be overwritten next time they are regenerated. I have forwarded these changes internally, so they should reach the firmware team. > index b9866e389e6d..b00e7c100566 100644 > --- a/drivers/net/ethernet/sfc/mcdi_pcol.h > +++ b/drivers/net/ethernet/sfc/mcdi_pcol.h > @@ -5736,7 +5736,7 @@ > /* Enum values, see field(s): */ > /* MC_CMD_ETH_TECH/TECH */ > /* Pause abilities to advertise during auto-negotiation. Valid when auto- > - * negotation is enabled and MC_CMD_SET_MAC_IN/FCTL is set to > + * negotiation is enabled and MC_CMD_SET_MAC_IN/FCTL is set to > * MC_CMD_FCNTL_AUTO. If auto-negotiation is disabled the driver must > * explicitly configure pause mode with MC_CMD_SET_MAC. > */ > @@ -10173,7 +10173,7 @@ > * configuration. > */ > #define MC_CMD_GET_FIXED_PORT_PROPERTIES_IN_LEN 4 > -/* Handle to the port to from which to retreive properties */ > +/* Handle to the port to from which to retrieve properties */ > #define MC_CMD_GET_FIXED_PORT_PROPERTIES_IN_PORT_HANDLE_OFST 0 > #define MC_CMD_GET_FIXED_PORT_PROPERTIES_IN_PORT_HANDLE_LEN 4 > > @@ -13547,7 +13547,7 @@ > * an ancestor of the current user (see MC_CMD_SET_VI_USER). Note that LL > * queues require this to be called after allocation but before > initialisation > * of the queue. TLP options of a queue are fixed after queue is initialised, > - * with the values set to current global value or they can be overriden using > + * with the values set to current global value or they can be overridden > using > * this command. At LL queue allocation, all overrides are cleared. > */ > #define MC_CMD_SET_VI_TLP_PROCESSING 0xb1 > @@ -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 > diff --git a/drivers/net/ethernet/sfc/siena/mcdi_pcol.h > b/drivers/net/ethernet/sfc/siena/mcdi_pcol.h > index b81b0aa460d2..4d1da698c524 100644 > --- a/drivers/net/ethernet/sfc/siena/mcdi_pcol.h > +++ b/drivers/net/ethernet/sfc/siena/mcdi_pcol.h > @@ -6861,7 +6861,7 @@ > /***********************************/ > /* MC_CMD_CLP > * Perform a CLP related operation, see SF-110495-PS for details of CLP > - * processing. This command has been extended to accomodate the requirements > of > + * processing. This command has been extended to accommodate the > requirements of > * different manufacturers which are to be found in SF-119187-TC, > SF-119186-TC, > * SF-120509-TC and SF-117282-PS. > */ > @@ -16293,7 +16293,7 @@ > > /***********************************/ > /* MC_CMD_GET_LICENSED_V3_FEATURE_STATES > - * Query the state of an one or more licensed features. (Note that the actual > + * Query the state of a one or more licensed features. (Note that the actual Neither this nor the original text makes grammatical sense; I think the article 'an' should be deleted entirely rather than replaced with 'a'. > * state may be invalidated by the MC_CMD_LICENSING_V3 OP_UPDATE_LICENSE > * operation or a reboot of the MC.) Used for V3 licensing (Medford) > */ > @@ -16798,7 +16798,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 The other sfc changes outside of pcol look fine. -ed

