On 1/23/2025 3:35 AM, Maxime Bizon wrote:
On Wed, 2024-05-08 at 10:36 -0700, Pradeep Kumar Chitrapu wrote:
Hello,
This patch has been merged, but there may be a bug here:
+ if (sta->deflink.bandwidth == IEEE80211_STA_RX_BW_40)
+ arg->bw_40 = true;
+
+ if (sta->deflink.bandwidth == IEEE80211_STA_RX_BW_80)
+ arg->bw_80 = true;
+
+ if (sta->deflink.bandwidth == IEEE80211_STA_RX_BW_160)
+ arg->bw_160 = true;
+
+ if (sta->deflink.bandwidth == IEEE80211_STA_RX_BW_320)
+ arg->bw_320 = true;
+
Downstream driver does this instead:
if (sta->deflink.bandwidth >= IEEE80211_STA_RX_BW_40)
arg->bw_40 = true
[...]
So for a 320Mhz STA, bw_40/80/160/320 are set, resulting in multiple
bits WMI_PEER_xxxMHZ set the associated WMI command.
Which one is correct ?
(This is for EHT, but the same applies for VHT/HE code)
Thanks for reporting Maxime, Will look into it and fix this soon.
