On Tue, 2021-02-23 at 17:19 +0000, Ajay.Kathat@xxxxxxxxxxxxx wrote: > Hi David, > > On 23/02/21 9:53 pm, David Mosberger-Tang wrote: > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > > > OK, the problem below is caused by wilc_set_power_mgmt(). If I change > > that function into a no-op, the driver actually works! Does this make > > any sense to you? From what I saw so far, it looks like relevant code > > is pretty much identical to the one in the linux-at91 tree and that one > > works fine. > > > > As I understand you are testing with default powersave mode enabled. One > approach could be to disable the default powersave mode by compiling > with CFG80211_DEFAULT_PS disabled. When its disabled, .set_power_mgmt cb > is called with powersave disable. > If the powersave mode is enabled, the chip need to be wakeup by > following the wakeup sequence. We need to bring in chip_wake() API > changes, to set registers specific for WILC1000. I have plans to port > these changes to support PSM mode but it will take few weeks. Indeed. Sure, I'll just compile with powersave mode disabled for now until you've had a chance to bring forward the missing pieces. Thanks! --david
