Rajkumar Manoharan wrote:
> #ifdef CONFIG_PM
> +static int ieee80211_pre_suspend(struct wiphy *wiphy)
> +{
> + struct ieee80211_local *local = wiphy_priv(wiphy);
> + struct ieee80211_sub_if_data *sdata = NULL;
> + struct ieee80211_if_managed *ifmgd = NULL;
> + struct cfg80211_deauth_request req;
> +
> + list_for_each_entry(sdata, &local->interfaces, list) {
> + ifmgd = &sdata->u.mgd;
Shouldn't the interface type be checked here ?
> + .pre_suspend = ieee80211_pre_suspend,
> .suspend = ieee80211_suspend,
> .resume = ieee80211_resume,
> .scan = ieee80211_scan,
> diff --git a/net/wireless/sysfs.c b/net/wireless/sysfs.c
> index 4294fa2..944d589 100644
> --- a/net/wireless/sysfs.c
> +++ b/net/wireless/sysfs.c
> @@ -91,6 +91,12 @@ static int wiphy_suspend(struct device *dev, pm_message_t state)
>
> rdev->suspend_at = get_seconds();
>
> + if (rdev->ops->pre_suspend) {
> + rtnl_lock();
> + ret = rdev->ops->pre_suspend(&rdev->wiphy);
> + rtnl_unlock();
> + }
> +
Why a 'pre-suspend' ? :)
Can't this be done in __ieee80211_suspend() ?
Sujith
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
