Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-11-26 Thread Tsuchiya Yuto
On Fri, 2020-11-20 at 13:04 -0800, Brian Norris wrote: > On Fri, Oct 30, 2020 at 1:04 AM Tsuchiya Yuto wrote: > > On Thu, 2020-10-29 at 11:25 -0700, Brian Norris wrote: > > > For the record, Chrome OS supports plenty of mwifiex systems with 8897 > > > (SDIO only) and 8997 (PCIe), with PS enabled,

Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-11-20 Thread Brian Norris
On Fri, Oct 30, 2020 at 1:04 AM Tsuchiya Yuto wrote: > On Thu, 2020-10-29 at 11:25 -0700, Brian Norris wrote: > > For the record, Chrome OS supports plenty of mwifiex systems with 8897 > > (SDIO only) and 8997 (PCIe), with PS enabled, and you're hurting > > those. Your problem sounds to be

Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-10-30 Thread Tsuchiya Yuto
On Thu, 2020-10-29 at 11:25 -0700, Brian Norris wrote: > On Wed, Oct 28, 2020 at 7:04 PM Tsuchiya Yuto wrote: > > > > On Microsoft Surface devices (PCIe-88W8897), the ps_mode causes > > connection unstable, especially with 5GHz APs. Then, it eventually causes > > fw crash. > > > > This commit

Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-10-29 Thread Brian Norris
On Thu, Oct 29, 2020 at 11:37 AM Andy Shevchenko wrote: > And this feeling (that it's a FW issue) what I have. But the problem > here, that Marvell didn't fix and probably won't fix their FW... Sure, I wouldn't hold your breath. So some of these tactics (disabling PS, etc.) may be valid, but you

Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-10-29 Thread Andy Shevchenko
On Thu, Oct 29, 2020 at 8:29 PM Brian Norris wrote: > On Wed, Oct 28, 2020 at 7:04 PM Tsuchiya Yuto wrote: ... > For the record, Chrome OS supports plenty of mwifiex systems with 8897 > (SDIO only) and 8997 (PCIe), with PS enabled, and you're hurting > those. Your problem sounds to be

Re: [PATCH 1/3] mwifiex: disable ps_mode explicitly by default instead

2020-10-29 Thread Brian Norris
On Wed, Oct 28, 2020 at 7:04 PM Tsuchiya Yuto wrote: > > On Microsoft Surface devices (PCIe-88W8897), the ps_mode causes > connection unstable, especially with 5GHz APs. Then, it eventually causes > fw crash. > > This commit disables ps_mode by default instead of enabling it. > > Required code is