ath11k: use TCL_DATA_RING_0 for QCA6390
authorCarl Huang <cjhuang@codeaurora.org>
Mon, 17 Aug 2020 10:31:55 +0000 (13:31 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 18 Aug 2020 09:47:07 +0000 (12:47 +0300)
For QCA6390, wbm2sw1 is used for other purpose rather than
tx completion ring. So use TCL_DATA_RING 0 only for QCA6390.

Add MISC_CAPS_TCL_0_ONLY to control it.

Tested-on: QCA6390 hw2.0 PCI WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1
Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.1.0.1-01238-QCAHKSWPL_SILICONZ-2

Signed-off-by: Carl Huang <cjhuang@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1597555891-26112-9-git-send-email-kvalo@codeaurora.org
drivers/net/wireless/ath/ath11k/core.c
drivers/net/wireless/ath/ath11k/dp_tx.c
drivers/net/wireless/ath/ath11k/hw.h

index a279450..4e39766 100644 (file)
@@ -42,6 +42,7 @@ static const struct ath11k_hw_params ath11k_hw_params[] = {
                .rx_mac_buf_ring = false,
                .vdev_start_delay = false,
                .htt_peer_map_v2 = true,
+               .tcl_0_only = false,
        },
        {
                .name = "qca6390 hw2.0",
@@ -66,6 +67,7 @@ static const struct ath11k_hw_params ath11k_hw_params[] = {
                .rx_mac_buf_ring = true,
                .vdev_start_delay = true,
                .htt_peer_map_v2 = false,
+               .tcl_0_only = true,
        },
 };
 
index 953c435..a9632b7 100644 (file)
@@ -110,7 +110,12 @@ int ath11k_dp_tx(struct ath11k *ar, struct ath11k_vif *arvif,
 
 tcl_ring_sel:
        tcl_ring_retry = false;
-       ti.ring_id = ring_selector % DP_TCL_NUM_RING_MAX;
+       /* For some chip, it can only use tcl0 to tx */
+       if (ar->ab->hw_params.tcl_0_only)
+               ti.ring_id = 0;
+       else
+               ti.ring_id = ring_selector % DP_TCL_NUM_RING_MAX;
+
        ring_map |= BIT(ti.ring_id);
 
        tx_ring = &dp->tx_ring[ti.ring_id];
@@ -221,7 +226,8 @@ tcl_ring_sel:
                 * checking this ring earlier for each pkt tx.
                 * Restart ring selection if some rings are not checked yet.
                 */
-               if (ring_map != (BIT(DP_TCL_NUM_RING_MAX) - 1)) {
+               if (ring_map != (BIT(DP_TCL_NUM_RING_MAX) - 1) &&
+                   !ar->ab->hw_params.tcl_0_only) {
                        tcl_ring_retry = true;
                        ring_selector++;
                }
index 500060b..d15fa7f 100644 (file)
@@ -151,6 +151,7 @@ struct ath11k_hw_params {
        bool rx_mac_buf_ring;
        bool vdev_start_delay;
        bool htt_peer_map_v2;
+       bool tcl_0_only;
 };
 
 struct ath11k_hw_ops {