rtlwifi: btcoexist: Fix if == else warnings in halbtc8723b1ant.c
authorLarry Finger <Larry.Finger@lwfinger.net>
Sun, 12 Feb 2017 00:46:05 +0000 (18:46 -0600)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 15 Feb 2017 09:14:44 +0000 (11:14 +0200)
The 0-DAY kernel test infrastructure reports the following:
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8723b1ant.c:1875:2-4: WARNING: possible condition with no effect (if == else)
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8723b1ant.c:2253:3-5: WARNING: possible condition with no effect (if == else)

Reported-by: kbuild-all@01.org
Signed-off-by: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Julia Lawall <julia.lawall@lip6.fr>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8723b1ant.c

index e9f0a76..d67bbfb 100644 (file)
@@ -1872,16 +1872,8 @@ static void halbtc8723b1ant_action_wifi_connected_bt_acl_busy(
                }
        } else if (bt_link_info->hid_exist &&
                        bt_link_info->a2dp_exist) { /*HID+A2DP */
-               if ((bt_rssi_state == BTC_RSSI_STATE_HIGH) ||
-                   (bt_rssi_state == BTC_RSSI_STATE_STAY_HIGH)) {
-                       halbtc8723b1ant_ps_tdma(btcoexist, NORMAL_EXEC,
-                                               true, 14);
-                       coex_dm->auto_tdma_adjust = false;
-               } else { /*for low BT RSSI*/
-                       halbtc8723b1ant_ps_tdma(btcoexist, NORMAL_EXEC,
-                                               true, 14);
-                       coex_dm->auto_tdma_adjust = false;
-               }
+               halbtc8723b1ant_ps_tdma(btcoexist, NORMAL_EXEC, true, 14);
+               coex_dm->auto_tdma_adjust = false;
 
                halbtc8723b1ant_coex_table_with_type(btcoexist, NORMAL_EXEC, 6);
         /*PAN(OPP,FTP), HID+PAN(OPP,FTP) */
@@ -2250,16 +2242,8 @@ static void halbtc8723b1ant_run_coexist_mechanism(struct btc_coexist *btcoexist)
                        wifi_rssi_state =
                                halbtc8723b1ant_wifi_rssi_state(btcoexist,
                                                                1, 2, 30, 0);
-                       if ((wifi_rssi_state == BTC_RSSI_STATE_HIGH) ||
-                           (wifi_rssi_state == BTC_RSSI_STATE_STAY_HIGH)) {
-                               halbtc8723b1ant_limited_tx(btcoexist,
-                                                          NORMAL_EXEC,
-                                                          1, 1, 1, 1);
-                       } else {
-                               halbtc8723b1ant_limited_tx(btcoexist,
-                                                          NORMAL_EXEC,
-                                                          1, 1, 1, 1);
-                       }
+                       halbtc8723b1ant_limited_tx(btcoexist,
+                                                  NORMAL_EXEC, 1, 1, 1, 1);
                } else {
                        halbtc8723b1ant_limited_tx(btcoexist, NORMAL_EXEC,
                                                   0, 0, 0, 0);