Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Sender: Larry Finger <larry.finger@gmail.com>
Message-ID: <813e9c3d-7fbd-7589-ebd0-6c61f03f970b@lwfinger.net>
Date:   Thu, 3 Aug 2023 17:13:58 -0500
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.1
Content-Language: en-US
To:     Pkshih <pkshih@realtek.com>
Cc:     linux-wireless <linux-wireless@vger.kernel.org>
From:   Larry Finger <Larry.Finger@lwfinger.net>
Subject: Regression in rtw89
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Ping-Ke,

A user of my GitHub repo has reported a regression beginning with commit 
26a125f550a3 ("wifi: rtw89: correct PS calculation for SUPPORTS_DYNAMIC_PS").
The user finds interference between wifi and the BT device that is part of the 
package.

We were able to determine that loading rtw89_core with the "disable_ps_mode=y" 
option does fix the problem. Our discussion of the problem can be seen at 
https://github.com/lwfinger/rtw89/issues/262#issuecomment-1664658764.

Reading the patch in question, it seems that a vif count of 1 is not sufficient 
to enable PS, and that it is also necessary to make sure that BT is not active 
as well.

Thanks,

Larry