Re: [PATCH] wifi: fix typos in comments

[Christian Lamparter]

On 9/4/26 2:43 PM, 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 <hemanth.selam@xxxxxxxxx>
Sure, for the p54 and carl9170 parts:

Acked-by: Christian Lamparter <chunkeey@xxxxxxxxx>

Does anyone know what ever happened to the trivial-patches?
https://www.kernel.org/pub/linux/kernel/people//rusty/trivial/template-index.html
Is this still going?

> ---
>   drivers/net/wireless/ath/carl9170/fwcmd.h                 | 2 +-
>   drivers/net/wireless/ath/carl9170/main.c                  | 8 ++++----
>   drivers/net/wireless/broadcom/b43/b43.h                   | 8 ++++----
>   drivers/net/wireless/intersil/p54/p54pci.c                | 2 +-
>
> diff --git a/drivers/net/wireless/ath/carl9170/fwcmd.h b/drivers/net/wireless/ath/carl9170/fwcmd.h
> index e5bcc364f088..40554e4bd149 100644
> --- a/drivers/net/wireless/ath/carl9170/fwcmd.h
> +++ b/drivers/net/wireless/ath/carl9170/fwcmd.h
> @@ -69,7 +69,7 @@ enum carl9170_cmd_oids {
>   	CARL9170_CMD_FREQ_START		= 0x23,
>   	CARL9170_CMD_PSM		= 0x24,
>   
> -	/* Asychronous command flag */
> +	/* Asynchronous command flag */
>   	CARL9170_CMD_ASYNC_FLAG		= 0x40,
>   	CARL9170_CMD_WREG_ASYNC		= (CARL9170_CMD_WREG |
>   					   CARL9170_CMD_ASYNC_FLAG),
> diff --git a/drivers/net/wireless/ath/carl9170/main.c b/drivers/net/wireless/ath/carl9170/main.c
> index 61c7a1288743..21d8798dedb5 100644
> --- a/drivers/net/wireless/ath/carl9170/main.c
> +++ b/drivers/net/wireless/ath/carl9170/main.c
> @@ -575,7 +575,7 @@ static int carl9170_init_interface(struct ar9170 *ar,
>   	 * NOTE: If the previous main interface has already
>   	 * disabled hw crypto offload, we have to keep this
>   	 * previous disable_offload setting as it was.
> -	 * Altough ideally, we should notify mac80211 and tell
> +	 * Although ideally, we should notify mac80211 and tell
>   	 * it to forget about any HW crypto offload for now.
>   	 */
>   	ar->disable_offload |= ((vif->type != NL80211_IFTYPE_STATION) &&
> @@ -677,12 +677,12 @@ static int carl9170_op_add_interface(struct ieee80211_hw *hw,
>   	if (old_main) {
>   		/* We end up in here, if the main interface is being replaced.
>   		 * Put the new main interface at the HEAD of the list and the
> -		 * previous inteface will automatically become second in line.
> +		 * previous interface will automatically become second in line.
>   		 */
>   		list_add_rcu(&vif_priv->list, &ar->vif_list);
>   	} else {
> -		/* Add new inteface. If the list is empty, it will become the
> -		 * main inteface, otherwise it will be slave.
> +		/* Add new interface. If the list is empty, it will become the
> +		 * main interface, otherwise it will be slave.
>   		 */
>   		list_add_tail_rcu(&vif_priv->list, &ar->vif_list);
>   	}
> diff --git a/drivers/net/wireless/intersil/p54/p54pci.c b/drivers/net/wireless/intersil/p54/p54pci.c
> index 6588f5477762..68e929fa213c 100644
> --- a/drivers/net/wireless/intersil/p54/p54pci.c
> +++ b/drivers/net/wireless/intersil/p54/p54pci.c
> @@ -534,7 +534,7 @@ static void p54p_firmware_step2(const struct firmware *fw,
>   		/*
>   		 * This will indirectly result in a call to p54p_remove.
>   		 * Hence, we don't need to bother with freeing any
> -		 * allocated ressources at all.
> +		 * allocated resources at all.
>   		 */
>   		device_release_driver(&pdev->dev);
>