mwl8k: s/IEEE80211_ADDR_LEN/ETH_ALEN/g
[pandora-kernel.git] / drivers / net / wireless / mwl8k.c
index a9a9704..e8febab 100644 (file)
@@ -38,8 +38,6 @@ static DEFINE_PCI_DEVICE_TABLE(mwl8k_table) = {
 };
 MODULE_DEVICE_TABLE(pci, mwl8k_table);
 
-#define IEEE80211_ADDR_LEN                     ETH_ALEN
-
 /* Register definitions */
 #define MWL8K_HIU_GEN_PTR                      0x00000c10
 #define  MWL8K_MODE_STA                                0x0000005a
@@ -58,7 +56,6 @@ MODULE_DEVICE_TABLE(pci, mwl8k_table);
 #define MWL8K_HIU_H2A_INTERRUPT_STATUS_MASK    0x00000c28
 #define  MWL8K_H2A_INT_DUMMY                   (1 << 20)
 #define  MWL8K_H2A_INT_RESET                   (1 << 15)
-#define  MWL8K_H2A_INT_PS                      (1 << 2)
 #define  MWL8K_H2A_INT_DOORBELL                        (1 << 1)
 #define  MWL8K_H2A_INT_PPA_READY               (1 << 0)
 
@@ -161,10 +158,8 @@ struct mwl8k_priv {
 
        /* lock held over TX and TX reap */
        spinlock_t tx_lock;
-       u32 int_mask;
 
        struct ieee80211_vif *vif;
-       struct list_head vif_list;
 
        struct ieee80211_channel *current_channel;
 
@@ -173,10 +168,8 @@ struct mwl8k_priv {
        dma_addr_t cookie_dma;
 
        u16 num_mcaddrs;
-       u16 region_code;
        u8 hw_rev;
        __le32 fw_rev;
-       u32 wep_enabled;
 
        /*
         * Running count of TX packets in flight, to avoid
@@ -204,7 +197,7 @@ struct mwl8k_priv {
 
        /* XXX need to convert this to handle multiple interfaces */
        bool capture_beacon;
-       u8 capture_bssid[IEEE80211_ADDR_LEN];
+       u8 capture_bssid[ETH_ALEN];
        struct sk_buff *beacon_skb;
 
        /*
@@ -226,8 +219,6 @@ struct mwl8k_priv {
 
 /* Per interface specific private data */
 struct mwl8k_vif {
-       struct list_head node;
-
        /* backpointer to parent config block */
        struct mwl8k_priv *priv;
 
@@ -235,8 +226,8 @@ struct mwl8k_vif {
        struct ieee80211_bss_conf bss_info;
 
        /* BSSID of AP or IBSS */
-       u8      bssid[IEEE80211_ADDR_LEN];
-       u8      mac_addr[IEEE80211_ADDR_LEN];
+       u8      bssid[ETH_ALEN];
+       u8      mac_addr[ETH_ALEN];
 
        /*
         * Subset of supported legacy rates.
@@ -247,21 +238,14 @@ struct mwl8k_vif {
        /* number of supported legacy rates */
        u8      legacy_nrates;
 
-       /* Number of supported MCS rates. Work in progress */
-       u8      mcs_nrates;
-
         /* Index into station database.Returned by update_sta_db call */
        u8      peer_id;
 
        /* Non AMPDU sequence number assigned by driver */
        u16     seqno;
-
-       /* Note:There is no channel info,
-        * refer to the master channel info in priv
-        */
 };
 
-#define MWL8K_VIF(_vif) (struct mwl8k_vif *)(&((_vif)->drv_priv))
+#define MWL8K_VIF(_vif) ((struct mwl8k_vif *)&((_vif)->drv_priv))
 
 static const struct ieee80211_channel mwl8k_channels[] = {
        { .center_freq = 2412, .hw_value = 1, },
@@ -681,11 +665,9 @@ struct ewc_ht_info {
 
 /* Peer Entry flags - used to define the type of the peer node */
 #define MWL8K_PEER_TYPE_ACCESSPOINT    2
-#define MWL8K_PEER_TYPE_ADHOC_STATION  4
 
 #define MWL8K_IEEE_LEGACY_DATA_RATES   12
 #define MWL8K_MCS_BITMAP_SIZE          16
-#define pad_size                       16
 
 struct peer_capability_info {
        /* Peer type - AP vs. STA.  */
@@ -707,7 +689,7 @@ struct peer_capability_info {
 
        /* HT rate table. Intersection of our rates and peer rates.  */
        __u8    ht_rates[MWL8K_MCS_BITMAP_SIZE];
-       __u8    pad[pad_size];
+       __u8    pad[16];
 
        /* If set, interoperability mode, no proprietary extensions.  */
        __u8    interop;
@@ -717,15 +699,6 @@ struct peer_capability_info {
 } __attribute__((packed));
 
 /* Inline functions to manipulate QoS field in data descriptor.  */
-static inline u16 mwl8k_qos_setbit_tid(u16 qos, u8 tid)
-{
-       u16 val_mask = 0x000f;
-       u16 qos_mask = ~val_mask;
-
-       /* TID bits 0-3 */
-       return (qos & qos_mask) | (tid & val_mask);
-}
-
 static inline u16 mwl8k_qos_setbit_eosp(u16 qos)
 {
        u16 val_mask = 1 << 4;
@@ -810,7 +783,7 @@ static inline struct sk_buff *mwl8k_add_dma_header(struct sk_buff *skb)
                memmove(&tr->wh, wh, hdrlen);
 
        /* Clear addr4 */
-       memset(tr->wh.addr4, 0, IEEE80211_ADDR_LEN);
+       memset(tr->wh.addr4, 0, ETH_ALEN);
 
        /*
         * Firmware length is the length of the fully formed "802.11
@@ -826,9 +799,7 @@ static inline struct sk_buff *mwl8k_add_dma_header(struct sk_buff *skb)
 /*
  * Packet reception.
  */
-#define MWL8K_RX_CTRL_KEY_INDEX_MASK   0x30
 #define MWL8K_RX_CTRL_OWNED_BY_HOST    0x02
-#define MWL8K_RX_CTRL_AMPDU            0x01
 
 struct mwl8k_rx_desc {
        __le16 pkt_len;
@@ -979,7 +950,7 @@ static inline void mwl8k_save_beacon(struct mwl8k_priv *priv,
                                                        struct sk_buff *skb)
 {
        priv->capture_beacon = false;
-       memset(priv->capture_bssid, 0, IEEE80211_ADDR_LEN);
+       memset(priv->capture_bssid, 0, ETH_ALEN);
 
        /*
         * Use GFP_ATOMIC as rxq_process is called from
@@ -1012,6 +983,8 @@ static int rxq_process(struct ieee80211_hw *hw, int index, int limit)
                rmb();
 
                skb = rxq->rx_skb[rxq->rx_head];
+               if (skb == NULL)
+                       break;
                rxq->rx_skb[rxq->rx_head] = NULL;
 
                rxq->rx_head = (rxq->rx_head + 1) % MWL8K_RX_DESCS;
@@ -1047,7 +1020,8 @@ static int rxq_process(struct ieee80211_hw *hw, int index, int limit)
                status.flag = 0;
                status.band = IEEE80211_BAND_2GHZ;
                status.freq = ieee80211_channel_to_frequency(rx_desc->channel);
-               ieee80211_rx_irqsafe(hw, skb, &status);
+               memcpy(IEEE80211_SKB_RXCB(skb), &status, sizeof(status));
+               ieee80211_rx_irqsafe(hw, skb);
 
                processed++;
        }
@@ -1070,8 +1044,6 @@ enum {
 
 /* Transmit packet ACK policy */
 #define MWL8K_TXD_ACK_POLICY_NORMAL            0
-#define MWL8K_TXD_ACK_POLICY_NONE              1
-#define MWL8K_TXD_ACK_POLICY_NO_EXPLICIT       2
 #define MWL8K_TXD_ACK_POLICY_BLOCKACK          3
 
 #define GET_TXQ(_ac) (\
@@ -1080,20 +1052,11 @@ enum {
                ((_ac) == WME_AC_BK) ? MWL8K_WME_AC_BK : \
                MWL8K_WME_AC_BE)
 
-#define MWL8K_TXD_STATUS_IDLE                  0x00000000
-#define MWL8K_TXD_STATUS_USED                  0x00000001
 #define MWL8K_TXD_STATUS_OK                    0x00000001
 #define MWL8K_TXD_STATUS_OK_RETRY              0x00000002
 #define MWL8K_TXD_STATUS_OK_MORE_RETRY         0x00000004
 #define MWL8K_TXD_STATUS_MULTICAST_TX          0x00000008
-#define MWL8K_TXD_STATUS_BROADCAST_TX          0x00000010
-#define MWL8K_TXD_STATUS_FAILED_LINK_ERROR     0x00000020
-#define MWL8K_TXD_STATUS_FAILED_EXCEED_LIMIT   0x00000040
-#define MWL8K_TXD_STATUS_FAILED_AGING          0x00000080
-#define MWL8K_TXD_STATUS_HOST_CMD              0x40000000
 #define MWL8K_TXD_STATUS_FW_OWNED              0x80000000
-#define  MWL8K_TXD_SOFTSTALE                           0x80
-#define  MWL8K_TXD_SOFTSTALE_MGMT_RETRY                        0x01
 
 struct mwl8k_tx_desc {
        __le32 status;
@@ -1102,7 +1065,7 @@ struct mwl8k_tx_desc {
        __le16 qos_control;
        __le32 pkt_phys_addr;
        __le16 pkt_len;
-       __u8 dest_MAC_addr[IEEE80211_ADDR_LEN];
+       __u8 dest_MAC_addr[ETH_ALEN];
        __le32 next_tx_desc_phys_addr;
        __le32 reserved;
        __le16 rate_info;
@@ -1276,12 +1239,10 @@ static int mwl8k_tx_wait_empty(struct ieee80211_hw *hw, u32 delay_ms)
        return 0;
 }
 
-#define MWL8K_TXD_OK   (MWL8K_TXD_STATUS_OK | \
-                        MWL8K_TXD_STATUS_OK_RETRY | \
-                        MWL8K_TXD_STATUS_OK_MORE_RETRY)
-#define MWL8K_TXD_SUCCESS(stat)                ((stat) & MWL8K_TXD_OK)
-#define MWL8K_TXD_FAIL_RETRY(stat)     \
-       ((stat) & (MWL8K_TXD_STATUS_FAILED_EXCEED_LIMIT))
+#define MWL8K_TXD_SUCCESS(status)                              \
+       ((status) & (MWL8K_TXD_STATUS_OK |                      \
+                    MWL8K_TXD_STATUS_OK_RETRY |                \
+                    MWL8K_TXD_STATUS_OK_MORE_RETRY))
 
 static void mwl8k_txq_reclaim(struct ieee80211_hw *hw, int index, int force)
 {
@@ -1591,6 +1552,9 @@ static int mwl8k_post_cmd(struct ieee80211_hw *hw, struct mwl8k_cmd_pkt *cmd)
        timeout = wait_for_completion_timeout(&cmd_wait,
                                msecs_to_jiffies(MWL8K_CMD_TIMEOUT_MS));
 
+       pci_unmap_single(priv->pdev, dma_addr, dma_size,
+                                       PCI_DMA_BIDIRECTIONAL);
+
        result = &cmd->result;
        if (!timeout) {
                spin_lock_irq(&priv->fw_lock);
@@ -1610,8 +1574,6 @@ static int mwl8k_post_cmd(struct ieee80211_hw *hw, struct mwl8k_cmd_pkt *cmd)
                               *result);
        }
 
-       pci_unmap_single(priv->pdev, dma_addr, dma_size,
-                                       PCI_DMA_BIDIRECTIONAL);
        return rc;
 }
 
@@ -1623,7 +1585,7 @@ struct mwl8k_cmd_get_hw_spec {
        __u8 hw_rev;
        __u8 host_interface;
        __le16 num_mcaddrs;
-       __u8 perm_addr[IEEE80211_ADDR_LEN];
+       __u8 perm_addr[ETH_ALEN];
        __le16 region_code;
        __le32 fw_rev;
        __le32 ps_cookie;
@@ -1654,20 +1616,19 @@ static int mwl8k_cmd_get_hw_spec(struct ieee80211_hw *hw)
        memset(cmd->perm_addr, 0xff, sizeof(cmd->perm_addr));
        cmd->ps_cookie = cpu_to_le32(priv->cookie_dma);
        cmd->rx_queue_ptr = cpu_to_le32(priv->rxq[0].rx_desc_dma);
-       cmd->num_tx_queues = MWL8K_TX_QUEUES;
+       cmd->num_tx_queues = cpu_to_le32(MWL8K_TX_QUEUES);
        for (i = 0; i < MWL8K_TX_QUEUES; i++)
                cmd->tx_queue_ptrs[i] = cpu_to_le32(priv->txq[i].tx_desc_dma);
-       cmd->num_tx_desc_per_queue = MWL8K_TX_DESCS;
-       cmd->total_rx_desc = MWL8K_RX_DESCS;
+       cmd->num_tx_desc_per_queue = cpu_to_le32(MWL8K_TX_DESCS);
+       cmd->total_rx_desc = cpu_to_le32(MWL8K_RX_DESCS);
 
        rc = mwl8k_post_cmd(hw, &cmd->header);
 
        if (!rc) {
                SET_IEEE80211_PERM_ADDR(hw, cmd->perm_addr);
                priv->num_mcaddrs = le16_to_cpu(cmd->num_mcaddrs);
-               priv->fw_rev = cmd->fw_rev;
+               priv->fw_rev = le32_to_cpu(cmd->fw_rev);
                priv->hw_rev = cmd->hw_rev;
-               priv->region_code = le16_to_cpu(cmd->region_code);
        }
 
        kfree(cmd);
@@ -1681,7 +1642,7 @@ struct mwl8k_cmd_mac_multicast_adr {
        struct mwl8k_cmd_pkt header;
        __le16 action;
        __le16 numaddr;
-       __u8 addr[1][IEEE80211_ADDR_LEN];
+       __u8 addr[1][ETH_ALEN];
 };
 
 #define MWL8K_ENABLE_RX_MULTICAST 0x000F
@@ -1692,7 +1653,7 @@ static int mwl8k_cmd_mac_multicast_adr(struct ieee80211_hw *hw,
        struct mwl8k_cmd_mac_multicast_adr *cmd;
        int index = 0;
        int rc;
-       int size = sizeof(*cmd) + ((mc_count - 1) * IEEE80211_ADDR_LEN);
+       int size = sizeof(*cmd) + ((mc_count - 1) * ETH_ALEN);
        cmd = kzalloc(size, GFP_KERNEL);
        if (cmd == NULL)
                return -ENOMEM;
@@ -1702,11 +1663,11 @@ static int mwl8k_cmd_mac_multicast_adr(struct ieee80211_hw *hw,
        cmd->action = cpu_to_le16(MWL8K_ENABLE_RX_MULTICAST);
        cmd->numaddr = cpu_to_le16(mc_count);
        while ((index < mc_count) && mclist) {
-               if (mclist->da_addrlen != IEEE80211_ADDR_LEN) {
+               if (mclist->da_addrlen != ETH_ALEN) {
                        rc = -EINVAL;
                        goto mwl8k_cmd_mac_multicast_adr_exit;
                }
-               memcpy(cmd->addr[index], mclist->da_addr, IEEE80211_ADDR_LEN);
+               memcpy(cmd->addr[index], mclist->da_addr, ETH_ALEN);
                index++;
                mclist = mclist->next;
        }
@@ -1885,11 +1846,11 @@ static int mwl8k_cmd_set_pre_scan(struct ieee80211_hw *hw)
 struct mwl8k_cmd_set_post_scan {
        struct mwl8k_cmd_pkt header;
        __le32 isibss;
-       __u8 bssid[IEEE80211_ADDR_LEN];
+       __u8 bssid[ETH_ALEN];
 } __attribute__((packed));
 
 static int
-mwl8k_cmd_set_post_scan(struct ieee80211_hw *hw, __u8 mac[IEEE80211_ADDR_LEN])
+mwl8k_cmd_set_post_scan(struct ieee80211_hw *hw, __u8 mac[ETH_ALEN])
 {
        struct mwl8k_cmd_set_post_scan *cmd;
        int rc;
@@ -1901,7 +1862,7 @@ mwl8k_cmd_set_post_scan(struct ieee80211_hw *hw, __u8 mac[IEEE80211_ADDR_LEN])
        cmd->header.code = cpu_to_le16(MWL8K_CMD_SET_POST_SCAN);
        cmd->header.length = cpu_to_le16(sizeof(*cmd));
        cmd->isibss = 0;
-       memcpy(cmd->bssid, mac, IEEE80211_ADDR_LEN);
+       memcpy(cmd->bssid, mac, ETH_ALEN);
 
        rc = mwl8k_post_cmd(hw, &cmd->header);
        kfree(cmd);
@@ -2146,7 +2107,6 @@ struct mwl8k_cmd_set_edca_params {
        __u8 txq;
 } __attribute__((packed));
 
-#define MWL8K_GET_EDCA_ALL     0
 #define MWL8K_SET_EDCA_CW      0x01
 #define MWL8K_SET_EDCA_TXOP    0x02
 #define MWL8K_SET_EDCA_AIFS    0x04
@@ -2254,7 +2214,7 @@ struct mwl8k_cmd_update_sta_db {
        __le32  action;
 
        /* Peer MAC address */
-       __u8    peer_addr[IEEE80211_ADDR_LEN];
+       __u8    peer_addr[ETH_ALEN];
 
        __le32  reserved;
 
@@ -2270,7 +2230,6 @@ static int mwl8k_cmd_update_sta_db(struct ieee80211_hw *hw,
        struct mwl8k_cmd_update_sta_db *cmd;
        struct peer_capability_info *peer_info;
        struct ieee80211_rate *bitrates = mv_vif->legacy_rates;
-       DECLARE_MAC_BUF(mac);
        int rc;
        __u8 count, *rates;
 
@@ -2283,7 +2242,7 @@ static int mwl8k_cmd_update_sta_db(struct ieee80211_hw *hw,
 
        cmd->action = cpu_to_le32(action);
        peer_info = &cmd->peer_info;
-       memcpy(cmd->peer_addr, mv_vif->bssid, IEEE80211_ADDR_LEN);
+       memcpy(cmd->peer_addr, mv_vif->bssid, ETH_ALEN);
 
        switch (action) {
        case MWL8K_STA_DB_ADD_ENTRY:
@@ -2320,25 +2279,19 @@ static int mwl8k_cmd_update_sta_db(struct ieee80211_hw *hw,
 /*
  * CMD_SET_AID.
  */
-#define IEEE80211_OPMODE_DISABLED                      0x00
-#define IEEE80211_OPMODE_NON_MEMBER_PROT_MODE          0x01
-#define IEEE80211_OPMODE_ONE_20MHZ_STA_PROT_MODE       0x02
-#define IEEE80211_OPMODE_HTMIXED_PROT_MODE             0x03
-
 #define MWL8K_RATE_INDEX_MAX_ARRAY                     14
 
 #define MWL8K_FRAME_PROT_DISABLED                      0x00
 #define MWL8K_FRAME_PROT_11G                           0x07
 #define MWL8K_FRAME_PROT_11N_HT_40MHZ_ONLY             0x02
 #define MWL8K_FRAME_PROT_11N_HT_ALL                    0x06
-#define MWL8K_FRAME_PROT_MASK                          0x07
 
 struct mwl8k_cmd_update_set_aid {
        struct  mwl8k_cmd_pkt header;
        __le16  aid;
 
         /* AP's MAC address (BSSID) */
-       __u8    bssid[IEEE80211_ADDR_LEN];
+       __u8    bssid[ETH_ALEN];
        __le16  protection_mode;
        __u8    supp_rates[MWL8K_RATE_INDEX_MAX_ARRAY];
 } __attribute__((packed));
@@ -2362,14 +2315,14 @@ static int mwl8k_cmd_set_aid(struct ieee80211_hw *hw,
        cmd->header.length = cpu_to_le16(sizeof(*cmd));
        cmd->aid = cpu_to_le16(info->aid);
 
-       memcpy(cmd->bssid, mv_vif->bssid, IEEE80211_ADDR_LEN);
+       memcpy(cmd->bssid, mv_vif->bssid, ETH_ALEN);
 
        prot_mode = MWL8K_FRAME_PROT_DISABLED;
 
        if (info->use_cts_prot) {
                prot_mode = MWL8K_FRAME_PROT_11G;
        } else {
-               switch (info->ht.operation_mode &
+               switch (info->ht_operation_mode &
                        IEEE80211_HT_OP_MODE_PROTECTION) {
                case IEEE80211_HT_OP_MODE_PROTECTION_20MHZ:
                        prot_mode = MWL8K_FRAME_PROT_11N_HT_40MHZ_ONLY;
@@ -2436,10 +2389,6 @@ static int mwl8k_update_rateset(struct ieee80211_hw *hw,
  */
 #define MWL8K_RATE_TABLE_SIZE  8
 #define MWL8K_UCAST_RATE       0
-#define MWL8K_MCAST_RATE       1
-#define MWL8K_BCAST_RATE       2
-
-#define MWL8K_USE_FIXED_RATE   0x0001
 #define MWL8K_USE_AUTO_RATE    0x0002
 
 struct mwl8k_rate_entry {
@@ -2532,7 +2481,6 @@ static irqreturn_t mwl8k_interrupt(int irq, void *dev_id)
        status = ioread32(priv->regs + MWL8K_HIU_A2H_INTERRUPT_STATUS);
        iowrite32(~status, priv->regs + MWL8K_HIU_A2H_INTERRUPT_STATUS);
 
-       status &= priv->int_mask;
        if (!status)
                return IRQ_NONE;
 
@@ -2870,7 +2818,7 @@ static int mwl8k_start(struct ieee80211_hw *hw)
        }
 
        /* Enable interrupts */
-       iowrite32(priv->int_mask, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
+       iowrite32(MWL8K_A2H_EVENTS, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
 
        worker = kzalloc(sizeof(*worker), GFP_KERNEL);
        if (worker == NULL) {
@@ -2984,7 +2932,7 @@ static int mwl8k_add_interface(struct ieee80211_hw *hw,
        memset(mwl8k_vif, 0, sizeof(*mwl8k_vif));
 
        /* Save the mac address */
-       memcpy(mwl8k_vif->mac_addr, conf->mac_addr, IEEE80211_ADDR_LEN);
+       memcpy(mwl8k_vif->mac_addr, conf->mac_addr, ETH_ALEN);
 
        /* Back pointer to parent config block */
        mwl8k_vif->priv = priv;
@@ -3028,13 +2976,6 @@ static int mwl8k_config_wt(struct work_struct *wt)
        struct mwl8k_priv *priv = hw->priv;
        int rc = 0;
 
-       if (!conf->radio_enabled) {
-               mwl8k_cmd_802_11_radio_control(hw, MWL8K_RADIO_DISABLE);
-               priv->current_channel = NULL;
-               rc = 0;
-               goto mwl8k_config_exit;
-       }
-
        if (mwl8k_cmd_802_11_radio_control(hw, MWL8K_RADIO_ENABLE)) {
                rc = -EINVAL;
                goto mwl8k_config_exit;
@@ -3089,19 +3030,6 @@ static int mwl8k_config(struct ieee80211_hw *hw, u32 changed)
        return rc ? -EINVAL : 0;
 }
 
-static int mwl8k_config_interface(struct ieee80211_hw *hw,
-                                 struct ieee80211_vif *vif,
-                                 struct ieee80211_if_conf *conf)
-{
-       struct mwl8k_vif *mv_vif = MWL8K_VIF(vif);
-       u32 changed = conf->changed;
-
-       if (changed & IEEE80211_IFCC_BSSID)
-               memcpy(mv_vif->bssid, conf->bssid, IEEE80211_ADDR_LEN);
-
-       return 0;
-}
-
 struct mwl8k_bss_info_changed_worker {
        struct mwl8k_work_struct header;
        struct ieee80211_vif *vif;
@@ -3161,14 +3089,13 @@ static int mwl8k_bss_info_changed_wt(struct work_struct *wt)
                 * Finalize the join.  Tell rx handler to process
                 * next beacon from our BSSID.
                 */
-               memcpy(priv->capture_bssid,
-                               mwl8k_vif->bssid, IEEE80211_ADDR_LEN);
+               memcpy(priv->capture_bssid, mwl8k_vif->bssid, ETH_ALEN);
                priv->capture_beacon = true;
        } else {
                mwl8k_cmd_update_sta_db(hw, vif, MWL8K_STA_DB_DEL_ENTRY);
                memset(&mwl8k_vif->bss_info, 0,
                        sizeof(struct ieee80211_bss_conf));
-               memset(mwl8k_vif->bssid, 0, IEEE80211_ADDR_LEN);
+               memset(mwl8k_vif->bssid, 0, ETH_ALEN);
        }
 
 mwl8k_bss_info_changed_exit:
@@ -3183,8 +3110,12 @@ static void mwl8k_bss_info_changed(struct ieee80211_hw *hw,
 {
        struct mwl8k_bss_info_changed_worker *worker;
        struct mwl8k_priv *priv = hw->priv;
+       struct mwl8k_vif *mv_vif = MWL8K_VIF(vif);
        int rc;
 
+       if (changed & BSS_CHANGED_BSSID)
+               memcpy(mv_vif->bssid, info->bssid, ETH_ALEN);
+
        if ((changed & BSS_CHANGED_ASSOC) == 0)
                return;
 
@@ -3225,15 +3156,19 @@ static int mwl8k_configure_filter_wt(struct work_struct *wt)
        struct dev_addr_list *mclist = worker->mclist;
 
        struct mwl8k_priv *priv = hw->priv;
-       struct mwl8k_vif *mv_vif;
        int rc = 0;
 
        if (changed_flags & FIF_BCN_PRBRESP_PROMISC) {
                if (*total_flags & FIF_BCN_PRBRESP_PROMISC)
                        rc = mwl8k_cmd_set_pre_scan(hw);
                else {
-                       mv_vif = MWL8K_VIF(priv->vif);
-                       rc = mwl8k_cmd_set_post_scan(hw, mv_vif->bssid);
+                       u8 *bssid;
+
+                       bssid = "\x00\x00\x00\x00\x00\x00";
+                       if (priv->vif != NULL)
+                               bssid = MWL8K_VIF(priv->vif)->bssid;
+
+                       rc = mwl8k_cmd_set_post_scan(hw, bssid);
                }
        }
 
@@ -3253,31 +3188,50 @@ mwl8k_configure_filter_exit:
        return rc;
 }
 
+static u64 mwl8k_prepare_multicast(struct ieee80211_hw *hw,
+                                  int mc_count, struct dev_addr_list *mclist)
+{
+       struct mwl8k_configure_filter_worker *worker;
+
+       worker = kzalloc(sizeof(*worker), GFP_ATOMIC);
+
+       if (!worker)
+               return 0;
+
+       /*
+        * XXX: This is _HORRIBLY_ broken!!
+        *
+        *      No locking, the mclist pointer might be invalid as soon as this
+        *      function returns, something in the list might be invalidated
+        *      once we get to the worker, etc...
+        */
+       worker->mc_count = mc_count;
+       worker->mclist = mclist;
+
+       return (u64)worker;
+}
+
 static void mwl8k_configure_filter(struct ieee80211_hw *hw,
                                   unsigned int changed_flags,
                                   unsigned int *total_flags,
-                                  int mc_count,
-                                  struct dev_addr_list *mclist)
+                                  u64 multicast)
 {
 
-       struct mwl8k_configure_filter_worker *worker;
+       struct mwl8k_configure_filter_worker *worker = (void *)multicast;
        struct mwl8k_priv *priv = hw->priv;
 
        /* Clear unsupported feature flags */
        *total_flags &= MWL8K_SUPPORTED_IF_FLAGS;
 
-       if (!(changed_flags & MWL8K_SUPPORTED_IF_FLAGS) && !mc_count)
+       if (!(changed_flags & MWL8K_SUPPORTED_IF_FLAGS))
                return;
 
-       worker = kzalloc(sizeof(*worker), GFP_ATOMIC);
        if (worker == NULL)
                return;
 
        worker->header.options = MWL8K_WQ_QUEUE_ONLY | MWL8K_WQ_TX_WAIT_EMPTY;
        worker->changed_flags = changed_flags;
        worker->total_flags = total_flags;
-       worker->mc_count = mc_count;
-       worker->mclist = mclist;
 
        mwl8k_queue_work(hw, &worker->header, priv->config_wq,
                         mwl8k_configure_filter_wt);
@@ -3442,8 +3396,8 @@ static const struct ieee80211_ops mwl8k_ops = {
        .add_interface          = mwl8k_add_interface,
        .remove_interface       = mwl8k_remove_interface,
        .config                 = mwl8k_config,
-       .config_interface       = mwl8k_config_interface,
        .bss_info_changed       = mwl8k_bss_info_changed,
+       .prepare_multicast      = mwl8k_prepare_multicast,
        .configure_filter       = mwl8k_configure_filter,
        .set_rts_threshold      = mwl8k_set_rts_threshold,
        .conf_tx                = mwl8k_conf_tx,
@@ -3489,7 +3443,6 @@ static int __devinit mwl8k_probe(struct pci_dev *pdev,
 {
        struct ieee80211_hw *hw;
        struct mwl8k_priv *priv;
-       DECLARE_MAC_BUF(mac);
        int rc;
        int i;
        u8 *fw;
@@ -3523,7 +3476,6 @@ static int __devinit mwl8k_probe(struct pci_dev *pdev,
        priv->hostcmd_wait = NULL;
        priv->tx_wait = NULL;
        priv->inconfig = false;
-       priv->wep_enabled = 0;
        priv->wmm_mode = false;
        priv->pending_tx_pkts = 0;
        strncpy(priv->name, MWL8K_NAME, sizeof(priv->name));
@@ -3605,8 +3557,7 @@ static int __devinit mwl8k_probe(struct pci_dev *pdev,
        }
 
        iowrite32(0, priv->regs + MWL8K_HIU_A2H_INTERRUPT_STATUS);
-       priv->int_mask = 0;
-       iowrite32(priv->int_mask, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
+       iowrite32(0, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
        iowrite32(0, priv->regs + MWL8K_HIU_A2H_INTERRUPT_CLEAR_SEL);
        iowrite32(0xffffffff, priv->regs + MWL8K_HIU_A2H_INTERRUPT_STATUS_MASK);
 
@@ -3643,9 +3594,7 @@ static int __devinit mwl8k_probe(struct pci_dev *pdev,
         * commands use interrupts and avoids polling.  Disable
         * interrupts when done.
         */
-       priv->int_mask |= MWL8K_A2H_EVENTS;
-
-       iowrite32(priv->int_mask, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
+       iowrite32(MWL8K_A2H_EVENTS, priv->regs + MWL8K_HIU_A2H_INTERRUPT_MASK);
 
        /* Get config data, mac addrs etc */
        rc = mwl8k_cmd_get_hw_spec(hw);
@@ -3678,8 +3627,8 @@ static int __devinit mwl8k_probe(struct pci_dev *pdev,
                MWL8K_DESC);
        printk(KERN_INFO "%s: Driver Ver:%s  Firmware Ver:%u.%u.%u.%u\n",
                priv->name, MWL8K_VERSION, fw[3], fw[2], fw[1], fw[0]);
-       printk(KERN_INFO "%s: MAC Address: %s\n", priv->name,
-              print_mac(mac, hw->wiphy->perm_addr));
+       printk(KERN_INFO "%s: MAC Address: %pM\n", priv->name,
+               hw->wiphy->perm_addr);
 
        return 0;
 
@@ -3736,6 +3685,8 @@ static void __devexit mwl8k_remove(struct pci_dev *pdev)
 
        ieee80211_stop_queues(hw);
 
+       ieee80211_unregister_hw(hw);
+
        /* Remove tx reclaim tasklet */
        tasklet_kill(&priv->tx_reclaim_task);
 
@@ -3749,8 +3700,6 @@ static void __devexit mwl8k_remove(struct pci_dev *pdev)
        for (i = 0; i < MWL8K_TX_QUEUES; i++)
                mwl8k_txq_reclaim(hw, i, 1);
 
-       ieee80211_unregister_hw(hw);
-
        for (i = 0; i < MWL8K_TX_QUEUES; i++)
                mwl8k_txq_deinit(hw, i);