Re: [PATCH] rtlwifi: Remove excessive check in _rtl_ps_inactive_ps()

2019-10-01 Thread Kalle Valo
Denis Efremov wrote: > 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 > Acked-by: Larry Finger Patch applied to wireless-drivers-next.git, t

Re: [PATCH] rtlwifi: Remove excessive check in _rtl_ps_inactive_ps()

2019-09-25 Thread Larry Finger
On 9/25/19 3:58 PM, Denis Efremov wrote: 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 --- drivers/net/wireless/realtek/rtlwifi/ps.c | 3 +-- 1 fi