From: Ping-Ke Shih <[email protected]>

Return value may be false in some situations.

Signed-off-by: Ping-Ke Shih <[email protected]>
Signed-off-by: Larry Finger <[email protected]>
Cc: Yan-Hsuan Chuang <[email protected]>
Cc: Birming Chiu <[email protected]>
Cc: Shaofu <[email protected]>
Cc: Steven Ting <[email protected]>
---
 .../realtek/rtlwifi/btcoexist/halbtcoutsrc.c       | 24 +++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c 
b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c
index eaa916a0727b..d8fb2442f795 100644
--- a/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c
+++ b/drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c
@@ -381,6 +381,7 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
        u32 *u32_tmp = (u32 *)out_buf;
        u8 *u8_tmp = (u8 *)out_buf;
        bool tmp = false;
+       bool ret = true;
 
        if (!halbtc_is_bt_coexist_available(btcoexist))
                return false;
@@ -388,9 +389,11 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
        switch (get_type) {
        case BTC_GET_BL_HS_OPERATION:
                *bool_tmp = false;
+               ret = false;
                break;
        case BTC_GET_BL_HS_CONNECTING:
                *bool_tmp = false;
+               ret = false;
                break;
        case BTC_GET_BL_WIFI_CONNECTED:
                if (rtlpriv->mac80211.opmode == NL80211_IFTYPE_STATION &&
@@ -429,11 +432,16 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
                *bool_tmp = false;
                break;
        case BTC_GET_BL_WIFI_UNDER_5G:
-               /* TODO */
-               *bool_tmp = false;
+               if (rtlhal->current_bandtype == BAND_ON_5G)
+                       *bool_tmp = true;
+               else
+                       *bool_tmp = false;
                break;
        case BTC_GET_BL_WIFI_AP_MODE_ENABLE:
-               *bool_tmp = false;
+               if (mac->opmode == NL80211_IFTYPE_AP)
+                       *bool_tmp = true;
+               else
+                       *bool_tmp = false;
                break;
        case BTC_GET_BL_WIFI_ENABLE_ENCRYPTION:
                if (NO_ENCRYPTION == rtlpriv->sec.pairwise_enc_algorithm)
@@ -460,8 +468,8 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
                *s32_tmp = halbtc_get_wifi_rssi(rtlpriv);
                break;
        case BTC_GET_S4_HS_RSSI:
-               /* TODO */
-               *s32_tmp = halbtc_get_wifi_rssi(rtlpriv);
+               *s32_tmp = 0;
+               ret = false;
                break;
        case BTC_GET_U4_WIFI_BW:
                *u32_tmp = halbtc_get_wifi_bw(btcoexist);
@@ -491,7 +499,8 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
                *u8_tmp = halbtc_get_wifi_central_chnl(btcoexist);
                break;
        case BTC_GET_U1_WIFI_HS_CHNL:
-               *u8_tmp = 1;
+               *u8_tmp = 0;
+               ret = false;
                break;
        case BTC_GET_U1_AP_NUM:
                /* driver do not know AP num,
@@ -512,10 +521,11 @@ static bool halbtc_get(void *void_btcoexist, u8 get_type, 
void *out_buf)
                break;
 
        default:
+               ret = false;
                break;
        }
 
-       return true;
+       return ret;
 }
 
 static bool halbtc_set(void *void_btcoexist, u8 set_type, void *in_buf)
-- 
2.12.3

Reply via email to