On Tue, 18 Sep 2018 14:20:16 +0200, Stanislaw Gruszka wrote:
>On Mon, Sep 17, 2018 at 06:32:51PM +0200, Tomislav Požega wrote:
>> Use curr_band instead of rf->channel among various subroutines -
>> mostly for 2.4GHz band but in some circumstances for 5GHz band too.
>
>What is the reason for that change ?
Operating band should be fetched from device capabilities, not from userspace
variables. More changes will needed to be made to accomplish that.
>
>> @@ -9265,8 +9278,9 @@ static int rt2800_probe_hw_mode(struct rt2x00_dev *rt2x00dev)
>> if (WARN_ON_ONCE(!spec->channels))
>> return -ENODEV;
>>
>> - spec->supported_bands = SUPPORT_BAND_2GHZ;
>> - if (spec->num_channels > 14)
>> + if (rt2x00dev->curr_band == NL80211_BAND_2GHZ)
>> + spec->supported_bands = SUPPORT_BAND_2GHZ;
>> + if (rt2x00dev->curr_band == NL80211_BAND_5GHZ)
>> spec->supported_bands |= SUPPORT_BAND_5GHZ;
>>
>> /*
>> @@ -9336,7 +9350,7 @@ static int rt2800_probe_hw_mode(struct rt2x00_dev *rt2x00dev)
>> info[i].default_power3 = default_power3[i];
>> }
>>
>> - if (spec->num_channels > 14) {
>> + if (rt2x00dev->curr_band == NL80211_BAND_5GHZ) {
>> default_power1 = rt2800_eeprom_addr(rt2x00dev,
>> EEPROM_TXPOWER_A1);
>> default_power2 = rt2800_eeprom_addr(rt2x00dev,
>
>Above looks wrong.
>
>Thanks
>Stanislaw
Worked fine when I tried run two USB cards (RT3070-RF0005, RT5390-RF5370).
Why do you think it's wrong? Is there a dual-band card that operates on
both bands at the same time?
