wifi: rtw88: remove CPT execution branch never used

In 'rtw_coex_action_bt_a2dp_pan', 'wl_cpt_test' and 'bt_cpt_test' are
hardcoded to false, so corresponding 'table_case' and 'tdma_case'
assignments are never met.
Also 'rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[1])' is never
executed. Assuming that CPT was never fully implemented, remove
lookalike leftovers. Compile tested only.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Fixes: 76f631cb401f ("rtw88: coex: update the mechanism for A2DP + PAN")
Signed-off-by: Dmitry Kandybka <d.kandybka@gmail.com>
Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Link: https://patch.msgid.link/20240809085310.10512-1-d.kandybka@gmail.com

diff --git a/drivers/net/wireless/realtek/rtw88/coex.c b/drivers/net/wireless/realtek/rtw88/coex.c
index de3332eb7a227aaba99e255e31f90fc43b6572a4..a99776af56c27fe381058ec9f12bf50e52b30724 100644 (file)
--- a/drivers/net/wireless/realtek/rtw88/coex.c
+++ b/drivers/net/wireless/realtek/rtw88/coex.c
@@ -2194,7 +2194,6 @@ static void rtw_coex_action_bt_a2dp_pan(struct rtw_dev *rtwdev)
        struct rtw_coex_stat *coex_stat = &coex->stat;
        struct rtw_efuse *efuse = &rtwdev->efuse;
        u8 table_case, tdma_case;
-       bool wl_cpt_test = false, bt_cpt_test = false;
 
        rtw_dbg(rtwdev, RTW_DBG_COEX, "[BTCoex], %s()\n", __func__);
 
@@ -2202,29 +2201,16 @@ static void rtw_coex_action_bt_a2dp_pan(struct rtw_dev *rtwdev)
        rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[0]);
        if (efuse->share_ant) {
                /* Shared-Ant */
-               if (wl_cpt_test) {
-                       if (coex_stat->wl_gl_busy) {
-                               table_case = 20;
-                               tdma_case = 17;
-                       } else {
-                               table_case = 10;
-                               tdma_case = 15;
-                       }
-               } else if (bt_cpt_test) {
-                       table_case = 26;
-                       tdma_case = 26;
-               } else {
-                       if (coex_stat->wl_gl_busy &&
-                           coex_stat->wl_noisy_level == 0)
-                               table_case = 14;
-                       else
-                               table_case = 10;
+               if (coex_stat->wl_gl_busy &&
+                   coex_stat->wl_noisy_level == 0)
+                       table_case = 14;
+               else
+                       table_case = 10;
 
-                       if (coex_stat->wl_gl_busy)
-                               tdma_case = 15;
-                       else
-                               tdma_case = 20;
-               }
+               if (coex_stat->wl_gl_busy)
+                       tdma_case = 15;
+               else
+                       tdma_case = 20;
        } else {
                /* Non-Shared-Ant */
                table_case = 112;
@@ -2235,11 +2221,7 @@ static void rtw_coex_action_bt_a2dp_pan(struct rtw_dev *rtwdev)
                        tdma_case = 120;
        }
 
-       if (wl_cpt_test)
-               rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[1]);
-       else
-               rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[0]);
-
+       rtw_coex_set_rf_para(rtwdev, chip->wl_rf_para_rx[0]);
        rtw_coex_table(rtwdev, false, table_case);
        rtw_coex_tdma(rtwdev, false, tdma_case);
 }