2022-01-13 07:37:21

by Wen Gong

[permalink] [raw]
Subject: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz

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 <[email protected]>
---
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


2022-01-18 02:26:37

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz

Wen Gong <[email protected]> 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/[email protected]/

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

2022-01-19 04:08:10

by Wen Gong

[permalink] [raw]
Subject: Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz

On 1/17/2022 9:08 PM, Kalle Valo wrote:
> Wen Gong <[email protected]> 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.

2022-01-19 19:45:16

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz

Wen Gong <[email protected]> writes:

> On 1/17/2022 9:08 PM, Kalle Valo wrote:
>> Wen Gong <[email protected]> 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