5 files changed, 14 insertions, 30 deletions

diff --git a/drivers/net/wireless/ipw2100.c b/drivers/net/wireless/ipw2100.c
index fc6cdd8086c1..dc3813b26440 100644
--- a/drivers/net/wireless/ipw2100.c
+++ b/drivers/net/wireless/ipw2100.c
@@ -6591,8 +6591,7 @@ static const long ipw2100_frequencies[] = {
         2472, 2484
 };
 
-#define FREQ_COUNT (sizeof(ipw2100_frequencies) / \
-                    sizeof(ipw2100_frequencies[0]))
+#define FREQ_COUNT      ARRAY_SIZE(ipw2100_frequencies)
 
 static const long ipw2100_rates_11b[] = {
         1000000,
diff --git a/drivers/net/wireless/libertas/11d.c b/drivers/net/wireless/libertas/11d.c
index 9cf0211de67f..8bbd3227e6fb 100644
--- a/drivers/net/wireless/libertas/11d.c
+++ b/drivers/net/wireless/libertas/11d.c
@@ -46,13 +46,11 @@ static struct chan_freq_power channel_freq_power_UN_BG[] = {
 static u8 wlan_region_2_code(u8 * region)
 {
         u8 i;
-        u8 size = sizeof(region_code_mapping)/
-                  sizeof(struct region_code_mapping);
 
         for (i = 0; region[i] && i < COUNTRY_CODE_LEN; i++)
                 region[i] = toupper(region[i]);
 
-        for (i = 0; i < size; i++) {
+        for (i = 0; i < ARRAY_SIZE(region_code_mapping); i++) {
                 if (!memcmp(region, region_code_mapping[i].region,
                             COUNTRY_CODE_LEN))
                         return (region_code_mapping[i].code);
@@ -65,9 +63,8 @@ static u8 wlan_region_2_code(u8 * region)
 static u8 *wlan_code_2_region(u8 code)
 {
         u8 i;
-        u8 size = sizeof(region_code_mapping)
-                  / sizeof(struct region_code_mapping);
-        for (i = 0; i < size; i++) {
+
+        for (i = 0; i < ARRAY_SIZE(region_code_mapping); i++) {
                 if (region_code_mapping[i].code == code)
                         return (region_code_mapping[i].region);
         }
@@ -90,8 +87,7 @@ static u8 wlan_get_chan_11d(u8 band, u8 firstchan, u8 nrchan, u8 * chan)
         u8 cfp_no;
 
         cfp = channel_freq_power_UN_BG;
-        cfp_no = sizeof(channel_freq_power_UN_BG) /
-            sizeof(struct chan_freq_power);
+        cfp_no = ARRAY_SIZE(channel_freq_power_UN_BG);
 
         for (i = 0; i < cfp_no; i++) {
                 if ((cfp + i)->channel == firstchan) {
@@ -141,16 +137,12 @@ static u8 wlan_channel_known_11d(u8 chan,
 u32 libertas_chan_2_freq(u8 chan, u8 band)
 {
         struct chan_freq_power *cf;
-        u16 cnt;
         u16 i;
         u32 freq = 0;
 
         cf = channel_freq_power_UN_BG;
-        cnt =
-            sizeof(channel_freq_power_UN_BG) /
-            sizeof(struct chan_freq_power);
 
-        for (i = 0; i < cnt; i++) {
+        for (i = 0; i < ARRAY_SIZE(channel_freq_power_UN_BG); i++) {
                 if (chan == cf[i].channel)
                         freq = cf[i].freq;
         }
@@ -664,8 +656,7 @@ int libertas_create_dnld_countryinfo_11d(wlan_private * priv)
         if (priv->adapter->enable11d) {
                 /* update parsed_region_chan_11; dnld domaininf to FW */
 
-                for (j = 0; j < sizeof(adapter->region_channel) /
-                     sizeof(adapter->region_channel[0]); j++) {
+                for (j = 0; j < ARRAY_SIZE(adapter->region_channel); j++) {
                         region_chan = &adapter->region_channel[j];
 
                         lbs_deb_11d("%d region_chan->band %d\n", j,
@@ -679,8 +670,7 @@ int libertas_create_dnld_countryinfo_11d(wlan_private * priv)
                         break;
                 }
 
-                if (j >= sizeof(adapter->region_channel) /
-                    sizeof(adapter->region_channel[0])) {
+                if (j >= ARRAY_SIZE(adapter->region_channel)) {
                         lbs_deb_11d("region_chan not found, band %d\n",
                                adapter->curbssparams.band);
                         ret = -1;
diff --git a/drivers/net/wireless/libertas/wext.c b/drivers/net/wireless/libertas/wext.c
index 4dec5e5565bb..4ef5b9e00c77 100644
--- a/drivers/net/wireless/libertas/wext.c
+++ b/drivers/net/wireless/libertas/wext.c
@@ -52,11 +52,9 @@ struct chan_freq_power *libertas_find_cfp_by_band_and_channel(wlan_adapter * ada
 {
         struct chan_freq_power *cfp = NULL;
         struct region_channel *rc;
-        int count = sizeof(adapter->region_channel) /
-            sizeof(adapter->region_channel[0]);
         int i, j;
 
-        for (j = 0; !cfp && (j < count); j++) {
+        for (j = 0; !cfp && (j < ARRAY_SIZE(adapter->region_channel)); j++) {
                 rc = &adapter->region_channel[j];
 
                 if (adapter->enable11d)
@@ -93,11 +91,9 @@ static struct chan_freq_power *find_cfp_by_band_and_freq(wlan_adapter * adapter,
 {
         struct chan_freq_power *cfp = NULL;
         struct region_channel *rc;
-        int count = sizeof(adapter->region_channel) /
-            sizeof(adapter->region_channel[0]);
         int i, j;
 
-        for (j = 0; !cfp && (j < count); j++) {
+        for (j = 0; !cfp && (j < ARRAY_SIZE(adapter->region_channel)); j++) {
                 rc = &adapter->region_channel[j];
 
                 if (adapter->enable11d)
@@ -624,8 +620,7 @@ static int wlan_get_range(struct net_device *dev, struct iw_request_info *info,
         }
         if (!flag) {
                 for (j = 0; (range->num_frequency < IW_MAX_FREQUENCIES)
-                     && (j < sizeof(adapter->region_channel)
-                         / sizeof(adapter->region_channel[0])); j++) {
+                     && (j < ARRAY_SIZE(adapter->region_channel)); j++) {
                         cfp = adapter->region_channel[j].CFP;
                         for (i = 0; (range->num_frequency < IW_MAX_FREQUENCIES)
                              && adapter->region_channel[j].valid
diff --git a/drivers/net/wireless/wavelan.c b/drivers/net/wireless/wavelan.c
index a1f8a1687842..b343ce2235eb 100644
--- a/drivers/net/wireless/wavelan.c
+++ b/drivers/net/wireless/wavelan.c
@@ -3740,7 +3740,7 @@ static int wv_check_ioaddr(unsigned long ioaddr, u8 * mac)
          * non-NCR/AT&T/Lucent ISA card.  See wavelan.p.h for detail on
          * how to configure your card.
          */
-        for (i = 0; i < (sizeof(MAC_ADDRESSES) / sizeof(char) / 3); i++)
+        for (i = 0; i < ARRAY_SIZE(MAC_ADDRESSES); i++)
                 if ((mac[0] == MAC_ADDRESSES[i][0]) &&
                     (mac[1] == MAC_ADDRESSES[i][1]) &&
                     (mac[2] == MAC_ADDRESSES[i][2]))
diff --git a/drivers/net/wireless/wavelan_cs.c b/drivers/net/wireless/wavelan_cs.c
index 577c647824fe..f5b38de4f0df 100644
--- a/drivers/net/wireless/wavelan_cs.c
+++ b/drivers/net/wireless/wavelan_cs.c
@@ -3223,14 +3223,14 @@ wv_mmc_init(struct net_device *	dev)
    * non-NCR/AT&T/Lucent PCMCIA cards, see wavelan_cs.h for detail on
    * how to configure your card...
    */
-  for(i = 0; i < (sizeof(MAC_ADDRESSES) / sizeof(char) / 3); i++)
+  for(i = 0; i < ARRAY_SIZE(MAC_ADDRESSES); i++)
     if((psa.psa_univ_mac_addr[0] == MAC_ADDRESSES[i][0]) &&
        (psa.psa_univ_mac_addr[1] == MAC_ADDRESSES[i][1]) &&
        (psa.psa_univ_mac_addr[2] == MAC_ADDRESSES[i][2]))
       break;
 
   /* If we have not found it... */
-  if(i == (sizeof(MAC_ADDRESSES) / sizeof(char) / 3))
+  if(i == ARRAY_SIZE(MAC_ADDRESSES))
     {
 #ifdef DEBUG_CONFIG_ERRORS
       printk(KERN_WARNING "%s: wv_mmc_init(): Invalid MAC address: %02X:%02X:%02X:...\n",