iwlegacy: remove ctx interface_modes
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 3 Feb 2012 16:31:50 +0000 (17:31 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 6 Feb 2012 19:56:05 +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/3945-mac.c
drivers/net/wireless/iwlegacy/4965-mac.c
drivers/net/wireless/iwlegacy/common.c
drivers/net/wireless/iwlegacy/common.h

index 52cac0cd1064aeed594c42cee392bbffd888ecee..7c81f5aaf7a349406bb920ee0ba4e5826e85c226 100644 (file)
@@ -3557,7 +3557,8 @@ il3945_setup_mac(struct il_priv *il)
        /* Tell mac80211 our characteristics */
        hw->flags = IEEE80211_HW_SIGNAL_DBM | IEEE80211_HW_SPECTRUM_MGMT;
 
-       hw->wiphy->interface_modes = il->ctx.interface_modes;
+       hw->wiphy->interface_modes =
+           BIT(NL80211_IFTYPE_STATION) | BIT(NL80211_IFTYPE_ADHOC);
 
        hw->wiphy->flags |=
            WIPHY_FLAG_CUSTOM_REGULATORY | WIPHY_FLAG_DISABLE_BEACON_HINTS |
@@ -3617,9 +3618,6 @@ il3945_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 
        il->cmd_queue = IL39_CMD_QUEUE_NUM;
 
-       il->ctx.interface_modes =
-           BIT(NL80211_IFTYPE_STATION) | BIT(NL80211_IFTYPE_ADHOC);
-
        /*
         * Disabling hardware scan means that mac80211 will perform scans
         * "the hard way", rather than using device's scan.
index 35d868c4ba92d3847658140d0b7d56626ab1b02e..1dd001475522cf2ce3886da688d8e8063f6e7dc0 100644 (file)
@@ -5454,8 +5454,8 @@ il4965_mac_setup_register(struct il_priv *il, u32 max_probe_length)
        hw->sta_data_size = sizeof(struct il_station_priv);
        hw->vif_data_size = sizeof(struct il_vif_priv);
 
-       hw->wiphy->interface_modes |= il->ctx.interface_modes;
-       hw->wiphy->interface_modes |= il->ctx.exclusive_interface_modes;
+       hw->wiphy->interface_modes =
+           BIT(NL80211_IFTYPE_STATION) | BIT(NL80211_IFTYPE_ADHOC);
 
        hw->wiphy->flags |=
            WIPHY_FLAG_CUSTOM_REGULATORY | WIPHY_FLAG_DISABLE_BEACON_HINTS;
@@ -6141,8 +6141,6 @@ il4965_pci_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
        il->ctx.is_active = true;
        il->ctx.ac_to_fifo = il4965_bss_ac_to_fifo;
        il->ctx.ac_to_queue = il4965_bss_ac_to_queue;
-       il->ctx.exclusive_interface_modes = BIT(NL80211_IFTYPE_ADHOC);
-       il->ctx.interface_modes = BIT(NL80211_IFTYPE_STATION);
 
        SET_IEEE80211_DEV(hw, &pdev->dev);
 
index ac269856be14f260efd0065ca65f5c0b414abe28..7f4c6bf419950f925a60ed0b9b40d83d82fcd356 100644 (file)
@@ -4566,7 +4566,6 @@ il_mac_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
        struct il_priv *il = hw->priv;
        struct il_vif_priv *vif_priv = (void *)vif->drv_priv;
        int err;
-       u32 modes;
 
        D_MAC80211("enter: type %d, addr %pM\n", vif->type, vif->addr);
 
@@ -4578,15 +4577,7 @@ il_mac_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
                goto out;
        }
 
-       /* check if busy context is exclusive */
-       if (il->ctx.vif &&
-           (il->ctx.exclusive_interface_modes & BIT(il->ctx.vif->type))) {
-               err = -EINVAL;
-               goto out;
-       }
-
-       modes = il->ctx.interface_modes | il->ctx.exclusive_interface_modes;
-       if (!(modes & BIT(vif->type))) {
+       if (il->ctx.vif) {
                err = -EOPNOTSUPP;
                goto out;
        }
@@ -4979,10 +4970,10 @@ il_mac_change_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
 {
        struct il_priv *il = hw->priv;
        struct il_rxon_context *ctx = il_rxon_ctx_from_vif(vif);
-       u32 modes;
        int err;
 
-       newtype = ieee80211_iftype_p2p(newtype, newp2p);
+       if (newp2p)
+               return -EOPNOTSUPP;
 
        mutex_lock(&il->mutex);
 
@@ -4995,22 +4986,10 @@ il_mac_change_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                goto out;
        }
 
-       modes = ctx->interface_modes | ctx->exclusive_interface_modes;
-       if (!(modes & BIT(newtype))) {
-               err = -EOPNOTSUPP;
-               goto out;
-       }
-
-       if ((il->ctx.exclusive_interface_modes & BIT(il->ctx.vif->type)) ||
-           (il->ctx.exclusive_interface_modes & BIT(newtype))) {
-               err = -EINVAL;
-               goto out;
-       }
-
        /* success */
        il_teardown_interface(il, vif, true);
        vif->type = newtype;
-       vif->p2p = newp2p;
+       vif->p2p = false;
        err = il_setup_interface(il, ctx);
        WARN_ON(err);
        /*
index a872175f164c0bd8542f5edfdac133ff850420f3..eddf22d4f1b2bf02b66e30f60bb39ed6edd73a4f 100644 (file)
@@ -1165,8 +1165,6 @@ struct il_rxon_context {
         */
        bool always_active, is_active;
 
-       u32 interface_modes, exclusive_interface_modes;
-
        struct il_qos_info qos_data;
 
        struct {