Search Linux Wireless

Re: [PATCH v8 12/12] wifi: ath11k: discard BSS_CHANGED_TXPOWER when EXT_TPC_REG_SUPPORT for 6 GHz

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On 12/7/2023 11:31 AM, Aditya Kumar Singh wrote:
On 12/6/23 11:04, Baochen Qiang wrote:


On 12/4/2023 11:53 PM, Aditya Kumar Singh wrote:
On 12/4/23 13:43, Baochen Qiang wrote:
From: Wen Gong <quic_wgong@xxxxxxxxxxx>

When station is connected to a 6 GHz AP, it has 2 ways to configure
the power limit to firmware. The first way is to send 2 WMI commands
WMI_PDEV_PARAM_TXPOWER_LIMIT2G/WMI_PDEV_PARAM_TXPOWER_LIMIT5G to
firmware, the second way is to send WMI_VDEV_SET_TPC_POWER_CMDID to
firmware which include more parameters for power control.

When firmware support SERVICE_EXT_TPC_REG, it means firmware support
the second way for WMI_VDEV_SET_TPC_POWER_CMDID, then ath11k discard
BSS_CHANGED_TXPOWER flag from mac80211 which is used to the first way
for 6 GHz band and select the second way.

Tested-on: WCN6855 hw2.0 PCI WLAN.HSP.1.1-03125-QCAHSPSWPL_V1_V2_SILICONZ_LITE-3.6510.23

Signed-off-by: Wen Gong <quic_wgong@xxxxxxxxxxx>
Signed-off-by: Baochen Qiang <quic_bqiang@xxxxxxxxxxx>
---
...snip...
@@ -3596,9 +3608,13 @@ static void ath11k_mac_op_bss_info_changed(struct ieee80211_hw *hw,
      if (changed & BSS_CHANGED_TXPOWER) {
          ath11k_dbg(ar->ab, ATH11K_DBG_MAC, "vdev_id %i txpower %d\n",
                 arvif->vdev_id, info->txpower);
-
-        arvif->txpower = info->txpower;
-        ath11k_mac_txpower_recalc(ar);
+        if (ath11k_mac_supports_station_tpc(ar, arvif, &info->chandef)) {
+            ath11k_dbg(ar->ab, ATH11K_DBG_MAC,
+                   "discard tx power, change to set TPC power\n");
+        } else {
+            arvif->txpower = info->txpower;
+            ath11k_mac_txpower_recalc(ar);
+        }

Could you check v6 once? I remember Wen told he would drop this check and let FW take the min value. If we do like this, then user could not set his own desired value even if that is well inside the reg limits.
I did notice this comment in V6, but came out of a different opinion: it is OK to discard the TX power here, because that will be sent to firmware using WMI_VDEV_SET_TPC_POWER_CMDID command in another patch. Please correct me if wrong.
Yeah that is correct but applies only during initial bring up. What if after client gets connected and user still wants to lower power level by giving command "iw wlanX set txpower fixed 1000" something like this? This time again it will be ignored but it won't be sent to FW.
Exactly, will drop this patch in V9.





[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux