kernel: Use NELEM() in a number of places.
authorSascha Wildner <saw@online.de>
Tue, 22 Jul 2014 16:35:34 +0000 (18:35 +0200)
committerSascha Wildner <saw@online.de>
Tue, 22 Jul 2014 16:35:34 +0000 (18:35 +0200)
sys/bus/u4b/input/atp.c
sys/bus/u4b/input/ukbd.c
sys/bus/u4b/serial/u3g.c
sys/bus/u4b/serial/uchcom.c
sys/bus/u4b/serial/umcs.c
sys/bus/u4b/serial/uplcom.c
sys/dev/netif/bwn/bwn/if_bwn.c
sys/dev/netif/mxge/if_mxge.c

index 0a32026..5d924bc 100644 (file)
@@ -1446,7 +1446,7 @@ atp_compute_smoothening_scale_ratio(atp_stroke *stroke, int *numerator,
                155, 155, 155, 156, 156, 156, 157, 157,
                157, 158, 158, 158, 159, 159, 159, 160
        };
-       const u_int N = sizeof(sqrt_table) / sizeof(sqrt_table[0]);
+       const u_int N = NELEM(sqrt_table);
 
        dxdt = stroke->components[X].delta_mickeys;
        dydt = stroke->components[Y].delta_mickeys;
index aa4c7ab..3f06e6d 100644 (file)
@@ -2108,7 +2108,7 @@ ukbd_key2scan(struct ukbd_softc *sc, int code, int shift, int up)
                0x5c,   /* Keyboard Intl' 6 (Keypad ,) (For PC-9821 layout) */
        };
 
-       if ((code >= 89) && (code < (int)(89 + (sizeof(scan) / sizeof(scan[0]))))) {
+       if ((code >= 89) && (code < (int)(89 + NELEM(scan)))) {
                code = scan[code - 89];
        }
        /* Pause/Break */
index 8e62e13..4ad1938 100644 (file)
@@ -604,7 +604,7 @@ u3g_sael_m460_init(struct usb_device *udev)
                return;
        }
 
-       for (n = 0; n != (sizeof(setup)/sizeof(setup[0])); n++) {
+       for (n = 0; n != NELEM(setup); n++) {
 
                memcpy(&req, setup[n], sizeof(req));
 
index 686f865..6ffa80c 100644 (file)
@@ -197,8 +197,6 @@ static const struct uchcom_divider_record dividers[] =
        {367, 1, 11719, {0, 0, 0}},
 };
 
-#define        NUM_DIVIDERS    (sizeof (dividers) / sizeof (dividers[0]))
-
 static const STRUCT_USB_HOST_ID uchcom_devs[] = {
        {USB_VPI(USB_VENDOR_WCH, USB_PRODUCT_WCH_CH341SER, 0)},
        {USB_VPI(USB_VENDOR_WCH2, USB_PRODUCT_WCH2_CH341SER, 0)},
@@ -545,7 +543,7 @@ uchcom_calc_divider_settings(struct uchcom_divider *dp, uint32_t rate)
        uint8_t i;
 
        /* find record */
-       for (i = 0; i != NUM_DIVIDERS; i++) {
+       for (i = 0; i != NELEM(dividers); i++) {
                if (dividers[i].dvr_high >= rate &&
                    dividers[i].dvr_low <= rate) {
                        rp = &dividers[i];
index 7c351cd..8ffc9fd 100644 (file)
@@ -1051,7 +1051,7 @@ umcs7840_set_baudrate(struct umcs7840_softc *sc, uint8_t portno, uint32_t rate)
 
 /* Maximum speeds for standard frequences, when PLL is not used */
 static const uint32_t umcs7840_baudrate_divisors[] = {0, 115200, 230400, 403200, 460800, 806400, 921600, 1572864, 3145728,};
-static const uint8_t umcs7840_baudrate_divisors_len = sizeof(umcs7840_baudrate_divisors) / sizeof(umcs7840_baudrate_divisors[0]);
+static const uint8_t umcs7840_baudrate_divisors_len = NELEM(umcs7840_baudrate_divisors);
 
 static usb_error_t
 umcs7840_calc_baudrate(uint32_t rate, uint16_t *divisor, uint8_t *clk)
index 0077a87..d539e4f 100644 (file)
@@ -611,8 +611,6 @@ static const int32_t uplcom_rates[] = {
        230400, 460800, 614400, 921600, 1228800
 };
 
-#define        N_UPLCOM_RATES  (sizeof(uplcom_rates)/sizeof(uplcom_rates[0]))
-
 static int
 uplcom_pre_param(struct ucom_softc *ucom, struct termios *t)
 {
@@ -632,7 +630,7 @@ uplcom_pre_param(struct ucom_softc *ucom, struct termios *t)
         *      it the same as the PL2303X.
         */
        if (sc->sc_chiptype != TYPE_PL2303HX) {
-               for (i = 0; i < N_UPLCOM_RATES; i++) {
+               for (i = 0; i < NELEM(uplcom_rates); i++) {
                        if (uplcom_rates[i] == t->c_ospeed)
                                return (0);
                }
index 80e383a..781a2f9 100644 (file)
@@ -902,7 +902,7 @@ bwn_probe(device_t dev)
 
        wlan_serialize_enter();
 
-       for (i = 0; i < sizeof(bwn_devs) / sizeof(bwn_devs[0]); i++) {
+       for (i = 0; i < NELEM(bwn_devs); i++) {
                if (siba_get_vendor(dev) == bwn_devs[i].sd_vendor &&
                    siba_get_device(dev) == bwn_devs[i].sd_device &&
                    siba_get_revid(dev) == bwn_devs[i].sd_rev) {
index c9167af..816aedb 100644 (file)
@@ -2540,15 +2540,13 @@ mxge_media_probe(mxge_softc_t *sc)
        if (sc->connector == MXGE_XFP) {
                /* -R is XFP */
                mxge_media_types = mxge_xfp_media_types;
-               mxge_media_type_entries = sizeof(mxge_xfp_media_types) /
-                   sizeof(mxge_xfp_media_types[0]);
+               mxge_media_type_entries = NELEM(mxge_xfp_media_types);
                byte = MXGE_XFP_COMPLIANCE_BYTE;
                cage_type = "XFP";
        } else  if (sc->connector == MXGE_SFP) {
                /* -S or -2S is SFP+ */
                mxge_media_types = mxge_sfp_media_types;
-               mxge_media_type_entries = sizeof(mxge_sfp_media_types) /
-                   sizeof(mxge_sfp_media_types[0]);
+               mxge_media_type_entries = NELEM(mxge_sfp_media_types);
                cage_type = "SFP+";
                byte = 3;
        } else {