[PATCH V5 4/6] wifi: ath12k: add 6 GHz params in peer assoc command

Pradeep Kumar Chitrapu quic_pradeepc at quicinc.com
Wed Feb 19 09:27:02 PST 2025



On 2/12/2025 6:05 PM, Pradeep Kumar Chitrapu wrote:
> 
> 
> 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)
>>
> Hi Maxime,
> 
> I noticed that currently only one of the bandwidth flags is set in the 
> driver. I am inquiring with the firmware team to determine if we need to 
> set multiple flags from the driver. I will update you soon and, if 
> required, will post the next revision.
> 
> Thanks
> Pradeep
> 
Hi Maxime,

Currently, the firmware sets all lower bandwidth flags based on the 
highest bandwidth set. While there is no harm in setting all bandwidth 
flags, this may not be mandatory. Please let me know your thoughts.

Thanks,
Pradeep



More information about the ath12k mailing list