rtlwifi: Remove excessive check in _rtl_ps_inactive_ps()
authorDenis Efremov <efremov@linux.com>
Wed, 25 Sep 2019 20:58:58 +0000 (23:58 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 1 Oct 2019 09:20:59 +0000 (12:20 +0300)
There is no need to check "rtlhal->interface == INTF_PCI" twice in
_rtl_ps_inactive_ps(). The nested check is always true. Thus, the
expression can be simplified.

Signed-off-by: Denis Efremov <efremov@linux.com>
Acked-by: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/realtek/rtlwifi/ps.c

index 70f04c2..6a81275 100644 (file)
@@ -161,8 +161,7 @@ static void _rtl_ps_inactive_ps(struct ieee80211_hw *hw)
        if (ppsc->inactive_pwrstate == ERFON &&
            rtlhal->interface == INTF_PCI) {
                if ((ppsc->reg_rfps_level & RT_RF_OFF_LEVL_ASPM) &&
-                   RT_IN_PS_LEVEL(ppsc, RT_PS_LEVEL_ASPM) &&
-                   rtlhal->interface == INTF_PCI) {
+                   RT_IN_PS_LEVEL(ppsc, RT_PS_LEVEL_ASPM)) {
                        rtlpriv->intf_ops->disable_aspm(hw);
                        RT_CLEAR_PS_LEVEL(ppsc, RT_PS_LEVEL_ASPM);
                }