| Message ID | 20220113023145.14292-1-quic_wgong@quicinc.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | ath11k: set WMI_PEER_40MHZ while peer assoc for 6 GHz | expand |
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")
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.
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.
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;
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(+) base-commit: 69a6130e6ab00306fe1552c73eb0d7a0dd4b4c04