Re: [PATCH] wilc1000: Allow setting power_save before driver is initialized

[<Ajay.Kathat@microchip.com>]

On 13/12/21 02:50, David Mosberger-Tang wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Unfortunately, this patch doesn't seem to be sufficient.  From what I
> can tell, if power-save mode is turned on before a station is
> associated with an access-point, there is no actual power savings.  If
> I issue the command after the station is associated, it works perfectly
> fine.
>
> Ajay, does this make sense to you?


I think the patch has no effect because wilc_init_fw_config() gets 
called before wilc_set_power_mgmt().

Also, you had mentioned that to enable the PSM(power-save mode), the 
toggling of PS mode is required that means the previous set_power_mgmt() 
was successful. I believe ".set_power_mgmt" cfg80211_ops doesn't get 
called when there is no change from the previous state.

Power-save mode is allowed to be enabled irrespective of station 
association state. Before association, the power consumption should be 
less with PSM enabled compared to PSM disabled. The WLAN automatic power 
save delivery gets enabled after the association with AP.

To check the power measurement before association,  test without 
wpa_supplicant.


Steps:
- load the module
- ifconfig wlan0 up
- iw dev wlan0 set power_save off (check the pwr measurement after PS 
mode disabled)
- iw dev wlan0 set power_save on (check the pwr measurement after PS 
mode enable)


Regards,
Ajay