diff mbox series

[3/3] wifi: ath12k: Fix a few spelling errors

Message ID 20230822-ath_spelling-v1-3-8e2698759564@quicinc.com
State New
Headers show
Series ath*: Fix a few spelling errors | expand

Commit Message

Jeff Johnson Aug. 22, 2023, 2:50 p.m. UTC
Fix a few issues flagged by 'codespell'.

Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
---
 drivers/net/wireless/ath/ath12k/dp.h      | 2 +-
 drivers/net/wireless/ath/ath12k/dp_rx.c   | 2 +-
 drivers/net/wireless/ath/ath12k/dp_tx.c   | 2 +-
 drivers/net/wireless/ath/ath12k/mac.c     | 2 +-
 drivers/net/wireless/ath/ath12k/rx_desc.h | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

Comments

Randy Dunlap Aug. 22, 2023, 4:18 p.m. UTC | #1
Hi--

On 8/22/23 07:50, Jeff Johnson wrote:
> Fix a few issues flagged by 'codespell'.
> 
> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
> ---
>  drivers/net/wireless/ath/ath12k/dp.h      | 2 +-
>  drivers/net/wireless/ath/ath12k/dp_rx.c   | 2 +-
>  drivers/net/wireless/ath/ath12k/dp_tx.c   | 2 +-
>  drivers/net/wireless/ath/ath12k/mac.c     | 2 +-
>  drivers/net/wireless/ath/ath12k/rx_desc.h | 2 +-
>  5 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
> index 0f2af2f14ef7..dbaf033c2527 100644
> --- a/drivers/net/wireless/ath/ath12k/mac.c
> +++ b/drivers/net/wireless/ath/ath12k/mac.c
> @@ -6771,7 +6771,7 @@ ath12k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
>  			/* After trigger disconnect, then upper layer will
>  			 * trigger connect again, then the PN number of
>  			 * upper layer will be reset to keep up with AP
> -			 * side, hence PN number mis-match will not happened.
> +			 * side, hence PN number mismatch will not happened.

The fix is good, but preferably also s/happened/happen/.

>  			 */
>  			if (arvif->is_up &&
>  			    arvif->vdev_type == WMI_VDEV_TYPE_STA &&
Jeff Johnson Aug. 22, 2023, 4:43 p.m. UTC | #2
On 8/22/2023 9:18 AM, Randy Dunlap wrote:
> Hi--
> 
> On 8/22/23 07:50, Jeff Johnson wrote:
>> Fix a few issues flagged by 'codespell'.
>>
>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>> ---
>>   drivers/net/wireless/ath/ath12k/dp.h      | 2 +-
>>   drivers/net/wireless/ath/ath12k/dp_rx.c   | 2 +-
>>   drivers/net/wireless/ath/ath12k/dp_tx.c   | 2 +-
>>   drivers/net/wireless/ath/ath12k/mac.c     | 2 +-
>>   drivers/net/wireless/ath/ath12k/rx_desc.h | 2 +-
>>   5 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
>> index 0f2af2f14ef7..dbaf033c2527 100644
>> --- a/drivers/net/wireless/ath/ath12k/mac.c
>> +++ b/drivers/net/wireless/ath/ath12k/mac.c
>> @@ -6771,7 +6771,7 @@ ath12k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
>>   			/* After trigger disconnect, then upper layer will
>>   			 * trigger connect again, then the PN number of
>>   			 * upper layer will be reset to keep up with AP
>> -			 * side, hence PN number mis-match will not happened.
>> +			 * side, hence PN number mismatch will not happened.
> 
> The fix is good, but preferably also s/happened/happen/.

Kalle, can you apply that when it goes to -pending?

> 
>>   			 */
>>   			if (arvif->is_up &&
>>   			    arvif->vdev_type == WMI_VDEV_TYPE_STA &&
>
Kalle Valo Aug. 22, 2023, 7:10 p.m. UTC | #3
Jeff Johnson <quic_jjohnson@quicinc.com> writes:

> On 8/22/2023 9:18 AM, Randy Dunlap wrote:
>
>> Hi--
>> On 8/22/23 07:50, Jeff Johnson wrote:
>>> Fix a few issues flagged by 'codespell'.
>>>
>>> Signed-off-by: Jeff Johnson <quic_jjohnson@quicinc.com>
>>> ---
>>>   drivers/net/wireless/ath/ath12k/dp.h      | 2 +-
>>>   drivers/net/wireless/ath/ath12k/dp_rx.c   | 2 +-
>>>   drivers/net/wireless/ath/ath12k/dp_tx.c   | 2 +-
>>>   drivers/net/wireless/ath/ath12k/mac.c     | 2 +-
>>>   drivers/net/wireless/ath/ath12k/rx_desc.h | 2 +-
>>>   5 files changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
>>> index 0f2af2f14ef7..dbaf033c2527 100644
>>> --- a/drivers/net/wireless/ath/ath12k/mac.c
>>> +++ b/drivers/net/wireless/ath/ath12k/mac.c
>>> @@ -6771,7 +6771,7 @@ ath12k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
>>>   			/* After trigger disconnect, then upper layer will
>>>   			 * trigger connect again, then the PN number of
>>>   			 * upper layer will be reset to keep up with AP
>>> -			 * side, hence PN number mis-match will not happened.
>>> +			 * side, hence PN number mismatch will not happened.
>> The fix is good, but preferably also s/happened/happen/.
>
> Kalle, can you apply that when it goes to -pending?

Yes, did that now:

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=212796e28fe44c0a4956c797d8b685b905784cd6
diff mbox series

Patch

diff --git a/drivers/net/wireless/ath/ath12k/dp.h b/drivers/net/wireless/ath/ath12k/dp.h
index 7c5dafce5a68..6e7b7281d549 100644
--- a/drivers/net/wireless/ath/ath12k/dp.h
+++ b/drivers/net/wireless/ath/ath12k/dp.h
@@ -712,7 +712,7 @@  enum htt_stats_internal_ppdu_frametype {
  *          b'24    - status_swap: 1 is to swap status TLV
  *          b'25    - pkt_swap:  1 is to swap packet TLV
  *          b'26:31 - rsvd1:  reserved for future use
- * dword1 - b'0:16  - ring_buffer_size: size of bufferes referenced by rx ring,
+ * dword1 - b'0:16  - ring_buffer_size: size of buffers referenced by rx ring,
  *                    in byte units.
  *                    Valid only for HW_TO_SW_RING and SW_TO_HW_RING
  *        - b'16:31 - rsvd2: Reserved for future use
diff --git a/drivers/net/wireless/ath/ath12k/dp_rx.c b/drivers/net/wireless/ath/ath12k/dp_rx.c
index 67f8c140840f..e6e64d437c47 100644
--- a/drivers/net/wireless/ath/ath12k/dp_rx.c
+++ b/drivers/net/wireless/ath/ath12k/dp_rx.c
@@ -3027,7 +3027,7 @@  static int ath12k_dp_rx_h_defrag_reo_reinject(struct ath12k *ar,
 					desc_info->cookie,
 					HAL_RX_BUF_RBM_SW3_BM);
 
-	/* Fill mpdu details into reo entrace ring */
+	/* Fill mpdu details into reo entrance ring */
 	srng = &ab->hal.srng_list[dp->reo_reinject_ring.ring_id];
 
 	spin_lock_bh(&srng->lock);
diff --git a/drivers/net/wireless/ath/ath12k/dp_tx.c b/drivers/net/wireless/ath/ath12k/dp_tx.c
index d661fe586651..8874c815d7fa 100644
--- a/drivers/net/wireless/ath/ath12k/dp_tx.c
+++ b/drivers/net/wireless/ath/ath12k/dp_tx.c
@@ -301,7 +301,7 @@  int ath12k_dp_tx(struct ath12k *ar, struct ath12k_vif *arvif,
 		spin_unlock_bh(&tcl_ring->lock);
 		ret = -ENOMEM;
 
-		/* Checking for available tcl descritors in another ring in
+		/* Checking for available tcl descriptors in another ring in
 		 * case of failure due to full tcl ring now, is better than
 		 * checking this ring earlier for each pkt tx.
 		 * Restart ring selection if some rings are not checked yet.
diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
index 0f2af2f14ef7..dbaf033c2527 100644
--- a/drivers/net/wireless/ath/ath12k/mac.c
+++ b/drivers/net/wireless/ath/ath12k/mac.c
@@ -6771,7 +6771,7 @@  ath12k_mac_op_reconfig_complete(struct ieee80211_hw *hw,
 			/* After trigger disconnect, then upper layer will
 			 * trigger connect again, then the PN number of
 			 * upper layer will be reset to keep up with AP
-			 * side, hence PN number mis-match will not happened.
+			 * side, hence PN number mismatch will not happened.
 			 */
 			if (arvif->is_up &&
 			    arvif->vdev_type == WMI_VDEV_TYPE_STA &&
diff --git a/drivers/net/wireless/ath/ath12k/rx_desc.h b/drivers/net/wireless/ath/ath12k/rx_desc.h
index f99556a253e5..bfa87cb8d021 100644
--- a/drivers/net/wireless/ath/ath12k/rx_desc.h
+++ b/drivers/net/wireless/ath/ath12k/rx_desc.h
@@ -221,7 +221,7 @@  struct rx_mpdu_start_qcn9274 {
  *		PPE routing even if RXOLE CCE or flow search indicate 'Use_PPE'
  *		This is set by SW for peers which are being handled by a
  *		host SW/accelerator subsystem that also handles packet
- *		uffer management for WiFi-to-PPE routing.
+ *		buffer management for WiFi-to-PPE routing.
  *
  *		This is cleared by SW for peers which are being handled
  *		by a different subsystem, completely disabling WiFi-to-PPE