Marcin Ślusarz <marcin.slusarz@xxxxxxxxx> wrote: > From: Marcin Ślusarz <mslusarz@xxxxxxxxx> > > Needed by the next patch that disables power off operation for one chip. > > Signed-off-by: Marcin Ślusarz <mslusarz@xxxxxxxxx> > Cc: Ping-Ke Shih <pkshih@xxxxxxxxxxx> > Cc: Larry Finger <Larry.Finger@xxxxxxxxxxxx> > Cc: Kalle Valo <kvalo@xxxxxxxxxx> > Cc: linux-wireless@xxxxxxxxxxxxxxx > --- > drivers/net/wireless/realtek/rtw88/ps.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/realtek/rtw88/ps.c b/drivers/net/wireless/realtek/rtw88/ps.c > index add5a20b8432..f9fbc9b3174b 100644 > --- a/drivers/net/wireless/realtek/rtw88/ps.c > +++ b/drivers/net/wireless/realtek/rtw88/ps.c > @@ -26,7 +26,7 @@ static int rtw_ips_pwr_up(struct rtw_dev *rtwdev) > > int rtw_enter_ips(struct rtw_dev *rtwdev) > { > - if (!test_bit(RTW_FLAG_POWERON, rtwdev->flags)) > + if (!test_bit(RTW_FLAG_RUNNING, rtwdev->flags)) RTW_FLAG_POWERON is to maintain power state (i.e. ips) of WiFi card, and prevent entering/leaving IPS twice suddenly. Changing this is confused to me.