linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz
@ 2022-01-13  2:31 Wen Gong
  2022-01-17 13:08 ` Kalle Valo
  0 siblings, 1 reply; 4+ messages in thread
From: Wen Gong @ 2022-01-13  2:31 UTC (permalink / raw)
  To: ath11k; +Cc: linux-wireless, quic_wgong

When station connect to AP of 6 GHz with 40 MHz bandwidth, the TX is
always stay 20 MHz, it is because the flag WMI_PEER_40MHZ is not set
while peer assoc. Add the flag if remote peer is 40 MHz bandwidth.

Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-02892.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1

Fixes: 2cdf2b3cdf54 ("ath11k: add 6ghz params in peer assoc command")
Signed-off-by: Wen Gong <quic_wgong@quicinc.com>
---
 drivers/net/wireless/ath/ath11k/mac.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index 07f499d5ec92..fe4103bcfee3 100644
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -2319,6 +2319,9 @@ static void ath11k_peer_assoc_h_he_6ghz(struct ath11k *ar,
 	if (!arg->he_flag || band != NL80211_BAND_6GHZ || !sta->he_6ghz_capa.capa)
 		return;
 
+	if (sta->bandwidth == IEEE80211_STA_RX_BW_40)
+		arg->bw_40 = true;
+
 	if (sta->bandwidth == IEEE80211_STA_RX_BW_80)
 		arg->bw_80 = true;
 

base-commit: 69a6130e6ab00306fe1552c73eb0d7a0dd4b4c04
-- 
2.31.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz
  2022-01-13  2:31 [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz Wen Gong
@ 2022-01-17 13:08 ` Kalle Valo
  2022-01-18  2:43   ` Wen Gong
  0 siblings, 1 reply; 4+ messages in thread
From: Kalle Valo @ 2022-01-17 13:08 UTC (permalink / raw)
  To: Wen Gong; +Cc: ath11k, linux-wireless, quic_wgong

Wen Gong <quic_wgong@quicinc.com> wrote:

> When station connect to AP of 6 GHz with 40 MHz bandwidth, the TX is
> always stay 20 MHz, it is because the flag WMI_PEER_40MHZ is not set
> while peer assoc. Add the flag if remote peer is 40 MHz bandwidth.
> 
> Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-02892.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1

Fixes tag

  Fixes: 2cdf2b3cdf54 ("ath11k: add 6ghz params in peer assoc command")

has these problem(s):

  - Target SHA1 does not exist

Did you mean:

Fixes: c3a7d7eb4c98 ("ath11k: add 6 GHz params in peer assoc command")

-- 
https://patchwork.kernel.org/project/linux-wireless/patch/20220113023145.14292-1-quic_wgong@quicinc.com/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz
  2022-01-17 13:08 ` Kalle Valo
@ 2022-01-18  2:43   ` Wen Gong
  2022-01-18  6:33     ` Kalle Valo
  0 siblings, 1 reply; 4+ messages in thread
From: Wen Gong @ 2022-01-18  2:43 UTC (permalink / raw)
  To: Kalle Valo; +Cc: ath11k, linux-wireless

On 1/17/2022 9:08 PM, Kalle Valo wrote:
> Wen Gong <quic_wgong@quicinc.com> wrote:
>
>> When station connect to AP of 6 GHz with 40 MHz bandwidth, the TX is
>> always stay 20 MHz, it is because the flag WMI_PEER_40MHZ is not set
>> while peer assoc. Add the flag if remote peer is 40 MHz bandwidth.
>>
>> Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-02892.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1
> Fixes tag
>
>    Fixes: 2cdf2b3cdf54 ("ath11k: add 6ghz params in peer assoc command")
>
> has these problem(s):
>
>    - Target SHA1 does not exist
>
> Did you mean:
>
> Fixes: c3a7d7eb4c98 ("ath11k: add 6 GHz params in peer assoc command")

Thanks.

yes, it is.

I see you have already changed it in master-pending.


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz
  2022-01-18  2:43   ` Wen Gong
@ 2022-01-18  6:33     ` Kalle Valo
  0 siblings, 0 replies; 4+ messages in thread
From: Kalle Valo @ 2022-01-18  6:33 UTC (permalink / raw)
  To: Wen Gong; +Cc: ath11k, linux-wireless

Wen Gong <quic_wgong@quicinc.com> writes:

> On 1/17/2022 9:08 PM, Kalle Valo wrote:
>> Wen Gong <quic_wgong@quicinc.com> wrote:
>>
>>> When station connect to AP of 6 GHz with 40 MHz bandwidth, the TX is
>>> always stay 20 MHz, it is because the flag WMI_PEER_40MHZ is not set
>>> while peer assoc. Add the flag if remote peer is 40 MHz bandwidth.
>>>
>>> Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-02892.1-QCAHSPSWPL_V1_V2_SILICONZ_LITE-1
>> Fixes tag
>>
>>    Fixes: 2cdf2b3cdf54 ("ath11k: add 6ghz params in peer assoc command")
>>
>> has these problem(s):
>>
>>    - Target SHA1 does not exist
>>
>> Did you mean:
>>
>> Fixes: c3a7d7eb4c98 ("ath11k: add 6 GHz params in peer assoc command")
>
> Thanks.
>
> yes, it is.
>
> I see you have already changed it in master-pending.

Yeah, so no need to resend because of this.

-- 
https://patchwork.kernel.org/project/linux-wireless/list/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-01-18  6:33 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-13  2:31 [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz Wen Gong
2022-01-17 13:08 ` Kalle Valo
2022-01-18  2:43   ` Wen Gong
2022-01-18  6:33     ` Kalle Valo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).