On 6/11/20 1:29 PM, Markus Theil wrote:
> On 3/27/20 10:31 AM, Robert Marko wrote:
>> Enable advertising support for 5G channels: 32, 68 and 96.
>> These channels are legal and available for use in ETSI countries.
>> So lets advertise these and they will be available in accordance with the 
>> regulatory domain used.
>>
>> Signed-off-by: Robert Marko <[email protected]>
>> Cc: Luka Perkov <[email protected]>
>> ---
>>  drivers/net/wireless/ath/ath10k/core.h | 2 +-
>>  drivers/net/wireless/ath/ath10k/mac.c  | 3 +++
>>  2 files changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath10k/core.h 
>> b/drivers/net/wireless/ath/ath10k/core.h
>> index 5101bf2b5b15..480efaa1278c 100644
>> --- a/drivers/net/wireless/ath/ath10k/core.h
>> +++ b/drivers/net/wireless/ath/ath10k/core.h
>> @@ -37,7 +37,7 @@
>>  #define WMI_READY_TIMEOUT (5 * HZ)
>>  #define ATH10K_FLUSH_TIMEOUT_HZ (5 * HZ)
>>  #define ATH10K_CONNECTION_LOSS_HZ (3 * HZ)
>> -#define ATH10K_NUM_CHANS 41
>> +#define ATH10K_NUM_CHANS 44
>>  #define ATH10K_MAX_5G_CHAN 173
>>  
>>  /* Antenna noise floor */
>> diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
>> b/drivers/net/wireless/ath/ath10k/mac.c
>> index 7fee35ff966b..f98422427b27 100644
>> --- a/drivers/net/wireless/ath/ath10k/mac.c
>> +++ b/drivers/net/wireless/ath/ath10k/mac.c
>> @@ -8363,6 +8363,7 @@ static const struct ieee80211_channel 
>> ath10k_2ghz_channels[] = {
>>  };
>>  
>>  static const struct ieee80211_channel ath10k_5ghz_channels[] = {
>> +    CHAN5G(32, 5160, 0),
>>      CHAN5G(36, 5180, 0),
>>      CHAN5G(40, 5200, 0),
>>      CHAN5G(44, 5220, 0),
>> @@ -8371,6 +8372,8 @@ static const struct ieee80211_channel 
>> ath10k_5ghz_channels[] = {
>>      CHAN5G(56, 5280, 0),
>>      CHAN5G(60, 5300, 0),
>>      CHAN5G(64, 5320, 0),
>> +    CHAN5G(68, 5340, 0),
>> +    CHAN5G(96, 5480, 0),
>>      CHAN5G(100, 5500, 0),
>>      CHAN5G(104, 5520, 0),
>>      CHAN5G(108, 5540, 0),
> Hi, your patch is incomplete. Mgmt frames cannot be received on channel 32, 
> because of the following lines in wmi.c:
>
>       /* Hardware can Rx CCK rates on 5GHz. In that case phy_mode is set to
>        * MODE_11B. This means phy_mode is not a reliable source for the band
>        * of mgmt rx.
>        */
>       if (channel >= 1 && channel <= 14) {
>               status->band = NL80211_BAND_2GHZ;
>       } else if (channel >= 36 && channel <= ATH10K_MAX_5G_CHAN) {
>               status->band = NL80211_BAND_5GHZ;
>       } else {
>               /* Shouldn't happen unless list of advertised channels to
>                * mac80211 has been changed.
>                */
>               WARN_ON_ONCE(1);
>               dev_kfree_skb(skb);
>               return 0;
>       }
>
> Have you tested this patch?
I've tested your patch (changed 36 to 32 in wmi.c). And channel 32 does
not work for me, even with that change.
Nevertheless, I was able to use channels 68 and 96 between two ath10k
cards in AP and STA mode. I do not know,
why channel 32 does not work.



_______________________________________________
ath10k mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/ath10k

Reply via email to