]> git.karo-electronics.de Git - mv-sheeva.git/blobdiff - drivers/staging/brcm80211/brcmfmac/wl_iw.c
Merge tag 'v2.6.38' of git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6
[mv-sheeva.git] / drivers / staging / brcm80211 / brcmfmac / wl_iw.c
index 979a494fda59eaff5b23c5464bff79cd07096b07..db6e68eab290ce8fdf24b7066cf9780f06779d92 100644 (file)
@@ -15,8 +15,9 @@
  */
 
 #include <linux/kthread.h>
+#include <linux/semaphore.h>
 #include <bcmdefs.h>
-#include <linuxver.h>
+#include <linux/netdevice.h>
 #include <osl.h>
 #include <wlioctl.h>
 
 #include <dhd.h>
 #include <dhdioctl.h>
 
-typedef void wlc_info_t;
-typedef void wl_info_t;
 typedef const struct si_pub si_t;
 #include <wlioctl.h>
 
 #include <proto/ethernet.h>
 #include <dngl_stats.h>
 #include <dhd.h>
-#define WL_ERROR(x) printf x
-#define WL_TRACE(x)
-#define WL_ASSOC(x)
-#define WL_INFORM(x)
-#define WL_WSEC(x)
-#define WL_SCAN(x)
+
+#define WL_ERROR(fmt, args...) printk(fmt, ##args)
+#define WL_TRACE(fmt, args...) no_printk(fmt, ##args)
+#define WL_INFORM(fmt, args...)        no_printk(fmt, ##args)
+#define WL_WSEC(fmt, args...)  no_printk(fmt, ##args)
+#define WL_SCAN(fmt, args...)  no_printk(fmt, ##args)
 
 #include <wl_iw.h>
 
@@ -187,12 +186,12 @@ static int dev_wlc_ioctl(struct net_device *dev, int cmd, void *arg, int len)
        int ret = -EINVAL;
 
        if (!dev) {
-               WL_ERROR(("%s: dev is null\n", __func__));
+               WL_ERROR("%s: dev is null\n", __func__);
                return ret;
        }
 
-       WL_INFORM(("\n%s, PID:%x: send Local IOCTL -> dhd: cmd:0x%x, buf:%p, "
-               "len:%d ,\n", __func__, current->pid, cmd, arg, len));
+       WL_INFORM("\n%s, PID:%x: send Local IOCTL -> dhd: cmd:0x%x, buf:%p, len:%d\n",
+                 __func__, current->pid, cmd, arg, len);
 
        if (g_onoff == G_WLAN_SET_ON) {
                memset(&ioc, 0, sizeof(ioc));
@@ -205,7 +204,7 @@ static int dev_wlc_ioctl(struct net_device *dev, int cmd, void *arg, int len)
 
                ret = dev_open(dev);
                if (ret) {
-                       WL_ERROR(("%s: Error dev_open: %d\n", __func__, ret));
+                       WL_ERROR("%s: Error dev_open: %d\n", __func__, ret);
                        return ret;
                }
 
@@ -214,7 +213,7 @@ static int dev_wlc_ioctl(struct net_device *dev, int cmd, void *arg, int len)
                ret = dev->netdev_ops->ndo_do_ioctl(dev, &ifr, SIOCDEVPRIVATE);
                set_fs(fs);
        } else {
-               WL_TRACE(("%s: call after driver stop : ignored\n", __func__));
+               WL_TRACE("%s: call after driver stop : ignored\n", __func__);
        }
        return ret;
 }
