[PATCH 1/4] libertas: remove auto_deep_sleep functionality

Bing Zhao bzhao at marvell.com
Wed Aug 3 23:31:50 EDT 2011


> > Anyway, for users who need this feature we can add a module parameter to enable it with the patch
> below. Please let me know what you think.
> 
> I'd rather we figure out some nl80211/cfg80211 knob for this rather than
> a module parameter; we can certainly add calls and attributes to nl80211
> if we need to.
> 
> Dan

Hi Dan,

In "struct cfg80211_ops" (cfg80211.h) a "set_power_mgmt" handler is defined for IEEE Power Save.

---
 * @set_power_mgmt: Configure WLAN power management. A timeout value of -1
 *      allows the driver to adjust the dynamic ps timeout value.

        int     (*set_power_mgmt)(struct wiphy *wiphy, struct net_device *dev,
                                  bool enabled, int timeout);
---

And the corresponding iw command is "set power_save on|off".

Currently 'ath6kl', 'iwmc3200wifi', 'rndis_wlan', and 'mwifiex' drivers have implemented the set_power_mgmt callback function for power saving handling.

Do you think if we should add new ops for auto deep sleep?

Thanks,
Bing



More information about the libertas-dev mailing list