[lkml]   [2021]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] wilc1000: Allow setting power_save before driver is initialized
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

- 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)


 \ /
  Last update: 2021-12-15 14:02    [W:0.066 / U:0.752 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site