@@ -335,7 +334,7 @@ wl_iw_config_commit(struct net_device *dev,
        int error;
        struct sockaddr bssid;
 
-       WL_TRACE(("%s: SIOCSIWCOMMIT\n", dev->name));
+       WL_TRACE("%s: SIOCSIWCOMMIT\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_SSID, &ssid, sizeof(ssid));
        if (error)
@@ -346,11 +345,11 @@ wl_iw_config_commit(struct net_device *dev,
        if (!ssid.SSID_len)
                return 0;
 
-       bzero(&bssid, sizeof(struct sockaddr));
-       error = dev_wlc_ioctl(dev, WLC_REASSOC, &bssid, ETHER_ADDR_LEN);
+       memset(&bssid, 0, sizeof(struct sockaddr));
+       error = dev_wlc_ioctl(dev, WLC_REASSOC, &bssid, ETH_ALEN);
        if (error) {
-               WL_ERROR(("%s: WLC_REASSOC to %s failed \n", __func__,
-                         ssid.SSID));
+               WL_ERROR("%s: WLC_REASSOC to %s failed\n",
+                        __func__, ssid.SSID);
                return error;
        }
 
@@ -361,7 +360,7 @@ static int
 wl_iw_get_name(struct net_device *dev,
               struct iw_request_info *info, char *cwrq, char *extra)
 {
-       WL_TRACE(("%s: SIOCGIWNAME\n", dev->name));
+       WL_TRACE("%s: SIOCGIWNAME\n", dev->name);
 
        strcpy(cwrq, "IEEE 802.11-DS");
 
@@ -375,7 +374,7 @@ wl_iw_set_freq(struct net_device *dev,
        int error, chan;
        uint sf = 0;
 
-       WL_TRACE(("\n %s %s: SIOCSIWFREQ\n", __func__, dev->name));
+       WL_TRACE("\n %s %s: SIOCSIWFREQ\n", __func__, dev->name);
 
        if (fwrq->e == 0 && fwrq->m < MAXCHANNEL) {
                chan = fwrq->m;
@@ -410,7 +409,7 @@ wl_iw_get_freq(struct net_device *dev,
        channel_info_t ci;
        int error;
 
-       WL_TRACE(("%s: SIOCGIWFREQ\n", dev->name));
+       WL_TRACE("%s: SIOCGIWFREQ\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_CHANNEL, &ci, sizeof(ci));
        if (error)
@@ -427,7 +426,7 @@ wl_iw_set_mode(struct net_device *dev,
 {
        int infra = 0, ap = 0, error = 0;
 
-       WL_TRACE(("%s: SIOCSIWMODE\n", dev->name));
+       WL_TRACE("%s: SIOCSIWMODE\n", dev->name);
 
        switch (*uwrq) {
        case IW_MODE_MASTER:
@@ -462,7 +461,7 @@ wl_iw_get_mode(struct net_device *dev,
 {
        int error, infra = 0, ap = 0;
 
-       WL_TRACE(("%s: SIOCGIWMODE\n", dev->name));
+       WL_TRACE("%s: SIOCGIWMODE\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_INFRA, &infra, sizeof(infra));
        if (error)
@@ -501,14 +500,14 @@ wl_iw_get_range(struct net_device *dev,
        {30, 60, 90, 120, 180, 240, 270, 300}
        };
 
-       WL_TRACE(("%s: SIOCGIWRANGE\n", dev->name));
+       WL_TRACE("%s: SIOCGIWRANGE\n", dev->name);
 
        if (!extra)
                return -EINVAL;
 
        channels = kmalloc((MAXCHANNEL + 1) * 4, GFP_KERNEL);
        if (!channels) {
-               WL_ERROR(("Could not alloc channels\n"));
+               WL_ERROR("Could not alloc channels\n");
                return -ENOMEM;
        }
        list = (wl_u32_list_t *) channels;
@@ -684,14 +683,14 @@ wl_iw_set_spy(struct net_device *dev,
        struct sockaddr *addr = (struct sockaddr *)extra;
        int i;
 
-       WL_TRACE(("%s: SIOCSIWSPY\n", dev->name));
+       WL_TRACE("%s: SIOCSIWSPY\n", dev->name);
 
        if (!extra)
                return -EINVAL;
 
        iw->spy_num = min_t(int, ARRAY_SIZE(iw->spy_addr), dwrq->length);
        for (i = 0; i < iw->spy_num; i++)
-               memcpy(&iw->spy_addr[i], addr[i].sa_data, ETHER_ADDR_LEN);
+               memcpy(&iw->spy_addr[i], addr[i].sa_data, ETH_ALEN);
        memset(iw->spy_qual, 0, sizeof(iw->spy_qual));
 
        return 0;
@@ -706,14 +705,14 @@ wl_iw_get_spy(struct net_device *dev,
        struct iw_quality *qual = (struct iw_quality *)&addr[iw->spy_num];
        int i;
 
-       WL_TRACE(("%s: SIOCGIWSPY\n", dev->name));
+       WL_TRACE("%s: SIOCGIWSPY\n", dev->name);
 
        if (!extra)
                return -EINVAL;
 
        dwrq->length = iw->spy_num;
        for (i = 0; i < iw->spy_num; i++) {
-               memcpy(addr[i].sa_data, &iw->spy_addr[i], ETHER_ADDR_LEN);
+               memcpy(addr[i].sa_data, &iw->spy_addr[i], ETH_ALEN);
                addr[i].sa_family = AF_UNIX;
                memcpy(&qual[i], &iw->spy_qual[i], sizeof(struct iw_quality));
                iw->spy_qual[i].updated = 0;
@@ -751,8 +750,8 @@ wl_iw_ch_to_chanspec(int ch, wl_join_params_t *join_params,
                join_params->params.chanspec_num =
                    htod32(join_params->params.chanspec_num);
 
-               WL_TRACE(("%s  join_params->params.chanspec_list[0]= %X\n",
-                         __func__, join_params->params.chanspec_list[0]));
+               WL_TRACE("%s  join_params->params.chanspec_list[0]= %X\n",
+                        __func__, join_params->params.chanspec_list[0]);
        }
        return 1;
 }
@@ -765,16 +764,17 @@ wl_iw_set_wap(struct net_device *dev,
        wl_join_params_t join_params;
        int join_params_size;
 
-       WL_TRACE(("%s: SIOCSIWAP\n", dev->name));
+       WL_TRACE("%s: SIOCSIWAP\n", dev->name);
 
        if (awrq->sa_family != ARPHRD_ETHER) {
-               WL_ERROR(("Invalid Header...sa_family\n"));
+               WL_ERROR("Invalid Header...sa_family\n");
                return -EINVAL;
        }
 
-       if (ETHER_ISBCAST(awrq->sa_data) || ETHER_ISNULLADDR(awrq->sa_data)) {
+       if (is_broadcast_ether_addr(awrq->sa_data) ||
+           is_zero_ether_addr(awrq->sa_data)) {
                scb_val_t scbval;
-               bzero(&scbval, sizeof(scb_val_t));
+               memset(&scbval, 0, sizeof(scb_val_t));
                (void)dev_wlc_ioctl(dev, WLC_DISASSOC, &scbval,
                                    sizeof(scb_val_t));
                return 0;
@@ -785,23 +785,23 @@ wl_iw_set_wap(struct net_device *dev,
 
        memcpy(join_params.ssid.SSID, g_ssid.SSID, g_ssid.SSID_len);
        join_params.ssid.SSID_len = htod32(g_ssid.SSID_len);
-       memcpy(&join_params.params.bssid, awrq->sa_data, ETHER_ADDR_LEN);
+       memcpy(&join_params.params.bssid, awrq->sa_data, ETH_ALEN);
 
-       WL_TRACE(("%s  target_channel=%d\n", __func__,
-                 g_wl_iw_params.target_channel));
+       WL_TRACE("%s  target_channel=%d\n",
+                __func__, g_wl_iw_params.target_channel);
        wl_iw_ch_to_chanspec(g_wl_iw_params.target_channel, &join_params,
                             &join_params_size);
 
        error = dev_wlc_ioctl(dev, WLC_SET_SSID, &join_params,
                                join_params_size);
        if (error) {
-               WL_ERROR(("%s Invalid ioctl data=%d\n", __func__, error));
+               WL_ERROR("%s Invalid ioctl data=%d\n", __func__, error);
        }
 
        if (g_ssid.SSID_len) {
-               WL_TRACE(("%s: join SSID=%s BSSID=%pM ch=%d\n",
-                         __func__, g_ssid.SSID, awrq->sa_data,
-                         g_wl_iw_params.target_channel));
+               WL_TRACE("%s: join SSID=%s BSSID=%pM ch=%d\n",
+                        __func__, g_ssid.SSID, awrq->sa_data,
+                        g_wl_iw_params.target_channel);
        }
 
        memset(&g_ssid, 0, sizeof(g_ssid));
@@ -812,12 +812,12 @@ static int
 wl_iw_get_wap(struct net_device *dev,
              struct iw_request_info *info, struct sockaddr *awrq, char *extra)
 {
-       WL_TRACE(("%s: SIOCGIWAP\n", dev->name));
+       WL_TRACE("%s: SIOCGIWAP\n", dev->name);
 
        awrq->sa_family = ARPHRD_ETHER;
-       memset(awrq->sa_data, 0, ETHER_ADDR_LEN);
+       memset(awrq->sa_data, 0, ETH_ALEN);
 
-       (void)dev_wlc_ioctl(dev, WLC_GET_BSSID, awrq->sa_data, ETHER_ADDR_LEN);
+       (void)dev_wlc_ioctl(dev, WLC_GET_BSSID, awrq->sa_data, ETH_ALEN);
 
        return 0;
 }
@@ -831,16 +831,16 @@ wl_iw_mlme(struct net_device *dev,
        scb_val_t scbval;
        int error = -EINVAL;
 
-       WL_TRACE(("%s: SIOCSIWMLME DISASSOC/DEAUTH\n", dev->name));
+       WL_TRACE("%s: SIOCSIWMLME DISASSOC/DEAUTH\n", dev->name);
 
        mlme = (struct iw_mlme *)extra;
        if (mlme == NULL) {
-               WL_ERROR(("Invalid ioctl data.\n"));
+               WL_ERROR("Invalid ioctl data\n");
                return error;
        }
 
        scbval.val = mlme->reason_code;
-       bcopy(&mlme->addr.sa_data, &scbval.ea, ETHER_ADDR_LEN);
+       bcopy(&mlme->addr.sa_data, &scbval.ea, ETH_ALEN);
 
        if (mlme->cmd == IW_MLME_DISASSOC) {
                scbval.val = htod32(scbval.val);
@@ -853,7 +853,7 @@ wl_iw_mlme(struct net_device *dev,
                    dev_wlc_ioctl(dev, WLC_SCB_DEAUTHENTICATE_FOR_REASON,
                                  &scbval, sizeof(scb_val_t));
        } else {
-               WL_ERROR(("Invalid ioctl data.\n"));
+               WL_ERROR("Invalid ioctl data\n");
                return error;
        }
 
@@ -874,7 +874,7 @@ wl_iw_get_aplist(struct net_device *dev,
        int error, i;
        uint buflen = dwrq->length;
 
-       WL_TRACE(("%s: SIOCGIWAPLIST\n", dev->name));
+       WL_TRACE("%s: SIOCGIWAPLIST\n", dev->name);
 
        if (!extra)
                return -EINVAL;
@@ -886,7 +886,7 @@ wl_iw_get_aplist(struct net_device *dev,
        list->buflen = htod32(buflen);
        error = dev_wlc_ioctl(dev, WLC_SCAN_RESULTS, list, buflen);
        if (error) {
-               WL_ERROR(("%d: Scan results error %d\n", __LINE__, error));
+               WL_ERROR("%d: Scan results error %d\n", __LINE__, error);
                kfree(list);
                return error;
        }
@@ -894,8 +894,8 @@ wl_iw_get_aplist(struct net_device *dev,
        list->version = dtoh32(list->version);
        list->count = dtoh32(list->count);
        if (list->version != WL_BSS_INFO_VERSION) {
-               WL_ERROR(("%s : list->version %d != WL_BSS_INFO_VERSION\n",
-                         __func__, list->version));
+               WL_ERROR("%s : list->version %d != WL_BSS_INFO_VERSION\n",
+                        __func__, list->version);
                kfree(list);
                return -EINVAL;
        }
@@ -911,7 +911,7 @@ wl_iw_get_aplist(struct net_device *dev,
                if (!(dtoh16(bi->capability) & DOT11_CAP_ESS))
                        continue;
 
-               memcpy(addr[dwrq->length].sa_data, &bi->BSSID, ETHER_ADDR_LEN);
+               memcpy(addr[dwrq->length].sa_data, &bi->BSSID, ETH_ALEN);
                addr[dwrq->length].sa_family = ARPHRD_ETHER;
                qual[dwrq->length].qual = rssi_to_qual(dtoh16(bi->RSSI));
                qual[dwrq->length].level = 0x100 + dtoh16(bi->RSSI);
@@ -952,13 +952,13 @@ wl_iw_iscan_get_aplist(struct net_device *dev,
        wl_bss_info_t *bi = NULL;
        int i;
 
-       WL_TRACE(("%s: SIOCGIWAPLIST\n", dev->name));
+       WL_TRACE("%s: SIOCGIWAPLIST\n", dev->name);
 
        if (!extra)
                return -EINVAL;
 
        if ((!iscan) || (!iscan->sysioc_tsk)) {
-               WL_ERROR(("%s error\n", __func__));
+               WL_ERROR("%s error\n", __func__);
                return 0;
        }
 
@@ -966,9 +966,8 @@ wl_iw_iscan_get_aplist(struct net_device *dev,
        while (buf) {
                list = &((wl_iscan_results_t *) buf->iscan_buf)->results;
                if (list->version != WL_BSS_INFO_VERSION) {
-                       WL_ERROR(("%s : list->version %d != "
-                               "WL_BSS_INFO_VERSION\n",
-                               __func__, list->version));
+                       WL_ERROR("%s : list->version %d != WL_BSS_INFO_VERSION\n",
+                                __func__, list->version);
                        return -EINVAL;
                }
 
@@ -985,7 +984,7 @@ wl_iw_iscan_get_aplist(struct net_device *dev,
                                continue;
 
                        memcpy(addr[dwrq->length].sa_data, &bi->BSSID,
-                              ETHER_ADDR_LEN);
+                              ETH_ALEN);
                        addr[dwrq->length].sa_family = ARPHRD_ETHER;
                        qual[dwrq->length].qual =
                            rssi_to_qual(dtoh16(bi->RSSI));
@@ -1016,7 +1015,7 @@ static int wl_iw_iscan_prep(wl_scan_params_t *params, wlc_ssid_t *ssid)
 {
        int err = 0;
 
-       memcpy(&params->bssid, &ether_bcast, ETHER_ADDR_LEN);
+       memcpy(&params->bssid, &ether_bcast, ETH_ALEN);
        params->bss_type = DOT11_BSSTYPE_ANY;
        params->scan_type = 0;
        params->nprobes = -1;
@@ -1043,15 +1042,15 @@ static int wl_iw_iscan(iscan_info_t *iscan, wlc_ssid_t *ssid, u16 action)
        iscan->iscan_ex_params_p->action = htod16(action);
        iscan->iscan_ex_params_p->scan_duration = htod16(0);
 
-       WL_SCAN(("%s : nprobes=%d\n", __func__,
-                iscan->iscan_ex_params_p->params.nprobes));
-       WL_SCAN(("active_time=%d\n",
-                iscan->iscan_ex_params_p->params.active_time));
-       WL_SCAN(("passive_time=%d\n",
-                iscan->iscan_ex_params_p->params.passive_time));
-       WL_SCAN(("home_time=%d\n", iscan->iscan_ex_params_p->params.home_time));
-       WL_SCAN(("scan_type=%d\n", iscan->iscan_ex_params_p->params.scan_type));
-       WL_SCAN(("bss_type=%d\n", iscan->iscan_ex_params_p->params.bss_type));
+       WL_SCAN("%s : nprobes=%d\n",
+               __func__, iscan->iscan_ex_params_p->params.nprobes);
+       WL_SCAN("active_time=%d\n",
+                iscan->iscan_ex_params_p->params.active_time);
+       WL_SCAN("passive_time=%d\n",
+                iscan->iscan_ex_params_p->params.passive_time);
+       WL_SCAN("home_time=%d\n", iscan->iscan_ex_params_p->params.home_time);
+       WL_SCAN("scan_type=%d\n", iscan->iscan_ex_params_p->params.scan_type);
+       WL_SCAN("bss_type=%d\n", iscan->iscan_ex_params_p->params.bss_type);
 
        (void)dev_iw_iovar_setbuf(iscan->dev, "iscan", iscan->iscan_ex_params_p,
                                  iscan->iscan_ex_param_size, iscan->ioctlbuf,
@@ -1066,7 +1065,7 @@ static void wl_iw_timerfunc(unsigned long data)
        if (iscan) {
                iscan->timer_on = 0;
                if (iscan->iscan_state != ISCAN_STATE_IDLE) {
-                       WL_TRACE(("timer trigger\n"));
+                       WL_TRACE("timer trigger\n");
                        up(&iscan->sysioc_sem);
                }
        }
@@ -1101,8 +1100,8 @@ static u32 wl_iw_iscan_get(iscan_info_t *iscan)
        } else {
                buf = kmalloc(sizeof(iscan_buf_t), GFP_KERNEL);
                if (!buf) {
-                       WL_ERROR(("%s can't alloc iscan_buf_t : going to abort "
-                               "currect iscan\n", __func__));
+                       WL_ERROR("%s can't alloc iscan_buf_t : going to abort current iscan\n",
+                                __func__);
                        MUTEX_UNLOCK_WL_SCAN_SET();
                        return WL_SCAN_RESULTS_NO_MEM;
                }
@@ -1135,11 +1134,11 @@ static u32 wl_iw_iscan_get(iscan_info_t *iscan)
                results->buflen = dtoh32(results->buflen);
                results->version = dtoh32(results->version);
                results->count = dtoh32(results->count);
-               WL_TRACE(("results->count = %d\n", results->count));
-               WL_TRACE(("results->buflen = %d\n", results->buflen));
+               WL_TRACE("results->count = %d\n", results->count);
+               WL_TRACE("results->buflen = %d\n", results->buflen);
                status = dtoh32(list_buf->status);
        } else {
-               WL_ERROR(("%s returns error %d\n", __func__, res));
+               WL_ERROR("%s returns error %d\n", __func__, res);
                status = WL_SCAN_RESULTS_NO_MEM;
        }
        MUTEX_UNLOCK_WL_SCAN_SET();
@@ -1148,8 +1147,8 @@ static u32 wl_iw_iscan_get(iscan_info_t *iscan)
 
 static void wl_iw_force_specific_scan(iscan_info_t *iscan)
 {
-       WL_TRACE(("%s force Specific SCAN for %s\n", __func__,
-                 g_specific_ssid.SSID));
+       WL_TRACE("%s force Specific SCAN for %s\n",
+                __func__, g_specific_ssid.SSID);
        rtnl_lock();
 
        (void)dev_wlc_ioctl(iscan->dev, WLC_SCAN, &g_specific_ssid,
@@ -1166,7 +1165,7 @@ static void wl_iw_send_scan_complete(iscan_info_t *iscan)
        memset(&wrqu, 0, sizeof(wrqu));
 
        wireless_send_event(iscan->dev, SIOCGIWSCAN, &wrqu, NULL);
-       WL_TRACE(("Send Event ISCAN complete\n"));
+       WL_TRACE("Send Event ISCAN complete\n");
 #endif
 }
 
@@ -1190,8 +1189,8 @@ static int _iscan_sysioc_thread(void *data)
                status = wl_iw_iscan_get(iscan);
                rtnl_unlock();
                if (g_scan_specified_ssid && (iscan_pass_abort == true)) {
-                       WL_TRACE(("%s Get results from specific scan "
-                               "status = %d\n", __func__, status));
+                       WL_TRACE("%s Get results from specific scan status = %d\n",
+                                __func__, status);
                        wl_iw_send_scan_complete(iscan);
                        iscan_pass_abort = false;
                        status = -1;
@@ -1199,7 +1198,7 @@ static int _iscan_sysioc_thread(void *data)
 
                switch (status) {
                case WL_SCAN_RESULTS_PARTIAL:
-                       WL_TRACE(("iscanresults incomplete\n"));
+                       WL_TRACE("iscanresults incomplete\n");
                        rtnl_lock();
                        wl_iw_iscan(iscan, NULL, WL_SCAN_ACTION_CONTINUE);
                        rtnl_unlock();
@@ -1208,18 +1207,18 @@ static int _iscan_sysioc_thread(void *data)
                        iscan->timer_on = 1;
                        break;
                case WL_SCAN_RESULTS_SUCCESS:
-                       WL_TRACE(("iscanresults complete\n"));
+                       WL_TRACE("iscanresults complete\n");
                        iscan->iscan_state = ISCAN_STATE_IDLE;
                        wl_iw_send_scan_complete(iscan);
                        break;
                case WL_SCAN_RESULTS_PENDING:
-                       WL_TRACE(("iscanresults pending\n"));
+                       WL_TRACE("iscanresults pending\n");
                        mod_timer(&iscan->timer,
                                  jiffies + iscan->timer_ms * HZ / 1000);
                        iscan->timer_on = 1;
                        break;
                case WL_SCAN_RESULTS_ABORTED:
-                       WL_TRACE(("iscanresults aborted\n"));
+                       WL_TRACE("iscanresults aborted\n");
                        iscan->iscan_state = ISCAN_STATE_IDLE;
                        if (g_scan_specified_ssid == 0)
                                wl_iw_send_scan_complete(iscan);
@@ -1229,12 +1228,12 @@ static int _iscan_sysioc_thread(void *data)
                        }
                        break;
                case WL_SCAN_RESULTS_NO_MEM:
-                       WL_TRACE(("iscanresults can't alloc memory: skip\n"));
+                       WL_TRACE("iscanresults can't alloc memory: skip\n");
                        iscan->iscan_state = ISCAN_STATE_IDLE;
                        break;
                default:
-                       WL_TRACE(("iscanresults returned unknown status %d\n",
-                                 status));
+                       WL_TRACE("iscanresults returned unknown status %d\n",
+                                status);
                        break;
                }
        }
@@ -1253,11 +1252,11 @@ wl_iw_set_scan(struct net_device *dev,
               union iwreq_data *wrqu, char *extra)
 {
        int error;
-       WL_TRACE(("\n:%s dev:%s: SIOCSIWSCAN : SCAN\n", __func__, dev->name));
+       WL_TRACE("\n:%s dev:%s: SIOCSIWSCAN : SCAN\n", __func__, dev->name);
 
        g_set_essid_before_scan = false;
 #if defined(CSCAN)
-       WL_ERROR(("%s: Scan from SIOCGIWSCAN not supported\n", __func__));
+       WL_ERROR("%s: Scan from SIOCGIWSCAN not supported\n", __func__);
        return -EINVAL;
 #endif
 
@@ -1274,9 +1273,8 @@ wl_iw_set_scan(struct net_device *dev,
                if (wrqu->data.flags & IW_SCAN_THIS_ESSID) {
                        struct iw_scan_req *req = (struct iw_scan_req *)extra;
                        if (g_scan_specified_ssid) {
-                               WL_TRACE(("%s Specific SCAN is not done ignore "
-                                       "scan for = %s\n",
-                                       __func__, req->essid));
+                               WL_TRACE("%s Specific SCAN is not done ignore scan for = %s\n",
+                                        __func__, req->essid);
                                return -EBUSY;
                        } else {
                                g_specific_ssid.SSID_len = min_t(size_t,
@@ -1287,9 +1285,9 @@ wl_iw_set_scan(struct net_device *dev,
                                g_specific_ssid.SSID_len =
                                    htod32(g_specific_ssid.SSID_len);
                                g_scan_specified_ssid = 1;
-                               WL_TRACE(("### Specific scan ssid=%s len=%d\n",
-                                         g_specific_ssid.SSID,
-                                         g_specific_ssid.SSID_len));
+                               WL_TRACE("### Specific scan ssid=%s len=%d\n",
+                                        g_specific_ssid.SSID,
+                                        g_specific_ssid.SSID_len);
                        }
                }
        }
@@ -1297,8 +1295,8 @@ wl_iw_set_scan(struct net_device *dev,
        error = dev_wlc_ioctl(dev, WLC_SCAN, &g_specific_ssid,
                                sizeof(g_specific_ssid));
        if (error) {
-               WL_TRACE(("#### Set SCAN for %s failed with %d\n",
-                         g_specific_ssid.SSID, error));
+               WL_TRACE("#### Set SCAN for %s failed with %d\n",
+                        g_specific_ssid.SSID, error);
                g_scan_specified_ssid = 0;
                return -EBUSY;
        }
@@ -1317,7 +1315,7 @@ int wl_iw_iscan_set_scan_broadcast_prep(struct net_device *dev, uint flag)
 
        wl_iw_set_event_mask(dev);
 
-       WL_TRACE(("+++: Set Broadcast ISCAN\n"));
+       WL_TRACE("+++: Set Broadcast ISCAN\n");
        memset(&ssid, 0, sizeof(ssid));
 
        iscan->list_cur = iscan->list_hdr;
@@ -1346,20 +1344,20 @@ wl_iw_iscan_set_scan(struct net_device *dev,
        wlc_ssid_t ssid;
        iscan_info_t *iscan = g_iscan;
 
-       WL_TRACE(("%s: SIOCSIWSCAN : ISCAN\n", dev->name));
+       WL_TRACE("%s: SIOCSIWSCAN : ISCAN\n", dev->name);
 
 #if defined(CSCAN)
-       WL_ERROR(("%s: Scan from SIOCGIWSCAN not supported\n", __func__));
+       WL_ERROR("%s: Scan from SIOCGIWSCAN not supported\n", __func__);
        return -EINVAL;
 #endif
 
        if (g_onoff == G_WLAN_SET_OFF) {
-               WL_TRACE(("%s: driver is not up yet after START\n", __func__));
+               WL_TRACE("%s: driver is not up yet after START\n", __func__);
                return 0;
        }
 #ifdef PNO_SUPPORT
        if (dhd_dev_get_pno_status(dev)) {
-               WL_ERROR(("%s: Scan called when PNO is active\n", __func__));
+               WL_ERROR("%s: Scan called when PNO is active\n", __func__);
        }
 #endif
 
@@ -1367,8 +1365,8 @@ wl_iw_iscan_set_scan(struct net_device *dev,
                return wl_iw_set_scan(dev, info, wrqu, extra);
 
        if (g_scan_specified_ssid) {
-               WL_TRACE(("%s Specific SCAN already running ignoring BC scan\n",
-                         __func__));
+               WL_TRACE("%s Specific SCAN already running ignoring BC scan\n",
+                        __func__);
                return EBUSY;
        }
 
@@ -1386,8 +1384,8 @@ wl_iw_iscan_set_scan(struct net_device *dev,
                        g_scan_specified_ssid = 0;
 
                        if (iscan->iscan_state == ISCAN_STATE_SCANING) {
-                               WL_TRACE(("%s ISCAN already in progress \n",
-                                         __func__));
+                               WL_TRACE("%s ISCAN already in progress\n",
+                                        __func__);
                                return 0;
                        }
                }
@@ -1406,7 +1404,7 @@ static bool ie_is_wpa_ie(u8 **wpaie, u8 **tlvs, int *tlvs_len)
        u8 *ie = *wpaie;
 
        if ((ie[1] >= 6) &&
-           !bcmp((const void *)&ie[2], (const void *)(WPA_OUI "\x01"), 4)) {
+           !memcmp((const void *)&ie[2], (const void *)(WPA_OUI "\x01"), 4)) {
                return true;
        }
 
@@ -1422,7 +1420,7 @@ static bool ie_is_wps_ie(u8 **wpsie, u8 **tlvs, int *tlvs_len)
        u8 *ie = *wpsie;
 
        if ((ie[1] >= 4) &&
-           !bcmp((const void *)&ie[2], (const void *)(WPA_OUI "\x04"), 4)) {
+           !memcmp((const void *)&ie[2], (const void *)(WPA_OUI "\x04"), 4)) {
                return true;
        }
 
@@ -1501,9 +1499,8 @@ wl_iw_get_scan_prep(wl_scan_results_t *list,
 
        for (i = 0; i < list->count && i < IW_MAX_AP; i++) {
                if (list->version != WL_BSS_INFO_VERSION) {
-                       WL_ERROR(("%s : list->version %d != "
-                               "WL_BSS_INFO_VERSION\n",
-                               __func__, list->version));
+                       WL_ERROR("%s : list->version %d != WL_BSS_INFO_VERSION\n",
+                                __func__, list->version);
                        return ret;
                }
 
@@ -1511,11 +1508,11 @@ wl_iw_get_scan_prep(wl_scan_results_t *list,
                                             dtoh32(bi->length)) : list->
                    bss_info;
 
-               WL_TRACE(("%s : %s\n", __func__, bi->SSID));
+               WL_TRACE("%s : %s\n", __func__, bi->SSID);
 
                iwe.cmd = SIOCGIWAP;
                iwe.u.ap_addr.sa_family = ARPHRD_ETHER;
-               memcpy(iwe.u.ap_addr.sa_data, &bi->BSSID, ETHER_ADDR_LEN);
+               memcpy(iwe.u.ap_addr.sa_data, &bi->BSSID, ETH_ALEN);
                event =
                    IWE_STREAM_ADD_EVENT(info, event, end, &iwe,
                                         IW_EV_ADDR_LEN);
@@ -1590,11 +1587,11 @@ wl_iw_get_scan_prep(wl_scan_results_t *list,
 
        ret = event - extra;
        if (ret < 0) {
-               WL_ERROR(("==> Wrong size\n"));
+               WL_ERROR("==> Wrong size\n");
                ret = 0;
        }
-       WL_TRACE(("%s: size=%d bytes prepared\n", __func__,
-                 (unsigned int)(event - extra)));
+       WL_TRACE("%s: size=%d bytes prepared\n",
+                __func__, (unsigned int)(event - extra));
        return (uint)ret;
 }
 
@@ -1614,10 +1611,10 @@ wl_iw_get_scan(struct net_device *dev,
        iscan_buf_t *p_buf;
 #endif
 
-       WL_TRACE(("%s: buflen_from_user %d: \n", dev->name, buflen_from_user));
+       WL_TRACE("%s: buflen_from_user %d:\n", dev->name, buflen_from_user);
 
        if (!extra) {
-               WL_TRACE(("%s: wl_iw_get_scan return -EINVAL\n", dev->name));
+               WL_TRACE("%s: wl_iw_get_scan return -EINVAL\n", dev->name);
                return -EINVAL;
        }
 
@@ -1631,8 +1628,8 @@ wl_iw_get_scan(struct net_device *dev,
        if (g_scan_specified_ssid) {
                list = kmalloc(len, GFP_KERNEL);
                if (!list) {
-                       WL_TRACE(("%s: wl_iw_get_scan return -ENOMEM\n",
-                                 dev->name));
+                       WL_TRACE("%s: wl_iw_get_scan return -ENOMEM\n",
+                                dev->name);
                        g_scan_specified_ssid = 0;
                        return -ENOMEM;
                }
@@ -1642,8 +1639,8 @@ wl_iw_get_scan(struct net_device *dev,
        list->buflen = htod32(len);
        error = dev_wlc_ioctl(dev, WLC_SCAN_RESULTS, list, len);
        if (error) {
-               WL_ERROR(("%s: %s : Scan_results ERROR %d\n", dev->name,
-                         __func__, error));
+               WL_ERROR("%s: %s : Scan_results ERROR %d\n",
+                        dev->name, __func__, error);
                dwrq->length = len;
                if (g_scan_specified_ssid) {
                        g_scan_specified_ssid = 0;
@@ -1656,8 +1653,8 @@ wl_iw_get_scan(struct net_device *dev,
        list->count = dtoh32(list->count);
 
        if (list->version != WL_BSS_INFO_VERSION) {
-               WL_ERROR(("%s : list->version %d != WL_BSS_INFO_VERSION\n",
-                         __func__, list->version));
+               WL_ERROR("%s : list->version %d != WL_BSS_INFO_VERSION\n",
+                        __func__, list->version);
                if (g_scan_specified_ssid) {
                        g_scan_specified_ssid = 0;
                        kfree(list);
@@ -1666,8 +1663,8 @@ wl_iw_get_scan(struct net_device *dev,
        }
 
        if (g_scan_specified_ssid) {
-               WL_TRACE(("%s: Specified scan APs in the list =%d\n",
-                         __func__, list->count));
+               WL_TRACE("%s: Specified scan APs in the list =%d\n",
+                        __func__, list->count);
                len_ret =
                    (__u16) wl_iw_get_scan_prep(list, info, extra,
                                                buflen_from_user);
@@ -1678,8 +1675,8 @@ wl_iw_get_scan(struct net_device *dev,
                while (p_buf != iscan->list_cur) {
                        list_merge =
                            &((wl_iscan_results_t *) p_buf->iscan_buf)->results;
-                       WL_TRACE(("%s: Bcast APs list=%d\n", __func__,
-                                 list_merge->count));
+                       WL_TRACE("%s: Bcast APs list=%d\n",
+                                __func__, list_merge->count);
                        if (list_merge->count > 0)
                                len_ret +=
                                    (__u16) wl_iw_get_scan_prep(list_merge,
@@ -1689,8 +1686,8 @@ wl_iw_get_scan(struct net_device *dev,
                }
 #else
                list_merge = (wl_scan_results_t *) g_scan;
-               WL_TRACE(("%s: Bcast APs list=%d\n", __func__,
-                         list_merge->count));
+               WL_TRACE("%s: Bcast APs list=%d\n",
+                        __func__, list_merge->count);
                if (list_merge->count > 0)
                        len_ret +=
                            (__u16) wl_iw_get_scan_prep(list_merge, info,
@@ -1714,8 +1711,8 @@ wl_iw_get_scan(struct net_device *dev,
        dwrq->length = len;
        dwrq->flags = 0;
 
-       WL_TRACE(("%s return to WE %d bytes APs=%d\n", __func__,
-                 dwrq->length, list->count));
+       WL_TRACE("%s return to WE %d bytes APs=%d\n",
+                __func__, dwrq->length, list->count);
        return 0;
 }
 
@@ -1736,26 +1733,26 @@ wl_iw_iscan_get_scan(struct net_device *dev,
        u32 counter = 0;
        u8 channel;
 
-       WL_TRACE(("%s %s buflen_from_user %d:\n", dev->name, __func__,
-                 dwrq->length));
+       WL_TRACE("%s %s buflen_from_user %d:\n",
+                dev->name, __func__, dwrq->length);
 
        if (!extra) {
-               WL_TRACE(("%s: INVALID SIOCGIWSCAN GET bad parameter\n",
-                         dev->name));
+               WL_TRACE("%s: INVALID SIOCGIWSCAN GET bad parameter\n",
+                        dev->name);
                return -EINVAL;
        }
 
        if ((!iscan) || (!iscan->sysioc_tsk)) {
-               WL_ERROR(("%ssysioc_tsk\n", __func__));
+               WL_ERROR("%ssysioc_tsk\n", __func__);
                return wl_iw_get_scan(dev, info, dwrq, extra);
        }
 
        if (iscan->iscan_state == ISCAN_STATE_SCANING) {
-               WL_TRACE(("%s: SIOCGIWSCAN GET still scanning\n", dev->name));
+               WL_TRACE("%s: SIOCGIWSCAN GET still scanning\n", dev->name);
                return -EAGAIN;
        }
 
-       WL_TRACE(("%s: SIOCGIWSCAN GET broadcast results\n", dev->name));
+       WL_TRACE("%s: SIOCGIWSCAN GET broadcast results\n", dev->name);
        apcnt = 0;
        p_buf = iscan->list_hdr;
        while (p_buf != iscan->list_cur) {
@@ -1764,9 +1761,8 @@ wl_iw_iscan_get_scan(struct net_device *dev,
                counter += list->count;
 
                if (list->version != WL_BSS_INFO_VERSION) {
-                       WL_ERROR(("%s : list->version %d != "
-                               "WL_BSS_INFO_VERSION\n",
-                               __func__, list->version));
+                       WL_ERROR("%s : list->version %d != WL_BSS_INFO_VERSION\n",
+                                __func__, list->version);
                        return -EINVAL;
                }
 
@@ -1779,14 +1775,14 @@ wl_iw_iscan_get_scan(struct net_device *dev,
                        ASSERT(((unsigned long)bi + dtoh32(bi->length)) <=
                               ((unsigned long)list + WLC_IW_ISCAN_MAXLEN));
 
-                       if (event + ETHER_ADDR_LEN + bi->SSID_len +
+                       if (event + ETH_ALEN + bi->SSID_len +
                            IW_EV_UINT_LEN + IW_EV_FREQ_LEN + IW_EV_QUAL_LEN >=
                            end)
                                return -E2BIG;
                        iwe.cmd = SIOCGIWAP;
                        iwe.u.ap_addr.sa_family = ARPHRD_ETHER;
                        memcpy(iwe.u.ap_addr.sa_data, &bi->BSSID,
-                              ETHER_ADDR_LEN);
+                              ETH_ALEN);
                        event =
                            IWE_STREAM_ADD_EVENT(info, event, end, &iwe,
                                                 IW_EV_ADDR_LEN);
@@ -1876,8 +1872,8 @@ wl_iw_iscan_get_scan(struct net_device *dev,
        dwrq->length = event - extra;
        dwrq->flags = 0;
 
-       WL_TRACE(("%s return to WE %d bytes APs=%d\n", __func__,
-                 dwrq->length, counter));
+       WL_TRACE("%s return to WE %d bytes APs=%d\n",
+                __func__, dwrq->length, counter);
 
        if (!dwrq->length)
                return -EAGAIN;
@@ -1895,7 +1891,7 @@ wl_iw_set_essid(struct net_device *dev,
        wl_join_params_t join_params;
        int join_params_size;
 
-       WL_TRACE(("%s: SIOCSIWESSID\n", dev->name));
+       WL_TRACE("%s: SIOCSIWESSID\n", dev->name);
 
        if (g_set_essid_before_scan)
                return -EAGAIN;
@@ -1923,7 +1919,7 @@ wl_iw_set_essid(struct net_device *dev,
 
        memcpy(&join_params.ssid.SSID, g_ssid.SSID, g_ssid.SSID_len);
        join_params.ssid.SSID_len = htod32(g_ssid.SSID_len);
-       memcpy(&join_params.params.bssid, &ether_bcast, ETHER_ADDR_LEN);
+       memcpy(&join_params.params.bssid, &ether_bcast, ETH_ALEN);
 
        wl_iw_ch_to_chanspec(g_wl_iw_params.target_channel, &join_params,
                             &join_params_size);
@@ -1931,11 +1927,11 @@ wl_iw_set_essid(struct net_device *dev,
        error = dev_wlc_ioctl(dev, WLC_SET_SSID, &join_params,
                                join_params_size);
        if (error)
-               WL_ERROR(("Invalid ioctl data=%d\n", error));
+               WL_ERROR("Invalid ioctl data=%d\n", error);
 
        if (g_ssid.SSID_len) {
-               WL_TRACE(("%s: join SSID=%s ch=%d\n", __func__,
-                         g_ssid.SSID, g_wl_iw_params.target_channel));
+               WL_TRACE("%s: join SSID=%s ch=%d\n",
+                        __func__, g_ssid.SSID, g_wl_iw_params.target_channel);
        }
        return 0;
 }
@@ -1948,14 +1944,14 @@ wl_iw_get_essid(struct net_device *dev,
        wlc_ssid_t ssid;
        int error;
 
-       WL_TRACE(("%s: SIOCGIWESSID\n", dev->name));
+       WL_TRACE("%s: SIOCGIWESSID\n", dev->name);
 
        if (!extra)
                return -EINVAL;
 
        error = dev_wlc_ioctl(dev, WLC_GET_SSID, &ssid, sizeof(ssid));
        if (error) {
-               WL_ERROR(("Error getting the SSID\n"));
+               WL_ERROR("Error getting the SSID\n");
                return error;
        }
 
@@ -1976,7 +1972,7 @@ wl_iw_set_nick(struct net_device *dev,
 {
        wl_iw_t *iw = *(wl_iw_t **) netdev_priv(dev);
 
-       WL_TRACE(("%s: SIOCSIWNICKN\n", dev->name));
+       WL_TRACE("%s: SIOCSIWNICKN\n", dev->name);
 
        if (!extra)
                return -EINVAL;
@@ -1996,7 +1992,7 @@ wl_iw_get_nick(struct net_device *dev,
 {
        wl_iw_t *iw = *(wl_iw_t **) netdev_priv(dev);
 
-       WL_TRACE(("%s: SIOCGIWNICKN\n", dev->name));
+       WL_TRACE("%s: SIOCGIWNICKN\n", dev->name);
 
        if (!extra)
                return -EINVAL;
@@ -2014,7 +2010,7 @@ wl_iw_set_rate(struct net_device *dev,
        wl_rateset_t rateset;
        int error, rate, i, error_bg, error_a;
 
-       WL_TRACE(("%s: SIOCSIWRATE\n", dev->name));
+       WL_TRACE("%s: SIOCSIWRATE\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_CURR_RATESET, &rateset,
                                sizeof(rateset));
@@ -2063,7 +2059,7 @@ wl_iw_get_rate(struct net_device *dev,
 {
        int error, rate;
 
-       WL_TRACE(("%s: SIOCGIWRATE\n", dev->name));
+       WL_TRACE("%s: SIOCGIWRATE\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_RATE, &rate, sizeof(rate));
        if (error)
@@ -2080,7 +2076,7 @@ wl_iw_set_rts(struct net_device *dev,
 {
        int error, rts;
 
-       WL_TRACE(("%s: SIOCSIWRTS\n", dev->name));
+       WL_TRACE("%s: SIOCSIWRTS\n", dev->name);
 
        if (vwrq->disabled)
                rts = DOT11_DEFAULT_RTS_LEN;
@@ -2102,7 +2098,7 @@ wl_iw_get_rts(struct net_device *dev,
 {
        int error, rts;
 
-       WL_TRACE(("%s: SIOCGIWRTS\n", dev->name));
+       WL_TRACE("%s: SIOCGIWRTS\n", dev->name);
 
        error = dev_wlc_intvar_get(dev, "rtsthresh", &rts);
        if (error)
@@ -2121,7 +2117,7 @@ wl_iw_set_frag(struct net_device *dev,
 {
        int error, frag;
 
-       WL_TRACE(("%s: SIOCSIWFRAG\n", dev->name));
+       WL_TRACE("%s: SIOCSIWFRAG\n", dev->name);
 
        if (vwrq->disabled)
                frag = DOT11_DEFAULT_FRAG_LEN;
@@ -2143,7 +2139,7 @@ wl_iw_get_frag(struct net_device *dev,
 {
        int error, fragthreshold;
 
-       WL_TRACE(("%s: SIOCGIWFRAG\n", dev->name));
+       WL_TRACE("%s: SIOCGIWFRAG\n", dev->name);
 
        error = dev_wlc_intvar_get(dev, "fragthresh", &fragthreshold);
        if (error)
@@ -2163,7 +2159,7 @@ wl_iw_set_txpow(struct net_device *dev,
 {
        int error, disable;
        u16 txpwrmw;
-       WL_TRACE(("%s: SIOCSIWTXPOW\n", dev->name));
+       WL_TRACE("%s: SIOCSIWTXPOW\n", dev->name);
 
        disable = vwrq->disabled ? WL_RADIO_SW_DISABLE : 0;
        disable += WL_RADIO_SW_DISABLE << 16;
@@ -2200,7 +2196,7 @@ wl_iw_get_txpow(struct net_device *dev,
        int error, disable, txpwrdbm;
        u8 result;
 
-       WL_TRACE(("%s: SIOCGIWTXPOW\n", dev->name));
+       WL_TRACE("%s: SIOCGIWTXPOW\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_RADIO, &disable, sizeof(disable));
        if (error)
@@ -2229,7 +2225,7 @@ wl_iw_set_retry(struct net_device *dev,
 {
        int error, lrl, srl;
 
-       WL_TRACE(("%s: SIOCSIWRETRY\n", dev->name));
+       WL_TRACE("%s: SIOCSIWRETRY\n", dev->name);
 
        if (vwrq->disabled || (vwrq->flags & IW_RETRY_LIFETIME))
                return -EINVAL;
@@ -2277,7 +2273,7 @@ wl_iw_get_retry(struct net_device *dev,
 {
        int error, lrl, srl;
 
-       WL_TRACE(("%s: SIOCGIWRETRY\n", dev->name));
+       WL_TRACE("%s: SIOCGIWRETRY\n", dev->name);
 
        vwrq->disabled = 0;
 
@@ -2317,7 +2313,7 @@ wl_iw_set_encode(struct net_device *dev,
        wl_wsec_key_t key;
        int error, val, wsec;
 
-       WL_TRACE(("%s: SIOCSIWENCODE\n", dev->name));
+       WL_TRACE("%s: SIOCSIWENCODE\n", dev->name);
 
        memset(&key, 0, sizeof(key));
 
@@ -2409,9 +2405,9 @@ wl_iw_get_encode(struct net_device *dev,
        wl_wsec_key_t key;
        int error, val, wsec, auth;
 
-       WL_TRACE(("%s: SIOCGIWENCODE\n", dev->name));
+       WL_TRACE("%s: SIOCGIWENCODE\n", dev->name);
 
-       bzero(&key, sizeof(wl_wsec_key_t));
+       memset(&key, 0, sizeof(wl_wsec_key_t));
 
        if ((dwrq->flags & IW_ENCODE_INDEX) == 0) {
                for (key.index = 0; key.index < DOT11_MAX_DEFAULT_KEYS;
@@ -2465,7 +2461,7 @@ wl_iw_set_power(struct net_device *dev,
 {
        int error, pm;
 
-       WL_TRACE(("%s: SIOCSIWPOWER\n", dev->name));
+       WL_TRACE("%s: SIOCSIWPOWER\n", dev->name);
 
        pm = vwrq->disabled ? PM_OFF : PM_MAX;
 
@@ -2484,7 +2480,7 @@ wl_iw_get_power(struct net_device *dev,
 {
        int error, pm;
 
-       WL_TRACE(("%s: SIOCGIWPOWER\n", dev->name));
+       WL_TRACE("%s: SIOCGIWPOWER\n", dev->name);
 
        error = dev_wlc_ioctl(dev, WLC_GET_PM, &pm, sizeof(pm));
        if (error)
@@ -2503,7 +2499,7 @@ wl_iw_set_wpaie(struct net_device *dev,
                struct iw_request_info *info, struct iw_point *iwp, char *extra)
 {
 
-       WL_TRACE(("%s: SIOCSIWGENIE\n", dev->name));
+       WL_TRACE("%s: SIOCSIWGENIE\n", dev->name);
 
        CHECK_EXTRA_FOR_NULL(extra);
 
@@ -2516,7 +2512,7 @@ static int
 wl_iw_get_wpaie(struct net_device *dev,
                struct iw_request_info *info, struct iw_point *iwp, char *extra)
 {
-       WL_TRACE(("%s: SIOCGIWGENIE\n", dev->name));
+       WL_TRACE("%s: SIOCGIWGENIE\n", dev->name);
        iwp->length = 64;
        dev_wlc_bufvar_get(dev, "wpaie", extra, iwp->length);
        return 0;
@@ -2531,7 +2527,7 @@ wl_iw_set_encodeext(struct net_device *dev,
        int error;
        struct iw_encode_ext *iwe;
 
-       WL_TRACE(("%s: SIOCSIWENCODEEXT\n", dev->name));
+       WL_TRACE("%s: SIOCSIWENCODEEXT\n", dev->name);
 
        CHECK_EXTRA_FOR_NULL(extra);
 
@@ -2548,14 +2544,14 @@ wl_iw_set_encodeext(struct net_device *dev,
 
        key.len = iwe->key_len;
 
-       if (!ETHER_ISMULTI(iwe->addr.sa_data))
+       if (!is_multicast_ether_addr(iwe->addr.sa_data))
                bcopy((void *)&iwe->addr.sa_data, (char *)&key.ea,
-                     ETHER_ADDR_LEN);
+                     ETH_ALEN);
 
        if (key.len == 0) {
                if (iwe->ext_flags & IW_ENCODE_EXT_SET_TX_KEY) {
-                       WL_WSEC(("Changing the the primary Key to %d\n",
-                                key.index));
+                       WL_WSEC("Changing the the primary Key to %d\n",
+                               key.index);
                        key.index = htod32(key.index);
                        error = dev_wlc_ioctl(dev, WLC_SET_KEY_PRIMARY,
                                              &key.index, sizeof(key.index));
@@ -2569,9 +2565,9 @@ wl_iw_set_encodeext(struct net_device *dev,
                if (iwe->key_len > sizeof(key.data))
                        return -EINVAL;
 
-               WL_WSEC(("Setting the key index %d\n", key.index));
+               WL_WSEC("Setting the key index %d\n", key.index);
                if (iwe->ext_flags & IW_ENCODE_EXT_SET_TX_KEY) {
-                       WL_WSEC(("key is a Primary Key\n"));
+                       WL_WSEC("key is a Primary Key\n");
                        key.flags = WL_PRIMARY_KEY;
                }
 
@@ -2638,15 +2634,15 @@ wl_iw_set_pmksa(struct net_device *dev,
        uint i;
        int ret = 0;
 
-       WL_WSEC(("%s: SIOCSIWPMKSA\n", dev->name));
+       WL_WSEC("%s: SIOCSIWPMKSA\n", dev->name);
 
        CHECK_EXTRA_FOR_NULL(extra);
 
        iwpmksa = (struct iw_pmksa *)extra;
 
        if (iwpmksa->cmd == IW_PMKSA_FLUSH) {
-               WL_WSEC(("wl_iw_set_pmksa - IW_PMKSA_FLUSH\n"));
-               bzero((char *)&pmkid_list, sizeof(pmkid_list));
+               WL_WSEC("wl_iw_set_pmksa - IW_PMKSA_FLUSH\n");
+               memset((char *)&pmkid_list, 0, sizeof(pmkid_list));
        }
 
        else if (iwpmksa->cmd == IW_PMKSA_REMOVE) {
@@ -2656,30 +2652,30 @@ wl_iw_set_pmksa(struct net_device *dev,
                        pmkidptr = &pmkid;
 
                        bcopy(&iwpmksa->bssid.sa_data[0],
-                             &pmkidptr->pmkid[0].BSSID, ETHER_ADDR_LEN);
+                             &pmkidptr->pmkid[0].BSSID, ETH_ALEN);
                        bcopy(&iwpmksa->pmkid[0], &pmkidptr->pmkid[0].PMKID,
                              WPA2_PMKID_LEN);
 
-                       WL_WSEC(("wl_iw_set_pmksa:IW_PMKSA_REMOVE:PMKID: "
-                               "%pM = ", &pmkidptr->pmkid[0].BSSID));
+                       WL_WSEC("wl_iw_set_pmksa:IW_PMKSA_REMOVE:PMKID: %pM = ",
+                               &pmkidptr->pmkid[0].BSSID);
                        for (j = 0; j < WPA2_PMKID_LEN; j++)
-                               WL_WSEC(("%02x ", pmkidptr->pmkid[0].PMKID[j]));
-                       WL_WSEC(("\n"));
+                               WL_WSEC("%02x ", pmkidptr->pmkid[0].PMKID[j]);
+                       WL_WSEC("\n");
                }
 
                for (i = 0; i < pmkid_list.pmkids.npmkid; i++)
-                       if (!bcmp
+                       if (!memcmp
                            (&iwpmksa->bssid.sa_data[0],
-                            &pmkid_list.pmkids.pmkid[i].BSSID, ETHER_ADDR_LEN))
+                            &pmkid_list.pmkids.pmkid[i].BSSID, ETH_ALEN))
                                break;
 
                if ((pmkid_list.pmkids.npmkid > 0)
                    && (i < pmkid_list.pmkids.npmkid)) {
-                       bzero(&pmkid_list.pmkids.pmkid[i], sizeof(pmkid_t));
+                       memset(&pmkid_list.pmkids.pmkid[i], 0, sizeof(pmkid_t));
                        for (; i < (pmkid_list.pmkids.npmkid - 1); i++) {
                                bcopy(&pmkid_list.pmkids.pmkid[i + 1].BSSID,
                                      &pmkid_list.pmkids.pmkid[i].BSSID,
-                                     ETHER_ADDR_LEN);
+                                     ETH_ALEN);
                                bcopy(&pmkid_list.pmkids.pmkid[i + 1].PMKID,
                                      &pmkid_list.pmkids.pmkid[i].PMKID,
                                      WPA2_PMKID_LEN);
@@ -2691,14 +2687,14 @@ wl_iw_set_pmksa(struct net_device *dev,
 
        else if (iwpmksa->cmd == IW_PMKSA_ADD) {
                for (i = 0; i < pmkid_list.pmkids.npmkid; i++)
-                       if (!bcmp
+                       if (!memcmp
                            (&iwpmksa->bssid.sa_data[0],
-                            &pmkid_list.pmkids.pmkid[i].BSSID, ETHER_ADDR_LEN))
+                            &pmkid_list.pmkids.pmkid[i].BSSID, ETH_ALEN))
                                break;
                if (i < MAXPMKID) {
                        bcopy(&iwpmksa->bssid.sa_data[0],
                              &pmkid_list.pmkids.pmkid[i].BSSID,
-                             ETHER_ADDR_LEN);
+                             ETH_ALEN);
                        bcopy(&iwpmksa->pmkid[0],
                              &pmkid_list.pmkids.pmkid[i].PMKID,
                              WPA2_PMKID_LEN);
@@ -2710,25 +2706,25 @@ wl_iw_set_pmksa(struct net_device *dev,
                        uint j;
                        uint k;
                        k = pmkid_list.pmkids.npmkid;
-                       WL_WSEC(("wl_iw_set_pmksa,IW_PMKSA_ADD - PMKID: %pM = ",
-                               &pmkid_list.pmkids.pmkid[k].BSSID));
+                       WL_WSEC("wl_iw_set_pmksa,IW_PMKSA_ADD - PMKID: %pM = ",
+                               &pmkid_list.pmkids.pmkid[k].BSSID);
                        for (j = 0; j < WPA2_PMKID_LEN; j++)
-                               WL_WSEC(("%02x ",
-                                        pmkid_list.pmkids.pmkid[k].PMKID[j]));
-                       WL_WSEC(("\n"));
+                               WL_WSEC("%02x ",
+                                       pmkid_list.pmkids.pmkid[k].PMKID[j]);
+                       WL_WSEC("\n");
                }
        }
-       WL_WSEC(("PRINTING pmkid LIST - No of elements %d\n",
-                pmkid_list.pmkids.npmkid));
+       WL_WSEC("PRINTING pmkid LIST - No of elements %d\n",
+               pmkid_list.pmkids.npmkid);
        for (i = 0; i < pmkid_list.pmkids.npmkid; i++) {
                uint j;
-               WL_WSEC(("PMKID[%d]: %pM = ", i,
-                       &pmkid_list.pmkids.pmkid[i].BSSID));
+               WL_WSEC("PMKID[%d]: %pM = ",
+                       i, &pmkid_list.pmkids.pmkid[i].BSSID);
                for (j = 0; j < WPA2_PMKID_LEN; j++)
-                       WL_WSEC(("%02x ", pmkid_list.pmkids.pmkid[i].PMKID[j]));
-               WL_WSEC(("\n"));
+                       WL_WSEC("%02x ", pmkid_list.pmkids.pmkid[i].PMKID[j]);
+               WL_WSEC("\n");
        }
-       WL_WSEC(("\n"));
+       WL_WSEC("\n");
 
        if (!ret)
                ret = dev_wlc_bufvar_set(dev, "pmkid_info", (char *)&pmkid_list,
@@ -2742,7 +2738,7 @@ wl_iw_get_encodeext(struct net_device *dev,
                    struct iw_request_info *info,
                    struct iw_param *vwrq, char *extra)
 {
-       WL_TRACE(("%s: SIOCGIWENCODEEXT\n", dev->name));
+       WL_TRACE("%s: SIOCGIWENCODEEXT\n", dev->name);
        return 0;
 }
 
@@ -2757,13 +2753,13 @@ wl_iw_set_wpaauth(struct net_device *dev,
        int val = 0;
        wl_iw_t *iw = *(wl_iw_t **) netdev_priv(dev);
 
-       WL_TRACE(("%s: SIOCSIWAUTH\n", dev->name));
+       WL_TRACE("%s: SIOCSIWAUTH\n", dev->name);
 
        paramid = vwrq->flags & IW_AUTH_INDEX;
        paramval = vwrq->value;
 
-       WL_TRACE(("%s: SIOCSIWAUTH, paramid = 0x%0x, paramval = 0x%0x\n",
-                 dev->name, paramid, paramval));
+       WL_TRACE("%s: SIOCSIWAUTH, paramid = 0x%0x, paramval = 0x%0x\n",
+                dev->name, paramid, paramval);
 
        switch (paramid) {
        case IW_AUTH_WPA_VERSION:
@@ -2773,8 +2769,8 @@ wl_iw_set_wpaauth(struct net_device *dev,
                        val = WPA_AUTH_PSK | WPA_AUTH_UNSPECIFIED;
                else if (paramval & IW_AUTH_WPA_VERSION_WPA2)
                        val = WPA2_AUTH_PSK | WPA2_AUTH_UNSPECIFIED;
-               WL_INFORM(("%s: %d: setting wpa_auth to 0x%0x\n", __func__,
-                          __LINE__, val));
+               WL_INFORM("%s: %d: setting wpa_auth to 0x%0x\n",
+                         __func__, __LINE__, val);
                error = dev_wlc_intvar_set(dev, "wpa_auth", val);
                if (error)
                        return error;
@@ -2797,20 +2793,19 @@ wl_iw_set_wpaauth(struct net_device *dev,
                }
 
                if (iw->privacy_invoked && !val) {
-                       WL_WSEC(("%s: %s: 'Privacy invoked' true but clearing "
-                               "wsec, assuming " "we're a WPS enrollee\n",
-                               dev->name, __func__));
+                       WL_WSEC("%s: %s: 'Privacy invoked' true but clearing wsec, assuming we're a WPS enrollee\n",
+                               dev->name, __func__);
                        error = dev_wlc_intvar_set(dev, "is_WPS_enrollee",
                                                        true);
                        if (error) {
-                               WL_WSEC(("Failed to set is_WPS_enrollee\n"));
+                               WL_WSEC("Failed to set is_WPS_enrollee\n");
                                return error;
                        }
                } else if (val) {
                        error = dev_wlc_intvar_set(dev, "is_WPS_enrollee",
                                                        false);
                        if (error) {
-                               WL_WSEC(("Failed to clear is_WPS_enrollee\n"));
+                               WL_WSEC("Failed to clear is_WPS_enrollee\n");
                                return error;
                        }
                }
@@ -2837,8 +2832,8 @@ wl_iw_set_wpaauth(struct net_device *dev,
                        else
                                val = WPA2_AUTH_UNSPECIFIED;
                }
-               WL_INFORM(("%s: %d: setting wpa_auth to %d\n", __func__,
-                          __LINE__, val));
+               WL_INFORM("%s: %d: setting wpa_auth to %d\n",
+                         __func__, __LINE__, val);
                error = dev_wlc_intvar_set(dev, "wpa_auth", val);
                if (error)
                        return error;
@@ -2850,7 +2845,7 @@ wl_iw_set_wpaauth(struct net_device *dev,
                break;
 
        case IW_AUTH_80211_AUTH_ALG:
-               WL_INFORM(("Setting the D11auth %d\n", paramval));
+               WL_INFORM("Setting the D11auth %d\n", paramval);
                if (paramval == IW_AUTH_ALG_OPEN_SYSTEM)
                        val = 0;
                else if (paramval == IW_AUTH_ALG_SHARED_KEY)
@@ -2879,8 +2874,8 @@ wl_iw_set_wpaauth(struct net_device *dev,
                                dev_wlc_intvar_set(dev, "wsec", val);
                        }
                        val = 0;
-                       WL_INFORM(("%s: %d: setting wpa_auth to %d\n",
-                                  __func__, __LINE__, val));
+                       WL_INFORM("%s: %d: setting wpa_auth to %d\n",
+                                 __func__, __LINE__, val);
                        dev_wlc_intvar_set(dev, "wpa_auth", 0);
                        return error;
                }
@@ -2897,7 +2892,7 @@ wl_iw_set_wpaauth(struct net_device *dev,
 
 #if WIRELESS_EXT > 17
        case IW_AUTH_ROAMING_CONTROL:
-               WL_INFORM(("%s: IW_AUTH_ROAMING_CONTROL\n", __func__));
+               WL_INFORM("%s: IW_AUTH_ROAMING_CONTROL\n", __func__);
                break;
        case IW_AUTH_PRIVACY_INVOKED:
                {
@@ -2908,8 +2903,7 @@ wl_iw_set_wpaauth(struct net_device *dev,
                                error = dev_wlc_intvar_set(dev,
                                                "is_WPS_enrollee", false);
                                if (error) {
-                                       WL_WSEC(("Failed to clear iovar "
-                                               "is_WPS_enrollee\n"));
+                                       WL_WSEC("Failed to clear iovar is_WPS_enrollee\n");
                                        return error;
                                }
                        } else {
@@ -2923,8 +2917,7 @@ wl_iw_set_wpaauth(struct net_device *dev,
                                                        "is_WPS_enrollee",
                                                        true);
                                        if (error) {
-                                               WL_WSEC(("Failed to set iovar "
-                                               "is_WPS_enrollee\n"));
+                                               WL_WSEC("Failed to set iovar is_WPS_enrollee\n");
                                                return error;
                                        }
                                } else {
@@ -2932,8 +2925,7 @@ wl_iw_set_wpaauth(struct net_device *dev,
                                                        "is_WPS_enrollee",
                                                        false);
                                        if (error) {
-                                               WL_WSEC(("Failed to clear "
-                                                       "is_WPS_enrollee\n"));
+                                               WL_WSEC("Failed to clear is_WPS_enrollee\n");
                                                return error;
                                        }
                                }
@@ -2960,7 +2952,7 @@ wl_iw_get_wpaauth(struct net_device *dev,
        int val;
        wl_iw_t *iw = *(wl_iw_t **) netdev_priv(dev);
 
-       WL_TRACE(("%s: SIOCGIWAUTH\n", dev->name));
+       WL_TRACE("%s: SIOCGIWAUTH\n", dev->name);
 
        paramid = vwrq->flags & IW_AUTH_INDEX;
 
@@ -3040,7 +3032,7 @@ wl_iw_get_wpaauth(struct net_device *dev,
                break;
 #if WIRELESS_EXT > 17
        case IW_AUTH_ROAMING_CONTROL:
-               WL_ERROR(("%s: IW_AUTH_ROAMING_CONTROL\n", __func__));
+               WL_ERROR("%s: IW_AUTH_ROAMING_CONTROL\n", __func__);
                break;
        case IW_AUTH_PRIVACY_INVOKED:
                paramval = iw->privacy_invoked;
@@ -3157,19 +3149,19 @@ int wl_iw_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
        char *extra = NULL;
        int token_size = 1, max_tokens = 0, ret = 0;
 
-       WL_TRACE(("\n%s, cmd:%x alled via dhd->do_ioctl()entry point\n",
-                 __func__, cmd));
+       WL_TRACE("\n%s, cmd:%x alled via dhd->do_ioctl()entry point\n",
+                __func__, cmd);
        if (cmd < SIOCIWFIRST ||
                IW_IOCTL_IDX(cmd) >= ARRAY_SIZE(wl_iw_handler)) {
-               WL_ERROR(("%s: error in cmd=%x : out of range\n", __func__,
-                       cmd));
+               WL_ERROR("%s: error in cmd=%x : out of range\n",
+                        __func__, cmd);
                return -EOPNOTSUPP;
        }
 
        handler = wl_iw_handler[IW_IOCTL_IDX(cmd)];
        if (!handler) {
-               WL_ERROR(("%s: error in cmd=%x : not supported\n",
-                       __func__, cmd));
+               WL_ERROR("%s: error in cmd=%x : not supported\n",
+                        __func__, cmd);
                return -EOPNOTSUPP;
        }
 
@@ -3234,9 +3226,8 @@ int wl_iw_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
 
        if (max_tokens && wrq->u.data.pointer) {
                if (wrq->u.data.length > max_tokens) {
-                       WL_ERROR(("%s: error in cmd=%x wrq->u.data.length=%d "
-                       "> max_tokens=%d\n",
-                       __func__, cmd, wrq->u.data.length, max_tokens));
+                       WL_ERROR("%s: error in cmd=%x wrq->u.data.length=%d > max_tokens=%d\n",
+                                __func__, cmd, wrq->u.data.length, max_tokens);
                        return -E2BIG;
                }
                extra = kmalloc(max_tokens * token_size, GFP_KERNEL);
@@ -3339,7 +3330,7 @@ wl_iw_conn_status_str(u32 event_type, u32 status, u32 reason,
                memset(stringBuf, 0, buflen);
                snprintf(stringBuf, buflen, "%s %s %02d %02d",
                         name, cause, status, reason);
-               WL_INFORM(("Connection status: %s\n", stringBuf));
+               WL_INFORM("Connection status: %s\n", stringBuf);
                return true;
        } else {
                return false;
@@ -3383,46 +3374,46 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
        iw = 0;
 
        if (!dev) {
-               WL_ERROR(("%s: dev is null\n", __func__));
+               WL_ERROR("%s: dev is null\n", __func__);
                return;
        }
 
        iw = *(wl_iw_t **) netdev_priv(dev);
 
-       WL_TRACE(("%s: dev=%s event=%d\n", __func__, dev->name, event_type));
+       WL_TRACE("%s: dev=%s event=%d\n", __func__, dev->name, event_type);
 
        switch (event_type) {
        case WLC_E_TXFAIL:
                cmd = IWEVTXDROP;
-               memcpy(wrqu.addr.sa_data, &e->addr, ETHER_ADDR_LEN);
+               memcpy(wrqu.addr.sa_data, &e->addr, ETH_ALEN);
                wrqu.addr.sa_family = ARPHRD_ETHER;
                break;
 #if WIRELESS_EXT > 14
        case WLC_E_JOIN:
        case WLC_E_ASSOC_IND:
        case WLC_E_REASSOC_IND:
-               memcpy(wrqu.addr.sa_data, &e->addr, ETHER_ADDR_LEN);
+               memcpy(wrqu.addr.sa_data, &e->addr, ETH_ALEN);
                wrqu.addr.sa_family = ARPHRD_ETHER;
                cmd = IWEVREGISTERED;
                break;
        case WLC_E_DEAUTH_IND:
        case WLC_E_DISASSOC_IND:
                cmd = SIOCGIWAP;
-               bzero(wrqu.addr.sa_data, ETHER_ADDR_LEN);
+               memset(wrqu.addr.sa_data, 0, ETH_ALEN);
                wrqu.addr.sa_family = ARPHRD_ETHER;
-               bzero(&extra, ETHER_ADDR_LEN);
+               memset(&extra, 0, ETH_ALEN);
                break;
        case WLC_E_LINK:
        case WLC_E_NDIS_LINK:
                cmd = SIOCGIWAP;
                if (!(flags & WLC_EVENT_MSG_LINK)) {
-                       bzero(wrqu.addr.sa_data, ETHER_ADDR_LEN);
-                       bzero(&extra, ETHER_ADDR_LEN);
+                       memset(wrqu.addr.sa_data, 0, ETH_ALEN);
+                       memset(&extra, 0, ETH_ALEN);
                        WAKE_LOCK_TIMEOUT(iw->pub, WAKE_LOCK_LINK_DOWN_TMOUT,
                                          20 * HZ);
                } else {
-                       memcpy(wrqu.addr.sa_data, &e->addr, ETHER_ADDR_LEN);
-                       WL_TRACE(("Link UP\n"));
+                       memcpy(wrqu.addr.sa_data, &e->addr, ETH_ALEN);
+                       WL_TRACE("Link UP\n");
 
                }
                wrqu.addr.sa_family = ARPHRD_ETHER;
@@ -3433,8 +3424,8 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                        wrqu.data.length = datalen + 1;
                        extra[0] = WLC_E_ACTION_FRAME;
                        memcpy(&extra[1], data, datalen);
-                       WL_TRACE(("WLC_E_ACTION_FRAME len %d \n",
-                                 wrqu.data.length));
+                       WL_TRACE("WLC_E_ACTION_FRAME len %d\n",
+                                wrqu.data.length);
                }
                break;
 
@@ -3464,7 +3455,7 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                        else
                                micerrevt->flags |= IW_MICFAILURE_PAIRWISE;
                        memcpy(micerrevt->src_addr.sa_data, &e->addr,
-                              ETHER_ADDR_LEN);
+                              ETH_ALEN);
                        micerrevt->src_addr.sa_family = ARPHRD_ETHER;
 
                        break;
@@ -3487,14 +3478,14 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                                wrqu.data.length = sizeof(struct iw_pmkid_cand);
                                pmkidcand = pmkcandlist->pmkid_cand;
                                while (count) {
-                                       bzero(iwpmkidcand,
+                                       memset(iwpmkidcand, 0,
                                              sizeof(struct iw_pmkid_cand));
                                        if (pmkidcand->preauth)
                                                iwpmkidcand->flags |=
                                                    IW_PMKID_CAND_PREAUTH;
                                        bcopy(&pmkidcand->BSSID,
                                              &iwpmkidcand->bssid.sa_data,
-                                             ETHER_ADDR_LEN);
+                                             ETH_ALEN);
 #ifndef SANDGATE2G
                                        wireless_send_event(dev, cmd, &wrqu,
                                                            extra);
@@ -3515,13 +3506,13 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                } else {
                        cmd = SIOCGIWSCAN;
                        wrqu.data.length = strlen(extra);
-                       WL_TRACE(("Event WLC_E_SCAN_COMPLETE from specific "
-                               "scan %d\n", g_iscan->iscan_state));
+                       WL_TRACE("Event WLC_E_SCAN_COMPLETE from specific scan %d\n",
+                                g_iscan->iscan_state);
                }
 #else
                cmd = SIOCGIWSCAN;
                wrqu.data.length = strlen(extra);
-               WL_TRACE(("Event WLC_E_SCAN_COMPLETE\n"));
+               WL_TRACE("Event WLC_E_SCAN_COMPLETE\n");
 #endif
                break;
 
@@ -3529,9 +3520,9 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                {
                        wlc_ssid_t *ssid;
                        ssid = (wlc_ssid_t *) data;
-                       WL_ERROR(("%s Event WLC_E_PFN_NET_FOUND, send %s up : "
-                               "find %s len=%d\n", __func__, PNO_EVENT_UP,
-                               ssid->SSID, ssid->SSID_len));
+                       WL_ERROR("%s Event WLC_E_PFN_NET_FOUND, send %s up : find %s len=%d\n",
+                                __func__, PNO_EVENT_UP,
+                                ssid->SSID, ssid->SSID_len);
                        WAKE_LOCK_TIMEOUT(iw->pub, WAKE_LOCK_PNO_FIND_TMOUT,
                                          20 * HZ);
                        cmd = IWEVCUSTOM;
@@ -3542,7 +3533,7 @@ void wl_iw_event(struct net_device *dev, wl_event_msg_t *e, void *data)
                break;
 
        default:
-               WL_TRACE(("Unknown Event %d: ignoring\n", event_type));
+               WL_TRACE("Unknown Event %d: ignoring\n", event_type);
                break;
        }
 #ifndef SANDGATE2G
@@ -3583,15 +3574,15 @@ wl_iw_get_wireless_stats(struct net_device *dev, struct iw_statistics *wstats)
                goto done;
 
        phy_noise = dtoh32(phy_noise);
-       WL_TRACE(("wl_iw_get_wireless_stats phy noise=%d\n", phy_noise));
+       WL_TRACE("wl_iw_get_wireless_stats phy noise=%d\n", phy_noise);
 
-       bzero(&scb_val, sizeof(scb_val_t));
+       memset(&scb_val, 0, sizeof(scb_val_t));
        res = dev_wlc_ioctl(dev, WLC_GET_RSSI, &scb_val, sizeof(scb_val_t));
        if (res)
                goto done;
 
        rssi = dtoh32(scb_val.val);
-       WL_TRACE(("wl_iw_get_wireless_stats rssi=%d\n", rssi));
+       WL_TRACE("wl_iw_get_wireless_stats rssi=%d\n", rssi);
        if (rssi <= WL_IW_RSSI_NO_SIGNAL)
                wstats->qual.qual = 0;
        else if (rssi <= WL_IW_RSSI_VERY_LOW)
@@ -3614,23 +3605,21 @@ wl_iw_get_wireless_stats(struct net_device *dev, struct iw_statistics *wstats)
 #endif
 
 #if WIRELESS_EXT > 11
-       WL_TRACE(("wl_iw_get_wireless_stats counters=%d\n",
-                 (int)sizeof(wl_cnt_t)));
+       WL_TRACE("wl_iw_get_wireless_stats counters=%zu\n", sizeof(wl_cnt_t));
 
        memset(&cnt, 0, sizeof(wl_cnt_t));
        res =
            dev_wlc_bufvar_get(dev, "counters", (char *)&cnt, sizeof(wl_cnt_t));
        if (res) {
-               WL_ERROR(("wl_iw_get_wireless_stats counters failed error=%d\n",
-                         res));
+               WL_ERROR("wl_iw_get_wireless_stats counters failed error=%d\n",
+                        res);
                goto done;
        }
 
        cnt.version = dtoh16(cnt.version);
        if (cnt.version != WL_CNT_T_VERSION) {
-               WL_TRACE(("\tIncorrect version of counters struct: expected "
-                       "%d; got %d\n",
-                       WL_CNT_T_VERSION, cnt.version));
+               WL_TRACE("\tIncorrect version of counters struct: expected %d; got %d\n",
+                        WL_CNT_T_VERSION, cnt.version);
                goto done;
        }
 
@@ -3641,22 +3630,22 @@ wl_iw_get_wireless_stats(struct net_device *dev, struct iw_statistics *wstats)
        wstats->discard.misc = dtoh32(cnt.rxrunt) + dtoh32(cnt.rxgiant);
        wstats->miss.beacon = 0;
 
-       WL_TRACE(("wl_iw_get_wireless_stats counters txframe=%d txbyte=%d\n",
-                 dtoh32(cnt.txframe), dtoh32(cnt.txbyte)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxfrmtoolong=%d\n",
-                 dtoh32(cnt.rxfrmtoolong)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxbadplcp=%d\n",
-                 dtoh32(cnt.rxbadplcp)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxundec=%d\n",
-                 dtoh32(cnt.rxundec)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxfragerr=%d\n",
-                 dtoh32(cnt.rxfragerr)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters txfail=%d\n",
-                 dtoh32(cnt.txfail)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxrunt=%d\n",
-                 dtoh32(cnt.rxrunt)));
-       WL_TRACE(("wl_iw_get_wireless_stats counters rxgiant=%d\n",
-                 dtoh32(cnt.rxgiant)));
+       WL_TRACE("wl_iw_get_wireless_stats counters txframe=%d txbyte=%d\n",
+                dtoh32(cnt.txframe), dtoh32(cnt.txbyte));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxfrmtoolong=%d\n",
+                 dtoh32(cnt.rxfrmtoolong));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxbadplcp=%d\n",
+                 dtoh32(cnt.rxbadplcp));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxundec=%d\n",
+                 dtoh32(cnt.rxundec));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxfragerr=%d\n",
+                 dtoh32(cnt.rxfragerr));
+       WL_TRACE("wl_iw_get_wireless_stats counters txfail=%d\n",
+                 dtoh32(cnt.txfail));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxrunt=%d\n",
+                 dtoh32(cnt.rxrunt));
+       WL_TRACE("wl_iw_get_wireless_stats counters rxgiant=%d\n",
+                 dtoh32(cnt.rxgiant));
 #endif                         /* WIRELESS_EXT > 11 */
 
 done:
@@ -3690,8 +3679,7 @@ int wl_iw_attach(struct net_device *dev, void *dhdp)
                return -ENOMEM;
        memset(iscan, 0, sizeof(iscan_info_t));
 
-       iscan->iscan_ex_params_p =
-           (wl_iscan_params_t *) kmalloc(params_size, GFP_KERNEL);
+       iscan->iscan_ex_params_p = kmalloc(params_size, GFP_KERNEL);
        if (!iscan->iscan_ex_params_p)
                return -ENOMEM;
        iscan->iscan_ex_param_size = params_size;
@@ -3723,9 +3711,7 @@ int wl_iw_attach(struct net_device *dev, void *dhdp)
        priv_dev = dev;
        MUTEX_LOCK_SOFTAP_SET_INIT(iw->pub);
 #endif
-       g_scan = NULL;
-
-       g_scan = (void *)kmalloc(G_SCAN_RESULTS, GFP_KERNEL);
+       g_scan = kmalloc(G_SCAN_RESULTS, GFP_KERNEL);
        if (!g_scan)
                return -ENOMEM;