rtlwifi: rtl8723ae: fix comparison pointer to bool warning in phy.c
authorZheng Bin <zhengbin13@huawei.com>
Thu, 10 Sep 2020 14:16:42 +0000 (22:16 +0800)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 16 Sep 2020 06:12:11 +0000 (09:12 +0300)
Fixes coccicheck warning:

drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c:191:5-13: WARNING: Comparison to bool
drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c:205:5-13: WARNING: Comparison to bool
drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c:211:5-13: WARNING: Comparison to bool
drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c:625:5-30: WARNING: Comparison to bool

Signed-off-by: Zheng Bin <zhengbin13@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200910141642.127006-4-zhengbin13@huawei.com
drivers/net/wireless/realtek/rtlwifi/rtl8723ae/phy.c

index 5ba645bc46dc9428cb4f36dcf2efded8b90d4eb8..fa0eed434d4f668cb0d986d807fd6a98a952092d 100644 (file)
@@ -188,7 +188,7 @@ static bool _rtl8723e_phy_bb8192c_config_parafile(struct ieee80211_hw *hw)
        rtl_dbg(rtlpriv, COMP_INIT, DBG_TRACE, "\n");
        rtstatus = _rtl8723e_phy_config_bb_with_headerfile(hw,
                                                BASEBAND_CONFIG_PHY_REG);
-       if (rtstatus != true) {
+       if (!rtstatus) {
                pr_err("Write BB Reg Fail!!\n");
                return false;
        }
@@ -202,13 +202,13 @@ static bool _rtl8723e_phy_bb8192c_config_parafile(struct ieee80211_hw *hw)
                rtstatus = _rtl8723e_phy_config_bb_with_pgheaderfile(hw,
                                        BASEBAND_CONFIG_PHY_REG);
        }
-       if (rtstatus != true) {
+       if (!rtstatus) {
                pr_err("BB_PG Reg Fail!!\n");
                return false;
        }
        rtstatus =
          _rtl8723e_phy_config_bb_with_headerfile(hw, BASEBAND_CONFIG_AGC_TAB);
-       if (rtstatus != true) {
+       if (!rtstatus) {
                pr_err("AGC Table Fail\n");
                return false;
        }
@@ -622,7 +622,7 @@ void rtl8723e_phy_set_txpower_level(struct ieee80211_hw *hw, u8 channel)
        struct rtl_efuse *rtlefuse = rtl_efuse(rtl_priv(hw));
        u8 cckpowerlevel[2], ofdmpowerlevel[2];
 
-       if (rtlefuse->txpwr_fromeprom == false)
+       if (!rtlefuse->txpwr_fromeprom)
                return;
        _rtl8723e_get_txpower_index(hw, channel,
                                    &cckpowerlevel[0], &ofdmpowerlevel[0]);