rtlwifi: btcoex: remove identical statements within if-else branches (8cf304a8) · Commits · 戴 / test

Since the statements are identical, we can safely remove the statements. The commit ("rtlwifi: btcoexist: Fix if == else warnings in halbtc8821a1ant.c") had fixed the statements, but the commit ("rtlwifi: btcoex: follow linux coding style") that converted coding style and upgraded btcoex didn't include the fix. Reported-by:

Gustavo A. R. Silva <gustavo@embeddedor.com> Signed-off-by:

Ping-Ke Shih <pkshih@realtek.com> Signed-off-by:

Kalle Valo <kvalo@codeaurora.org>

drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtc8821a1ant.c

+4 −15

Original line number	Diff line number	Diff line
		@@ -1584,10 +1584,6 @@ static void btc8821a1ant_act_bt_sco_hid_only_busy(struct btc_coexist *btcoexist,
		/* tdma and coex table */
		btc8821a1ant_ps_tdma(btcoexist, NORMAL_EXEC, true, 5);

		if (BT_8821A_1ANT_WIFI_STATUS_NON_CONNECTED_ASSO_AUTH_SCAN ==
		wifi_status)
		btc8821a1ant_coex_table_with_type(btcoexist, NORMAL_EXEC, 1);
		else
		btc8821a1ant_coex_table_with_type(btcoexist, NORMAL_EXEC, 1);
		}

		@@ -1991,16 +1987,9 @@ static void btc8821a1ant_run_coexist_mechanism(struct btc_coexist *btcoexist)
		wifi_rssi_state =
		btc8821a1ant_wifi_rssi_state(btcoexist, 1, 2,
		30, 0);
		if ((wifi_rssi_state == BTC_RSSI_STATE_HIGH) \|\|
		(wifi_rssi_state == BTC_RSSI_STATE_STAY_HIGH)) {
		btc8821a1ant_limited_tx(btcoexist,
		NORMAL_EXEC, 1, 1,
		0, 1);
		} else {
		btc8821a1ant_limited_tx(btcoexist,
		NORMAL_EXEC, 1, 1,
		0, 1);
		}
		} else {
		btc8821a1ant_limited_tx(btcoexist, NORMAL_EXEC,
		0, 0, 0, 0);

Admin message