mwifiex: cleanup work in scan.c
authorAmitkumar Karwar <akarwar@marvell.com>
Fri, 16 Dec 2011 05:00:37 +0000 (21:00 -0800)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 19 Dec 2011 19:37:14 +0000 (14:37 -0500)
Scan type derived from IEEE80211_CHAN_PASSIVE_SCAN bit is a boolean
flag representing passive scanning. We should not again compare it
with driver specific macro MWIFIEX_SCAN_TYPE_PASSIVE to determine
passive or active scan. We can also avoid the use of local variable
by using the flag directly.

Signed-off-by: Amitkumar Karwar <akarwar@marvell.com>
Signed-off-by: Bing Zhao <bzhao@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/mwifiex/scan.c

index e2e715666bcabf36815f088a914a8ee20fd7c9b8..6396d3318ead1e3b3afb26858d57873a228022b4 100644 (file)
@@ -500,7 +500,6 @@ mwifiex_scan_create_channel_list(struct mwifiex_private *priv,
        struct ieee80211_channel *ch;
        struct mwifiex_adapter *adapter = priv->adapter;
        int chan_idx = 0, i;
-       u8 scan_type;
 
        for (band = 0; (band < IEEE80211_NUM_BANDS) ; band++) {
 
@@ -514,19 +513,20 @@ mwifiex_scan_create_channel_list(struct mwifiex_private *priv,
                        if (ch->flags & IEEE80211_CHAN_DISABLED)
                                continue;
                        scan_chan_list[chan_idx].radio_type = band;
-                       scan_type = ch->flags & IEEE80211_CHAN_PASSIVE_SCAN;
+
                        if (user_scan_in &&
                                user_scan_in->chan_list[0].scan_time)
                                scan_chan_list[chan_idx].max_scan_time =
                                        cpu_to_le16((u16) user_scan_in->
                                        chan_list[0].scan_time);
-                       else if (scan_type == MWIFIEX_SCAN_TYPE_PASSIVE)
+                       else if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
                                scan_chan_list[chan_idx].max_scan_time =
                                        cpu_to_le16(adapter->passive_scan_time);
                        else
                                scan_chan_list[chan_idx].max_scan_time =
                                        cpu_to_le16(adapter->active_scan_time);
-                       if (scan_type == MWIFIEX_SCAN_TYPE_PASSIVE)
+
+                       if (ch->flags & IEEE80211_CHAN_PASSIVE_SCAN)
                                scan_chan_list[chan_idx].chan_scan_mode_bitmap
                                        |= MWIFIEX_PASSIVE_SCAN;
                        else