wlcore/wl18xx/wl12xx: convert bitmaps to unsigned longs
authorEliad Peller <eliad@wizery.com>
Fri, 11 Jul 2014 00:01:39 +0000 (03:01 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 15 Jul 2014 20:00:02 +0000 (16:00 -0400)
The *_bit operations expect unsigned longs.
Instead of casting the pointers, simply define various
bitmaps as unsigned long (instead of u32).

Signed-off-by: Eliad Peller <eliad@wizery.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ti/wl12xx/main.c
drivers/net/wireless/ti/wl18xx/main.c
drivers/net/wireless/ti/wlcore/debugfs.c
drivers/net/wireless/ti/wlcore/main.c
drivers/net/wireless/ti/wlcore/tx.c
drivers/net/wireless/ti/wlcore/wlcore.h

index d50dfac91631ebcbd4f685497ca237d3eb22893f..0bccf123831ec0ab80d8a8c62e99b01d7cd23c12 100644 (file)
@@ -1668,7 +1668,7 @@ static bool wl12xx_lnk_high_prio(struct wl1271 *wl, u8 hlid,
 {
        u8 thold;
 
-       if (test_bit(hlid, (unsigned long *)&wl->fw_fast_lnk_map))
+       if (test_bit(hlid, &wl->fw_fast_lnk_map))
                thold = wl->conf.tx.fast_link_thold;
        else
                thold = wl->conf.tx.slow_link_thold;
index edc3e4d8966dadd6e61ffbc03e39388d92ab880e..7af1936719eb879d0915fd9dda2db28ded54f39b 100644 (file)
@@ -1609,7 +1609,7 @@ static bool wl18xx_lnk_high_prio(struct wl1271 *wl, u8 hlid,
        u8 thold;
        struct wl18xx_fw_status_priv *status_priv =
                (struct wl18xx_fw_status_priv *)wl->fw_status->priv;
-       u32 suspend_bitmap;
+       unsigned long suspend_bitmap;
 
        /* if we don't have the link map yet, assume they all low prio */
        if (!status_priv)
@@ -1617,12 +1617,12 @@ static bool wl18xx_lnk_high_prio(struct wl1271 *wl, u8 hlid,
 
        /* suspended links are never high priority */
        suspend_bitmap = le32_to_cpu(status_priv->link_suspend_bitmap);
-       if (test_bit(hlid, (unsigned long *)&suspend_bitmap))
+       if (test_bit(hlid, &suspend_bitmap))
                return false;
 
        /* the priority thresholds are taken from FW */
-       if (test_bit(hlid, (unsigned long *)&wl->fw_fast_lnk_map) &&
-           !test_bit(hlid, (unsigned long *)&wl->ap_fw_ps_map))
+       if (test_bit(hlid, &wl->fw_fast_lnk_map) &&
+           !test_bit(hlid, &wl->ap_fw_ps_map))
                thold = status_priv->tx_fast_link_prio_threshold;
        else
                thold = status_priv->tx_slow_link_prio_threshold;
@@ -1636,17 +1636,17 @@ static bool wl18xx_lnk_low_prio(struct wl1271 *wl, u8 hlid,
        u8 thold;
        struct wl18xx_fw_status_priv *status_priv =
                (struct wl18xx_fw_status_priv *)wl->fw_status->priv;
-       u32 suspend_bitmap;
+       unsigned long suspend_bitmap;
 
        /* if we don't have the link map yet, assume they all low prio */
        if (!status_priv)
                return true;
 
        suspend_bitmap = le32_to_cpu(status_priv->link_suspend_bitmap);
-       if (test_bit(hlid, (unsigned long *)&suspend_bitmap))
+       if (test_bit(hlid, &suspend_bitmap))
                thold = status_priv->tx_suspend_threshold;
-       else if (test_bit(hlid, (unsigned long *)&wl->fw_fast_lnk_map) &&
-                !test_bit(hlid, (unsigned long *)&wl->ap_fw_ps_map))
+       else if (test_bit(hlid, &wl->fw_fast_lnk_map) &&
+                !test_bit(hlid, &wl->ap_fw_ps_map))
                thold = status_priv->tx_fast_stop_threshold;
        else
                thold = status_priv->tx_slow_stop_threshold;
index 89893c7170253c8cc5400bf7941cc9ce4a3d3051..0be21f62fcb0eb3a58e7c16886efa8c2bde096cc 100644 (file)
@@ -496,7 +496,7 @@ static ssize_t driver_state_read(struct file *file, char __user *user_buf,
        DRIVER_STATE_PRINT_INT(sg_enabled);
        DRIVER_STATE_PRINT_INT(enable_11a);
        DRIVER_STATE_PRINT_INT(noise);
-       DRIVER_STATE_PRINT_HEX(ap_fw_ps_map);
+       DRIVER_STATE_PRINT_LHEX(ap_fw_ps_map);
        DRIVER_STATE_PRINT_LHEX(ap_ps_map);
        DRIVER_STATE_PRINT_HEX(quirks);
        DRIVER_STATE_PRINT_HEX(irq);
index ec211413ffe6b3ff4e34bbd6d572cd622f5a735d..575c8f6d4009477dc6430b7eff91ab6d8a60e080 100644 (file)
@@ -333,7 +333,7 @@ static void wl12xx_irq_ps_regulate_link(struct wl1271 *wl,
 {
        bool fw_ps;
 
-       fw_ps = test_bit(hlid, (unsigned long *)&wl->ap_fw_ps_map);
+       fw_ps = test_bit(hlid, &wl->ap_fw_ps_map);
 
        /*
         * Wake up from high level PS if the STA is asleep with too little
@@ -360,13 +360,13 @@ static void wl12xx_irq_update_links_status(struct wl1271 *wl,
                                           struct wl12xx_vif *wlvif,
                                           struct wl_fw_status *status)
 {
-       u32 cur_fw_ps_map;
+       unsigned long cur_fw_ps_map;
        u8 hlid;
 
        cur_fw_ps_map = status->link_ps_bitmap;
        if (wl->ap_fw_ps_map != cur_fw_ps_map) {
                wl1271_debug(DEBUG_PSM,
-                            "link ps prev 0x%x cur 0x%x changed 0x%x",
+                            "link ps prev 0x%lx cur 0x%lx changed 0x%lx",
                             wl->ap_fw_ps_map, cur_fw_ps_map,
                             wl->ap_fw_ps_map ^ cur_fw_ps_map);
 
@@ -4754,7 +4754,7 @@ void wl1271_free_sta(struct wl1271 *wl, struct wl12xx_vif *wlvif, u8 hlid)
 
        clear_bit(hlid, wlvif->ap.sta_hlid_map);
        __clear_bit(hlid, &wl->ap_ps_map);
-       __clear_bit(hlid, (unsigned long *)&wl->ap_fw_ps_map);
+       __clear_bit(hlid, &wl->ap_fw_ps_map);
 
        /*
         * save the last used PN in the private part of iee80211_sta,
index 40b43115f83590b6a6cb4a38d8283e7b78460187..f0ac36139bcc1419a3963a5048e8c3013fce5cd4 100644 (file)
@@ -126,7 +126,7 @@ static void wl1271_tx_regulate_link(struct wl1271 *wl,
        if (WARN_ON(!test_bit(hlid, wlvif->links_map)))
                return;
 
-       fw_ps = test_bit(hlid, (unsigned long *)&wl->ap_fw_ps_map);
+       fw_ps = test_bit(hlid, &wl->ap_fw_ps_map);
        tx_pkts = wl->links[hlid].allocated_pkts;
 
        /*
index 13459c4f74d00836b1da7ac47c39abf5571a491a..df78cf12ef1574b475751aca21ab5995c397af95 100644 (file)
@@ -388,10 +388,10 @@ struct wl1271 {
        int active_link_count;
 
        /* Fast/slow links bitmap according to FW */
-       u32 fw_fast_lnk_map;
+       unsigned long fw_fast_lnk_map;
 
        /* AP-mode - a bitmap of links currently in PS mode according to FW */
-       u32 ap_fw_ps_map;
+       unsigned long ap_fw_ps_map;
 
        /* AP-mode - a bitmap of links currently in PS mode in mac80211 */
        unsigned long ap_ps_map;