iwlegacy: remove il_setup_interface()
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 3 Feb 2012 16:31:56 +0000 (17:31 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 6 Feb 2012 19:56:10 +0000 (14:56 -0500)
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/iwlegacy/common.c

index 13bd3a8c21b1ddc33d2113d1c38e3b83b0d9972f..ea2fa97fa36b112c4601ba21604b8b3d8620c362 100644 (file)
@@ -4533,23 +4533,6 @@ il_set_mode(struct il_priv *il, struct il_rxon_context *ctx)
        return il_commit_rxon(il, ctx);
 }
 
-static int
-il_setup_interface(struct il_priv *il, struct il_rxon_context *ctx)
-{
-       struct ieee80211_vif *vif = ctx->vif;
-
-       lockdep_assert_held(&il->mutex);
-
-       /*
-        * This variable will be correct only when there's just
-        * a single context, but all code using it is for hardware
-        * that supports only one context.
-        */
-       il->iw_mode = vif->type;
-
-       return il_set_mode(il, ctx);
-}
-
 int
 il_mac_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 {
@@ -4574,8 +4557,9 @@ il_mac_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 
        vif_priv->ctx = &il->ctx;
        il->ctx.vif = vif;
+       il->iw_mode = vif->type;
 
-       err = il_setup_interface(il, &il->ctx);
+       err = il_set_mode(il, &il->ctx);
        if (err) {
                il->ctx.vif = NULL;
                il->iw_mode = NL80211_IFTYPE_STATION;
@@ -4978,7 +4962,7 @@ il_mac_change_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
        il_teardown_interface(il, vif, true);
        vif->type = newtype;
        vif->p2p = false;
-       err = il_setup_interface(il, ctx);
+       err = il_set_mode(il, ctx);
        WARN_ON(err);
        /*
         * We've switched internally, but submitting to the