wifi: rtw88: add flag check before enter or leave IPS

Enter or leave IPS controlled by mac80211 before driver support HW
scan. After support HW scan, driver need to control IPS before start
HW scan and scan complete, but mac80211 also ask driver enter or leave
IPS. Therefore, we add flag check in IPS to prevent entering or
leaving IPS twice.

Signed-off-by: Chih-Kang Chang <[email protected]>
Signed-off-by: Ping-Ke Shih <[email protected]>
Signed-off-by: Kalle Valo <[email protected]>
Link: https://lore.kernel.org/r/[email protected]

diff --git a/drivers/net/wireless/realtek/rtw88/ps.c b/drivers/net/wireless/realtek/rtw88/ps.c
index bfa64c038f5f0a35d6721eba7f4976ce414588aa..c93da743681fc51954f635f4c1455f70a6e77196 100644 (file)
--- a/drivers/net/wireless/realtek/rtw88/ps.c
+++ b/drivers/net/wireless/realtek/rtw88/ps.c
@@ -19,14 +19,14 @@ static int rtw_ips_pwr_up(struct rtw_dev *rtwdev)
                rtw_err(rtwdev, "leave idle state failed\n");
 
        rtw_set_channel(rtwdev);
-       clear_bit(RTW_FLAG_INACTIVE_PS, rtwdev->flags);
 
        return ret;
 }
 
 int rtw_enter_ips(struct rtw_dev *rtwdev)
 {
-       set_bit(RTW_FLAG_INACTIVE_PS, rtwdev->flags);
+       if (test_and_set_bit(RTW_FLAG_INACTIVE_PS, rtwdev->flags))
+               return 0;
 
        rtw_coex_ips_notify(rtwdev, COEX_IPS_ENTER);
 
@@ -50,6 +50,9 @@ int rtw_leave_ips(struct rtw_dev *rtwdev)
 {
        int ret;
 
+       if (!test_and_clear_bit(RTW_FLAG_INACTIVE_PS, rtwdev->flags))
+               return 0;
+
        rtw_hci_link_ps(rtwdev, false);
 
        ret = rtw_ips_pwr_up(rtwdev);