rtlwifi: btcoex: Remove 21a 1ant configuration parameter
authorLarry Finger <Larry.Finger@lwfinger.net>
Thu, 4 May 2017 18:46:58 +0000 (13:46 -0500)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 18 May 2017 14:06:42 +0000 (17:06 +0300)
In file halbtc8821a1ant.c, there are directives that depend on an
undocumented configuration parameter BT_AUTO_REPORT_ONLY_8821A_1ANT
that cannot be set from Kconfig. This parameter is replaced by a
boolean in the main structure used by all routines. It still cannot
be changed dynamically, but it is easier to document.

Using a suggestion from Realtek, the auto report is turned on with this
patch.

Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Yan-Hsuan Chuang <yhchuang@realtek.com>
Cc: Pkshih <pkshih@realtek.com>
Cc: Birming Chiu <birming@realtek.com>
Cc: Shaofu <shaofu@realtek.com>
Cc: Steven Ting <steventing@realtek.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8821a1ant.c
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8821a1ant.h
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.h

index 5e9f3b0f7a2583e36bc5c747325a52fbfb73141a..bd5f752f153d10d78e2cc28e91b807927a347d62 100644 (file)
@@ -2116,6 +2116,7 @@ static void btc8821a1ant_init_hw_config(struct btc_coexist *btcoexist,
 void ex_btc8821a1ant_init_hwconfig(struct btc_coexist *btcoexist, bool wifionly)
 {
        btc8821a1ant_init_hw_config(btcoexist, true, wifionly);
+       btcoexist->auto_report_1ant = true;
 }
 
 void ex_btc8821a1ant_init_coex_dm(struct btc_coexist *btcoexist)
@@ -2406,9 +2407,8 @@ void ex_btc8821a1ant_display_coex_info(struct btc_coexist *btcoexist)
        RT_TRACE(rtlpriv, COMP_INIT, DBG_DMESG,
                 "\r\n %-35s = %d/ %d", "0x774(low-pri rx/tx)",
                 coex_sta->low_priority_rx, coex_sta->low_priority_tx);
-#if (BT_AUTO_REPORT_ONLY_8821A_1ANT == 1)
-       btc8821a1ant_monitor_bt_ctr(btcoexist);
-#endif
+       if (btcoexist->auto_report_1ant)
+               btc8821a1ant_monitor_bt_ctr(btcoexist);
        btcoexist->btc_disp_dbg_msg(btcoexist, BTC_DBG_DISP_COEX_STATISTICS);
 }
 
@@ -2964,10 +2964,10 @@ void ex_btc8821a1ant_periodical(struct btc_coexist *btcoexist)
                         "[BTCoex], ****************************************************************\n");
        }
 
-#if (BT_AUTO_REPORT_ONLY_8821A_1ANT == 0)
-       btc8821a1ant_query_bt_info(btcoexist);
-       btc8821a1ant_monitor_bt_ctr(btcoexist);
-#else
-       coex_sta->special_pkt_period_cnt++;
-#endif
+       if (!btcoexist->auto_report_1ant) {
+               btc8821a1ant_query_bt_info(btcoexist);
+               btc8821a1ant_monitor_bt_ctr(btcoexist);
+       } else {
+               coex_sta->special_pkt_period_cnt++;
+       }
 }
index 1bd1ebe3364ebe3e0ac44e17cce0507a9aba566e..d6dacf32a4c4a7d26fde9f100f4d5da50d95139f 100644 (file)
@@ -27,8 +27,6 @@
  * The following is for 8821A 1ANT BT Co-exist definition
  *===========================================
  */
-#define        BT_AUTO_REPORT_ONLY_8821A_1ANT                          0
-
 #define        BT_INFO_8821A_1ANT_B_FTP        BIT7
 #define        BT_INFO_8821A_1ANT_B_A2DP       BIT6
 #define        BT_INFO_8821A_1ANT_B_HID        BIT5
index c8271135aaaaed26af1ba359374fc1f133542b38..ab2e31a442536a9e111c318e06f9885b577a6b9a 100644 (file)
@@ -496,6 +496,11 @@ struct btc_coexist {
        enum btc_chip_interface chip_interface;
        struct btc_bt_link_info bt_link_info;
 
+       /* boolean variables to replace BT_AUTO_REPORT_ONLY_XXXXY_ZANT
+        * configuration parameters
+        */
+       bool auto_report_1ant;
+       bool auto_report_2ant;
        bool initilized;
        bool stop_coex_dm;
        bool manual_control;