Staging: ARRAY_SIZE changes
authorStoyan Gaydarov <sgayda2@uiuc.edu>
Wed, 22 Jul 2009 03:02:27 +0000 (22:02 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 15 Sep 2009 19:02:06 +0000 (12:02 -0700)
These changes were a direct result of using a semantic patch
More information can be found at http://www.emn.fr/x-info/coccinelle/

Signed-off-by: Stoyan Gaydarov <sgayda2@uiuc.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
29 files changed:
drivers/staging/comedi/drivers/amplc_pci230.c
drivers/staging/comedi/drivers/cb_das16_cs.c
drivers/staging/comedi/drivers/das08_cs.c
drivers/staging/comedi/drivers/ni_6527.c
drivers/staging/comedi/drivers/ni_65xx.c
drivers/staging/comedi/drivers/ni_660x.c
drivers/staging/comedi/drivers/ni_670x.c
drivers/staging/comedi/drivers/ni_atmio16d.c
drivers/staging/comedi/drivers/ni_pcidio.c
drivers/staging/comedi/drivers/ni_pcimio.c
drivers/staging/comedi/drivers/pcmad.c
drivers/staging/comedi/drivers/poc.c
drivers/staging/epl/EplDllk.c
drivers/staging/epl/proc_fs.c
drivers/staging/line6/driver.c
drivers/staging/meilhaus/me0600_device.h
drivers/staging/meilhaus/me0900_device.h
drivers/staging/meilhaus/me1400_device.h
drivers/staging/meilhaus/me1600_device.h
drivers/staging/meilhaus/me4600_device.h
drivers/staging/meilhaus/me6000_device.h
drivers/staging/meilhaus/me8100_device.h
drivers/staging/meilhaus/me8200_device.h
drivers/staging/meilhaus/metempl_device.h
drivers/staging/rtl8187se/r8180_wx.c
drivers/staging/rtl8192su/ieee80211/ieee80211_wx.c
drivers/staging/rtl8192su/r8192U_wx.c
drivers/staging/slicoss/slicoss.c
drivers/staging/sxg/sxg.c

index c4d8c4cae4a5454dffb35b29f20af85d57f82ae3..091a1a5822a85d4c4fbb5a9924422cece6161fb5 100644 (file)
@@ -513,7 +513,7 @@ MODULE_DEVICE_TABLE(pci, pci230_pci_table);
 /*
  * Useful for shorthand access to the particular board structure
  */
-#define n_pci230_boards (sizeof(pci230_boards)/sizeof(pci230_boards[0]))
+#define n_pci230_boards ARRAY_SIZE(pci230_boards)
 #define thisboard ((const struct pci230_board *)dev->board_ptr)
 
 /* this structure is for data unique to this hardware driver.  If
index 2f7aa3b5ef8ebf7e8fbb160ccdb3ff47b67e8f6c..12d12b43a6f1349381797c0837cc4eac0db761cf 100644 (file)
@@ -78,7 +78,7 @@ static const struct das16cs_board das16cs_boards[] = {
         },
 };
 
-#define n_boards (sizeof(das16cs_boards)/sizeof(das16cs_boards[0]))
+#define n_boards ARRAY_SIZE(das16cs_boards)
 #define thisboard ((const struct das16cs_board *)dev->board_ptr)
 
 struct das16cs_private {
index 74fcc1086c51a3716dfd04b104b4206502587ea8..9cab21eaaa186f23e8179f22e13b578dffa34f7f 100644 (file)
@@ -65,8 +65,7 @@ static struct comedi_driver driver_das08_cs = {
        .attach = das08_cs_attach,
        .detach = das08_common_detach,
        .board_name = &das08_cs_boards[0].name,
-       .num_names = sizeof(das08_cs_boards) /
-           sizeof(struct das08_board_struct),
+       .num_names = ARRAY_SIZE(das08_cs_boards),
        .offset = sizeof(struct das08_board_struct),
 };
 
index 7a49dfce67bec6a92a8a8767da244e8bfe760bff..b37ef37c2d2df454701b60c0662455ace55554bd 100644 (file)
@@ -103,7 +103,7 @@ static const struct ni6527_board ni6527_boards[] = {
         },
 };
 
-#define n_ni6527_boards (sizeof(ni6527_boards)/sizeof(ni6527_boards[0]))
+#define n_ni6527_boards ARRAY_SIZE(ni6527_boards)
 #define this_board ((const struct ni6527_board *)dev->board_ptr)
 
 static DEFINE_PCI_DEVICE_TABLE(ni6527_pci_table) = {
index 8079c41b73388699ef18aa92b7511e96764439c5..6b118c15b49e3d147d99491f2b7c86d62aa011a2 100644 (file)
@@ -246,7 +246,7 @@ static const struct ni_65xx_board ni_65xx_boards[] = {
         },
 };
 
-#define n_ni_65xx_boards (sizeof(ni_65xx_boards)/sizeof(ni_65xx_boards[0]))
+#define n_ni_65xx_boards ARRAY_SIZE(ni_65xx_boards)
 static inline const struct ni_65xx_board *board(struct comedi_device *dev)
 {
        return dev->board_ptr;
index 772c9372e4355718a19f88081e3e262fd15806ea..404d3c516ed18b0d8f282e13ebec8339e18d0757 100644 (file)
@@ -454,7 +454,7 @@ static inline const struct ni_660x_board *board(struct comedi_device *dev)
        return dev->board_ptr;
 }
 
-#define n_ni_660x_boards (sizeof(ni_660x_boards)/sizeof(ni_660x_boards[0]))
+#define n_ni_660x_boards ARRAY_SIZE(ni_660x_boards)
 
 static int ni_660x_attach(struct comedi_device *dev,
                          struct comedi_devconfig *it);
index 89ce5b873ffed6358d3cd318f4047a22821c405b..9b43547e80a1c9f3356c290de31b1ac78dc3ffd4 100644 (file)
@@ -111,7 +111,7 @@ struct ni_670x_private {
 };
 
 #define devpriv ((struct ni_670x_private *)dev->private)
-#define n_ni_670x_boards (sizeof(ni_670x_boards)/sizeof(ni_670x_boards[0]))
+#define n_ni_670x_boards ARRAY_SIZE(ni_670x_boards)
 
 static int ni_670x_attach(struct comedi_device *dev,
                          struct comedi_devconfig *it);
index cde30b77a15ff463f4a6ce3aef79951850da5ef4..901833d9b772b4bd83bcbdf9a4463dda4c98d604 100644 (file)
@@ -121,7 +121,7 @@ static const struct atmio16_board_t atmio16_boards[] = {
         },
 };
 
-#define n_atmio16_boards sizeof(atmio16_boards)/sizeof(atmio16_boards[0])
+#define n_atmio16_boards ARRAY_SIZE(atmio16_boards)
 
 #define boardtype ((const struct atmio16_board_t *)dev->board_ptr)
 
index 15148d1642d4dff49719b24dbb927ed30b34c47a..52b2eca9e73d37dec67cea2fbcbe92ab8f8556c3 100644 (file)
@@ -374,7 +374,7 @@ static const struct nidio_board nidio_boards[] = {
         },
 };
 
-#define n_nidio_boards (sizeof(nidio_boards)/sizeof(nidio_boards[0]))
+#define n_nidio_boards ARRAY_SIZE(nidio_boards)
 #define this_board ((const struct nidio_board *)dev->board_ptr)
 
 static DEFINE_PCI_DEVICE_TABLE(ni_pcidio_pci_table) = {
index 0dc639212adee62171323ccfff90f3b114008ace..19d87553d9061988799126879412f946cb89a3d1 100644 (file)
@@ -1208,7 +1208,7 @@ static const struct ni_board_struct ni_boards[] = {
         },
 };
 
-#define n_pcimio_boards ((sizeof(ni_boards)/sizeof(ni_boards[0])))
+#define n_pcimio_boards ARRAY_SIZE(ni_boards)
 
 static int pcimio_attach(struct comedi_device *dev,
                         struct comedi_devconfig *it);
index 7368bf3011c977611e03813c2b07329a2f08f953..acac6709081085adcca5bc98591695e129977a34 100644 (file)
@@ -69,7 +69,7 @@ static const struct pcmad_board_struct pcmad_boards[] = {
 };
 
 #define this_board ((const struct pcmad_board_struct *)(dev->board_ptr))
-#define n_pcmad_boards (sizeof(pcmad_boards)/sizeof(pcmad_boards[0]))
+#define n_pcmad_boards ARRAY_SIZE(pcmad_boards)
 
 struct pcmad_priv_struct {
        int differential;
index 729a0d8766e8df7ae6d5a8e398e9ff54b907ed5d..d23e588d0632214ffa95dd7866486a62dce6bb48 100644 (file)
@@ -102,7 +102,7 @@ static const struct boarddef_struct boards[] = {
         },
 };
 
-#define n_boards (sizeof(boards)/sizeof(boards[0]))
+#define n_boards ARRAY_SIZE(boards)
 #define this_board ((const struct boarddef_struct *)dev->board_ptr)
 
 static struct comedi_driver driver_poc = {
index 25d2c34325db27b6e1d5801c0e52d77bac115184..0572c3d0aefd4aa90d9e6d9eb33d7bfe3da5ea53 100644 (file)
@@ -328,7 +328,7 @@ tEplKernel EplDllkAddInstance(tEplDllkInitParam * pInitParam_p)
        // initialize and link pointers in instance structure to frame tables
        EplDllkInstance_g.m_pTxBuffer = aEplDllkTxBuffer_l;
        EplDllkInstance_g.m_uiMaxTxFrames =
-           sizeof(aEplDllkTxBuffer_l) / sizeof(tEdrvTxBuffer);
+           ARRAY_SIZE(aEplDllkTxBuffer_l);
 
        // initialize state
        EplDllkInstance_g.m_DllState = kEplDllGsInit;
index 9ccf079e67e1f82eb0fc5bdd37be441da7aef612..e48949d9ddf56570aaf41f81d97b8f76864b1d75 100644 (file)
@@ -196,7 +196,7 @@ void TgtDbgSignalTracePoint(u8 bTracePointNumber_p)
 {
 
        if (bTracePointNumber_p >=
-           (sizeof(aatmDbgTracePoint_l) / sizeof(aatmDbgTracePoint_l[0]))) {
+           ARRAY_SIZE(aatmDbgTracePoint_l)) {
                goto Exit;
        }
 
@@ -330,7 +330,7 @@ static int EplLinProcRead(char *pcBuffer_p,
                nSize += snprintf(pcBuffer_p + nSize, nBufferSize_p - nSize,
                                  "DbgTracePoints:\n");
                for (nNum = 0;
-                    nNum < (sizeof(aatmDbgTracePoint_l) / sizeof(atomic_t));
+                    nNum < ARRAY_SIZE(aatmDbgTracePoint_l);
                     nNum++) {
                        nSize +=
                            snprintf(pcBuffer_p + nSize, nBufferSize_p - nSize,
index 85a20d0002c0c9210df5e69c7988e1e140458a78..f188ecee502fe172c2faa257183294d18de65ccf 100644 (file)
@@ -682,7 +682,7 @@ static int line6_probe(struct usb_interface *interface, const struct usb_device_
                return -ENODEV;
 
        /* check vendor and product id */
-       for (devtype = sizeof(line6_id_table) / sizeof(line6_id_table[0]) - 1; devtype--;)
+       for (devtype = ARRAY_SIZE(line6_id_table) - 1; devtype--;)
                if ((le16_to_cpu(usbdev->descriptor.idVendor) == line6_id_table[devtype].idVendor) &&
                         (le16_to_cpu(usbdev->descriptor.idProduct) == line6_id_table[devtype].idProduct))
                        break;
index d93a8aee581b390351900f161d04033a2515da1c..75bd3b7351d20a466756814b51ec08db9597df05 100644 (file)
@@ -54,7 +54,7 @@ static me0600_version_t me0600_versions[] = {
        {0},
 };
 
-#define ME0600_DEVICE_VERSIONS (sizeof(me0600_versions) / sizeof(me0600_version_t) - 1)        /**< Returns the number of entries in #me0600_versions. */
+#define ME0600_DEVICE_VERSIONS (ARRAY_SIZE(me0600_versions) - 1)       /**< Returns the number of entries in #me0600_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me0600_versions.
index bd17f2521511fc13e442c02f3f31d089404bb137..53c05e83675dcd1d17207aa54f4e0dcff4131fa5 100644 (file)
@@ -53,7 +53,7 @@ static me0900_version_t me0900_versions[] = {
        {0},
 };
 
-#define ME0900_DEVICE_VERSIONS (sizeof(me0900_versions) / sizeof(me0900_version_t) - 1)        /**< Returns the number of entries in #me0900_versions. */
+#define ME0900_DEVICE_VERSIONS (ARRAY_SIZE(me0900_versions) - 1)       /**< Returns the number of entries in #me0900_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me0900_versions.
index 6215b250047daebbc4db0229de9ae3894cc912df..d20112d8da6016675176ea63e0b65e70aead25e5 100644 (file)
@@ -60,7 +60,7 @@ static me1400_version_t me1400_versions[] = {
        {0}
 };
 
-#define ME1400_DEVICE_VERSIONS (sizeof(me1400_versions) / sizeof(me1400_version_t) - 1)        /**< Returns the number of entries in #me1400_versions. */
+#define ME1400_DEVICE_VERSIONS (ARRAY_SIZE(me1400_versions) - 1)       /**< Returns the number of entries in #me1400_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me1400_versions.
index f7b231f73ac8d27c9562948b1603672b569ff742..c73aca11ae88847b18e76a2ec5a1bf32df145d24 100644 (file)
@@ -58,7 +58,7 @@ static me1600_version_t me1600_versions[] = {
 };
 
 /**< Returns the number of entries in #me1600_versions. */
-#define ME1600_DEVICE_VERSIONS (sizeof(me1600_versions) / sizeof(me1600_version_t) - 1)
+#define ME1600_DEVICE_VERSIONS (ARRAY_SIZE(me1600_versions) - 1)
 
 /**
  * @brief Returns the index of the device entry in #me1600_versions.
index fa812d4cc6dc06412bc0efb8940ba1946ff43a42..c755c574cda2e28f4a56d1bda498712b3788ae74 100644 (file)
@@ -80,7 +80,7 @@ static me4600_version_t me4600_versions[] = {
        {0},
 };
 
-#define ME4600_DEVICE_VERSIONS (sizeof(me4600_versions) / sizeof(me4600_version_t) - 1)        /**< Returns the number of entries in #me4600_versions. */
+#define ME4600_DEVICE_VERSIONS (ARRAY_SIZE(me4600_versions) - 1)       /**< Returns the number of entries in #me4600_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me4600_versions.
index 18cc7d1e14f19c39858a60db33415211b18719dc..eed4011521015121f579001641c04aeabad69614 100644 (file)
@@ -103,7 +103,7 @@ static me6000_version_t me6000_versions[] = {
        {0},
 };
 
-#define ME6000_DEVICE_VERSIONS (sizeof(me6000_versions) / sizeof(me6000_version_t) - 1)        /**< Returns the number of entries in #me6000_versions. */
+#define ME6000_DEVICE_VERSIONS (ARRAY_SIZE(me6000_versions) - 1)       /**< Returns the number of entries in #me6000_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me6000_versions.
index 44c42efb04e22ecc37dbb4d7cf7088ed679e7cb0..238dc062280196607e1f76cce8aec1a2486c7d27 100644 (file)
@@ -53,7 +53,7 @@ static me8100_version_t me8100_versions[] = {
        {0},
 };
 
-#define ME8100_DEVICE_VERSIONS (sizeof(me8100_versions) / sizeof(me8100_version_t) - 1)        /**< Returns the number of entries in #me8100_versions. */
+#define ME8100_DEVICE_VERSIONS (ARRAY_SIZE(me8100_versions) - 1)       /**< Returns the number of entries in #me8100_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me8100_versions.
index cbd2a01ddb411e36e76486933871cbd8ffc026bd..66f50842dac03f21c0e163defeb077ee6617808b 100644 (file)
@@ -53,7 +53,7 @@ static me8200_version_t me8200_versions[] = {
        {0},
 };
 
-#define ME8200_DEVICE_VERSIONS (sizeof(me8200_versions) / sizeof(me8200_version_t) - 1)        /**< Returns the number of entries in #me8200_versions. */
+#define ME8200_DEVICE_VERSIONS (ARRAY_SIZE(me8200_versions) - 1)       /**< Returns the number of entries in #me8200_versions. */
 
 /**
  * @brief Returns the index of the device entry in #me8200_versions.
index 3c3702cc72ebd0cf46342205b9908881222444d6..c0626e7b73a78a9c131dad90a5aa7270b248e38e 100644 (file)
@@ -50,7 +50,7 @@ static metempl_version_t metempl_versions[] = {
        {0},
 };
 
-#define METEMPL_DEVICE_VERSIONS (sizeof(metempl_versions) / sizeof(metempl_version_t) - 1) /**< Returns the number of entries in #metempl_versions. */
+#define METEMPL_DEVICE_VERSIONS (ARRAY_SIZE(metempl_versions) - 1) /**< Returns the number of entries in #metempl_versions. */
 
 /**
  * @brief Returns the index of the device entry in #metempl_versions.
index 081c022f0bc210e62adc1ad0732719003d0ded44..766892e31f52196ed2c914c47932c392c34f1d9e 100644 (file)
@@ -27,7 +27,7 @@
 u32 rtl8180_rates[] = {1000000,2000000,5500000,11000000,
        6000000,9000000,12000000,18000000,24000000,36000000,48000000,54000000};
 
-#define RATE_COUNT (sizeof(rtl8180_rates)/sizeof(rtl8180_rates[0]))
+#define RATE_COUNT ARRAY_SIZE(rtl8180_rates)
 
 static CHANNEL_LIST DefaultChannelPlan[] = {
 //     {{1,2,3,4,5,6,7,8,9,10,11,12,13,14},14},                        //Default channel plan
@@ -1578,9 +1578,9 @@ static struct iw_statistics *r8180_get_wireless_stats(struct net_device *dev)
 
 struct iw_handler_def  r8180_wx_handlers_def={
        .standard = r8180_wx_handlers,
-       .num_standard = sizeof(r8180_wx_handlers) / sizeof(iw_handler),
+       .num_standard = ARRAY_SIZE(r8180_wx_handlers),
        .private = r8180_private_handler,
-       .num_private = sizeof(r8180_private_handler) / sizeof(iw_handler),
+       .num_private = ARRAY_SIZE(r8180_private_handler),
        .num_private_args = sizeof(r8180_private_args) / sizeof(struct iw_priv_args),
        .get_wireless_stats = r8180_get_wireless_stats,
        .private_args = (struct iw_priv_args *)r8180_private_args,
index 3afd8b34f68cf2df1adccb95f69f2bf0cc2ccc81..145a3120c6bbf838476e8a30e1663b8f4afe5249 100644 (file)
@@ -87,7 +87,7 @@ static inline char *rtl819x_translate_scan(struct ieee80211_device *ieee,
         }
        /* Add the protocol name */
        iwe.cmd = SIOCGIWNAME;
-       for(i=0; i<(sizeof(ieee80211_modes)/sizeof(ieee80211_modes[0])); i++) {
+       for(i=0; i<ARRAY_SIZE(ieee80211_modes); i++) {
                if(network->mode&(1<<i)) {
                        sprintf(pname,ieee80211_modes[i].mode_string,ieee80211_modes[i].mode_size);
                        pname +=ieee80211_modes[i].mode_size;
index ad56faec596ee141d7e8b676ced98529ecab5d69..2208c9b1e726d90bb77a71258b2cda2204445ba8 100644 (file)
@@ -1221,9 +1221,9 @@ struct iw_statistics *r8192_get_wireless_stats(struct net_device *dev)
 
 struct iw_handler_def  r8192_wx_handlers_def={
        .standard = r8192_wx_handlers,
-       .num_standard = sizeof(r8192_wx_handlers) / sizeof(iw_handler),
+       .num_standard = ARRAY_SIZE(r8192_wx_handlers),
        .private = r8192_private_handler,
-       .num_private = sizeof(r8192_private_handler) / sizeof(iw_handler),
+       .num_private = ARRAY_SIZE(r8192_private_handler),
        .num_private_args = sizeof(r8192_private_args) / sizeof(struct iw_priv_args),
        .get_wireless_stats = r8192_get_wireless_stats,
        .private_args = (struct iw_priv_args *)r8192_private_args,
index b01a2822a8eeeb387897417891175a238c7b483c..e67a130f9e46eab61d3e1d571e1dca7b7a798eb2 100644 (file)
@@ -1306,7 +1306,7 @@ static void slic_mcast_init_crc32(void)
 
        static int p[] = { 0, 1, 2, 4, 5, 7, 8, 10, 11, 12, 16, 22, 23, 26 };
 
-       for (i = 0; i < sizeof(p) / sizeof(int); i++)
+       for (i = 0; i < ARRAY_SIZE(p); i++)
                e |= 1L << (31 - p[i]);
 
        for (i = 1; i < 256; i++) {
index 3a0dac962e4d51df19fdcd98ee415debfb110dd8..395e876c7dc9cf4cd08f2f558f6b3bfae73bb347 100644 (file)
@@ -3500,7 +3500,7 @@ static void sxg_mcast_init_crc32(void)
 
        static int p[] = { 0, 1, 2, 4, 5, 7, 8, 10, 11, 12, 16, 22, 23, 26 };
 
-       for (i = 0; i < sizeof(p) / sizeof(int); i++) {
+       for (i = 0; i < ARRAY_SIZE(p); i++) {
                e |= 1L << (31 - p[i]);
        }