On Tue, 2021-02-23 at 17:19 +0000, Ajay.Kathat@microchip.com wrote:

Hi David,

On 23/02/21 9:53 pm, David Mosberger-Tang wrote:

quoted

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