| Message ID | 1478437019-24689-1-git-send-email-hofrat@osadl.org (mailing list archive) |
|---|---|
| State | RFC |
| Delegated to: | Kalle Valo |
| Headers | show |
diff --git a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c index 91cc139..588c8ed 100644 --- a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c +++ b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c @@ -981,7 +981,7 @@ void exhalbtc_set_ant_num(struct rtl_priv *rtlpriv, u8 type, u8 ant_num) if ((gl_bt_coexist.board_info.pg_ant_num == 2) && (gl_bt_coexist.board_info.btdm_ant_num == 1)) gl_bt_coexist.board_info.btdm_ant_pos = - BTC_ANTENNA_AT_MAIN_PORT; + BTC_ANTENNA_AT_AUX_PORT; else gl_bt_coexist.board_info.btdm_ant_pos = BTC_ANTENNA_AT_MAIN_PORT;
The port assignment in the if case should be to AUX not MAIN. Fixes: commit baa170229095 ("rtlwifi: btcoexist: Implement antenna selection") Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org> --- problem located by coccinelle in: drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c:exhalbtc_set_ant_num() 973 /* The antenna position: 974 * Main (default) or Aux for pgAntNum=2 && btdmAntNum =1. 975 * The antenna position should be determined by 976 * auto-detect mechanism. 977 * The following is assumed to main, 978 * and those must be modified 979 * if y auto-detect mechanism is ready 980 */ 981 if ((gl_bt_coexist.board_info.pg_ant_num == 2) && 982 (gl_bt_coexist.board_info.btdm_ant_num == 1)) 983 gl_bt_coexist.board_info.btdm_ant_pos = 984 BTC_ANTENNA_AT_MAIN_PORT; 985 else 986 gl_bt_coexist.board_info.btdm_ant_pos = 987 BTC_ANTENNA_AT_MAIN_PORT; (line number from 4.9.0-rc2 linux-next 20161028) the if and else branch here are the same but the comment seems to indicate that the first case should be the AUX port and not the MAIN port here (the second sentence in the comment though is not really clear to me). If the intent is to set it to MAIN unconditionally and then let autodetect fix it then the if/else construct is useless. Looks like a cut&past bug, but this needs a check by someone who knows the details of the device. Patch was compile tested with: x86_64_defconfig + RTL8723AE=m (implies CONFIG_RTLBTCOEXIST) Patch is against 4.9.0-rc2 (localversion-next is next-20161028) drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)