Commit a0d46f7a authored Sep 25, 2019 by Denis Efremov Committed by Kalle Valo Oct 01, 2019

rtlwifi: Remove excessive check in _rtl_ps_inactive_ps()



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>

parent 3f936169

drivers/net/wireless/realtek/rtlwifi/ps.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -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);
		}