Denis Efremov <efremov@xxxxxxxxx> 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 <efremov@xxxxxxxxx> > Acked-by: Larry Finger <Larry.Finger@xxxxxxxxxxxx> Patch applied to wireless-drivers-next.git, thanks. a0d46f7a0fa5 rtlwifi: Remove excessive check in _rtl_ps_inactive_ps() -- https://patchwork.kernel.org/patch/11161357/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches