ath10k: fix incorrect txpower set by P2P_DEVICE interface
authorRyan Hsu <ryanhsu@qca.qualcomm.com>
Tue, 13 Dec 2016 22:55:19 +0000 (14:55 -0800)
committerKalle Valo <kvalo@qca.qualcomm.com>
Fri, 30 Dec 2016 09:12:34 +0000 (11:12 +0200)
Ath10k reports the phy capability that supports P2P_DEVICE interface.

When we use the P2P supported wpa_supplicant to start connection, it'll
create two interfaces, one is wlan0 (vdev_id=0) and one is P2P_DEVICE
p2p-dev-wlan0 which is for p2p control channel (vdev_id=1).

ath10k_pci mac vdev create 0 (add interface) type 2 subtype 0
ath10k_add_interface: vdev_id: 0, txpower: 0, bss_power: 0
...
ath10k_pci mac vdev create 1 (add interface) type 2 subtype 1
ath10k_add_interface: vdev_id: 1, txpower: 0, bss_power: 0

And the txpower in per vif bss_conf will only be set to valid tx power when
the interface is assigned with channel_ctx.

But this P2P_DEVICE interface will never be used for any connection, so
that the uninitialized bss_conf.txpower=0 is assinged to the
arvif->txpower when interface created.

Since the txpower configuration is firmware per physical interface.
So the smallest txpower of all vifs will be the one limit the tx power
of the physical device, that causing the low txpower issue on other
active interfaces.

wlan0: Limiting TX power to 21 (24 - 3) dBm
ath10k_pci mac vdev_id 0 txpower 21
ath10k_mac_txpower_recalc: vdev_id: 1, txpower: 0
ath10k_mac_txpower_recalc: vdev_id: 0, txpower: 21
ath10k_pci mac txpower 0

This issue only happens when we use the wpa_supplicant that supports
P2P or if we use the iw tool to create the control P2P_DEVICE interface.

Signed-off-by: Ryan Hsu <ryanhsu@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
drivers/net/wireless/ath/ath10k/mac.c

index 12a5fccf04cfef0ed29b8d694036423cd50f4835..1dbed8c271cdf345fe1b727099004af18d531762 100644 (file)
@@ -4702,7 +4702,8 @@ static int ath10k_mac_txpower_recalc(struct ath10k *ar)
        lockdep_assert_held(&ar->conf_mutex);
 
        list_for_each_entry(arvif, &ar->arvifs, list) {
-               WARN_ON(arvif->txpower < 0);
+               if (arvif->txpower <= 0)
+                       continue;
 
                if (txpower == -1)
                        txpower = arvif->txpower;
@@ -4710,8 +4711,8 @@ static int ath10k_mac_txpower_recalc(struct ath10k *ar)
                        txpower = min(txpower, arvif->txpower);
        }
 
-       if (WARN_ON(txpower == -1))
-               return -EINVAL;
+       if (txpower == -1)
+               return 0;
 
        ret = ath10k_mac_txpower_setup(ar, txpower);
        if (ret) {