kernel: Use nitems() from <sys/param.h> instead of custom define
authorAaron LI <aly@aaronly.me>
Tue, 20 Mar 2018 07:48:55 +0000 (15:48 +0800)
committerAaron LI <aly@aaronly.me>
Thu, 26 Jul 2018 02:01:27 +0000 (10:01 +0800)
Since <sys/param.h> is already included in these files.

sys/bus/u4b/wlan/if_uath.c
sys/bus/u4b/wlan/if_ural.c
sys/bus/u4b/wlan/if_urtw.c
sys/bus/u4b/wlan/if_zyd.c
sys/contrib/dev/ath/ath_hal/ar9300/ar9300_gpio.c
sys/contrib/dev/ath/ath_hal/ar9300/ar9300_misc.c
sys/dev/crypto/safe/safe.c
sys/dev/netif/bwn/siba/siba_bwn.c
sys/dev/netif/bwn/siba/siba_core.c

index c67d554..9e22484 100644 (file)
@@ -613,7 +613,6 @@ uath_dump_cmd(const uint8_t *buf, int len, char prefix)
 static const char *
 uath_codename(int code)
 {
-#define        N(a)    (sizeof(a)/sizeof(a[0]))
        static const char *names[] = {
            "0x00",
            "HOST_AVAILABLE",
@@ -676,13 +675,12 @@ uath_codename(int code)
        };
        static char buf[8];
 
-       if (code < N(names))
+       if (code < nitems(names))
                return names[code];
        if (code == WDCMSG_SET_DEFAULT_KEY)
                return "SET_DEFAULT_KEY";
        snprintf(buf, sizeof(buf), "0x%02x", code);
        return buf;
-#undef N
 }
 #endif
 
index d208a3e..a4bc7ea 100644 (file)
@@ -1955,7 +1955,6 @@ ural_read_eeprom(struct ural_softc *sc)
 static int
 ural_bbp_init(struct ural_softc *sc)
 {
-#define N(a)   (sizeof (a) / sizeof ((a)[0]))
        int i, ntries;
 
        /* wait for BBP to be ready */
@@ -1971,7 +1970,7 @@ ural_bbp_init(struct ural_softc *sc)
        }
 
        /* initialize BBP registers to default values */
-       for (i = 0; i < N(ural_def_bbp); i++)
+       for (i = 0; i < nitems(ural_def_bbp); i++)
                ural_bbp_write(sc, ural_def_bbp[i].reg, ural_def_bbp[i].val);
 
 #if 0
@@ -1984,7 +1983,6 @@ ural_bbp_init(struct ural_softc *sc)
 #endif
 
        return 0;
-#undef N
 }
 
 static void
@@ -2039,7 +2037,6 @@ ural_set_rxantenna(struct ural_softc *sc, int antenna)
 static void
 ural_init_locked(struct ural_softc *sc)
 {
-#define N(a)   (sizeof (a) / sizeof ((a)[0]))
        struct ifnet *ifp = sc->sc_ifp;
        struct ieee80211com *ic = ifp->if_l2com;
        uint16_t tmp;
@@ -2053,7 +2050,7 @@ ural_init_locked(struct ural_softc *sc)
        ural_stop(sc);
 
        /* initialize MAC registers to default values */
-       for (i = 0; i < N(ural_def_mac); i++)
+       for (i = 0; i < nitems(ural_def_mac); i++)
                ural_write(sc, ural_def_mac[i].reg, ural_def_mac[i].val);
 
        /* wait for BBP and RF to wake up (this can take a long time!) */
@@ -2113,7 +2110,6 @@ ural_init_locked(struct ural_softc *sc)
        return;
 
 fail:  ural_stop(sc);
-#undef N
 }
 
 static void
index 4e6ecba..a973103 100644 (file)
@@ -1092,7 +1092,6 @@ urtw_init(void *arg)
 static usb_error_t
 urtw_adapter_start_b(struct urtw_softc *sc)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        uint8_t data8;
        usb_error_t error;
 
@@ -1175,7 +1174,6 @@ urtw_adapter_start_b(struct urtw_softc *sc)
        urtw_write16_m(sc, 0x1ec, 0x800);       /* RX MAX SIZE */
 fail:
        return (error);
-#undef N
 }
 
 static usb_error_t
@@ -1941,31 +1939,27 @@ fail:
 static uint16_t
 urtw_rate2rtl(int rate)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int i;
 
-       for (i = 0; i < N(urtw_ratetable); i++) {
+       for (i = 0; i < nitems(urtw_ratetable); i++) {
                if (rate == urtw_ratetable[i].reg)
                        return urtw_ratetable[i].val;
        }
 
        return (3);
-#undef N
 }
 
 static uint16_t
 urtw_rtl2rate(int rate)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int i;
 
-       for (i = 0; i < N(urtw_ratetable); i++) {
+       for (i = 0; i < nitems(urtw_ratetable); i++) {
                if (rate == urtw_ratetable[i].val)
                        return urtw_ratetable[i].reg;
        }
 
        return (0);
-#undef N
 }
 
 static usb_error_t
@@ -2481,7 +2475,6 @@ fail:
 static usb_error_t
 urtw_8225_rf_init(struct urtw_softc *sc)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int i;
        uint16_t data;
        usb_error_t error;
@@ -2512,7 +2505,7 @@ urtw_8225_rf_init(struct urtw_softc *sc)
                goto fail;
        usb_pause_mtx(&sc->sc_mtx, 1000);
 
-       for (i = 0; i < N(urtw_8225_rf_part1); i++) {
+       for (i = 0; i < nitems(urtw_8225_rf_part1); i++) {
                urtw_8225_write(sc, urtw_8225_rf_part1[i].reg,
                    urtw_8225_rf_part1[i].val);
                usb_pause_mtx(&sc->sc_mtx, 1);
@@ -2544,7 +2537,7 @@ urtw_8225_rf_init(struct urtw_softc *sc)
                usb_pause_mtx(&sc->sc_mtx, 1);
        }
 
-       for (i = 0; i < N(urtw_8225_rf_part2); i++) {
+       for (i = 0; i < nitems(urtw_8225_rf_part2); i++) {
                urtw_8187_write_phy_ofdm(sc, urtw_8225_rf_part2[i].reg,
                    urtw_8225_rf_part2[i].val);
                usb_pause_mtx(&sc->sc_mtx, 1);
@@ -2554,7 +2547,7 @@ urtw_8225_rf_init(struct urtw_softc *sc)
        if (error)
                goto fail;
 
-       for (i = 0; i < N(urtw_8225_rf_part3); i++) {
+       for (i = 0; i < nitems(urtw_8225_rf_part3); i++) {
                urtw_8187_write_phy_cck(sc, urtw_8225_rf_part3[i].reg,
                    urtw_8225_rf_part3[i].val);
                usb_pause_mtx(&sc->sc_mtx, 1);
@@ -2580,7 +2573,6 @@ urtw_8225_rf_init(struct urtw_softc *sc)
        error = urtw_8225_rf_set_chan(sc, 1);
 fail:
        return (error);
-#undef N
 }
 
 static usb_error_t
@@ -2878,7 +2870,6 @@ fail:
 static usb_error_t
 urtw_8225v2_rf_init(struct urtw_softc *sc)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int i;
        uint16_t data;
        uint32_t data32;
@@ -2911,7 +2902,7 @@ urtw_8225v2_rf_init(struct urtw_softc *sc)
 
        usb_pause_mtx(&sc->sc_mtx, 500);
 
-       for (i = 0; i < N(urtw_8225v2_rf_part1); i++) {
+       for (i = 0; i < nitems(urtw_8225v2_rf_part1); i++) {
                urtw_8225_write(sc, urtw_8225v2_rf_part1[i].reg,
                    urtw_8225v2_rf_part1[i].val);
        }
@@ -2966,7 +2957,7 @@ urtw_8225v2_rf_init(struct urtw_softc *sc)
                urtw_8187_write_phy_ofdm(sc, 0xa, (uint8_t)i + 0x80);
        }
 
-       for (i = 0; i < N(urtw_8225v2_rf_part2); i++) {
+       for (i = 0; i < nitems(urtw_8225v2_rf_part2); i++) {
                urtw_8187_write_phy_ofdm(sc, urtw_8225v2_rf_part2[i].reg,
                    urtw_8225v2_rf_part2[i].val);
        }
@@ -2975,7 +2966,7 @@ urtw_8225v2_rf_init(struct urtw_softc *sc)
        if (error)
                goto fail;
 
-       for (i = 0; i < N(urtw_8225v2_rf_part3); i++) {
+       for (i = 0; i < nitems(urtw_8225v2_rf_part3); i++) {
                urtw_8187_write_phy_cck(sc, urtw_8225v2_rf_part3[i].reg,
                    urtw_8225v2_rf_part3[i].val);
        }
@@ -2998,7 +2989,6 @@ urtw_8225v2_rf_init(struct urtw_softc *sc)
        error = urtw_8225_rf_set_chan(sc, 1);
 fail:
        return (error);
-#undef N
 }
 
 static usb_error_t
@@ -3212,7 +3202,6 @@ fail:
 static usb_error_t
 urtw_8225v2b_rf_init(struct urtw_softc *sc)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int i;
        uint8_t data8;
        usb_error_t error;
@@ -3272,7 +3261,7 @@ urtw_8225v2b_rf_init(struct urtw_softc *sc)
        /*
         * MAC configuration
         */
-       for (i = 0; i < N(urtw_8225v2b_rf_part1); i++)
+       for (i = 0; i < nitems(urtw_8225v2b_rf_part1); i++)
                urtw_write8_m(sc, urtw_8225v2b_rf_part1[i].reg,
                    urtw_8225v2b_rf_part1[i].val);
        urtw_write16_m(sc, URTW_TID_AC_MAP, 0xfa50);
@@ -3305,7 +3294,7 @@ urtw_8225v2b_rf_init(struct urtw_softc *sc)
        urtw_write16_m(sc, URTW_RF_PINS_ENABLE, 0x1fff);
        usb_pause_mtx(&sc->sc_mtx, 1100);
 
-       for (i = 0; i < N(urtw_8225v2b_rf_part0); i++) {
+       for (i = 0; i < nitems(urtw_8225v2b_rf_part0); i++) {
                urtw_8225_write(sc, urtw_8225v2b_rf_part0[i].reg,
                    urtw_8225v2b_rf_part0[i].val);
                usb_pause_mtx(&sc->sc_mtx, 1);
@@ -3351,7 +3340,7 @@ urtw_8225v2b_rf_init(struct urtw_softc *sc)
        }
        urtw_8187_write_phy_ofdm(sc, 0x80, 0x10);
 
-       for (i = 0; i < N(urtw_8225v2b_rf_part2); i++)
+       for (i = 0; i < nitems(urtw_8225v2b_rf_part2); i++)
                urtw_8187_write_phy_ofdm(sc, i, urtw_8225v2b_rf_part2[i].val);
 
        urtw_write32_m(sc, URTW_8187B_AC_VO, (7 << 12) | (3 << 8) | 0x1c);
@@ -3366,7 +3355,6 @@ urtw_8225v2b_rf_init(struct urtw_softc *sc)
 
 fail:
        return (error);
-#undef N
 }
 
 static usb_error_t
index 54aa3c5..edf2217 100644 (file)
@@ -933,25 +933,23 @@ fail:
 static int
 zyd_rfmd_init(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_RFMD_PHY;
        static const uint32_t rfini[] = ZYD_RFMD_RF;
        int i, error;
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++) {
+       for (i = 0; i < nitems(phyini); i++) {
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
        }
 
        /* init RFMD radio */
-       for (i = 0; i < N(rfini); i++) {
+       for (i = 0; i < nitems(rfini); i++) {
                if ((error = zyd_rfwrite(sc, rfini[i])) != 0)
                        return (error);
        }
 fail:
        return (error);
-#undef N
 }
 
 static int
@@ -992,7 +990,6 @@ fail:
 static int
 zyd_al2230_init(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_AL2230_PHY;
        static const struct zyd_phy_pair phy2230s[] = ZYD_AL2230S_PHY_INIT;
@@ -1006,16 +1003,16 @@ zyd_al2230_init(struct zyd_rf *rf)
        int i, error;
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        if (sc->sc_rfrev == ZYD_RF_AL2230S || sc->sc_al2230s != 0) {
-               for (i = 0; i < N(phy2230s); i++)
+               for (i = 0; i < nitems(phy2230s); i++)
                        zyd_write16_m(sc, phy2230s[i].reg, phy2230s[i].val);
        }
 
        /* init AL2230 radio */
-       for (i = 0; i < N(rfini1); i++) {
+       for (i = 0; i < nitems(rfini1); i++) {
                error = zyd_rfwrite(sc, rfini1[i]);
                if (error != 0)
                        goto fail;
@@ -1028,34 +1025,32 @@ zyd_al2230_init(struct zyd_rf *rf)
        if (error != 0)
                goto fail;
 
-       for (i = 0; i < N(rfini2); i++) {
+       for (i = 0; i < nitems(rfini2); i++) {
                error = zyd_rfwrite(sc, rfini2[i]);
                if (error != 0)
                        goto fail;
        }
 
-       for (i = 0; i < N(phypll); i++)
+       for (i = 0; i < nitems(phypll); i++)
                zyd_write16_m(sc, phypll[i].reg, phypll[i].val);
 
-       for (i = 0; i < N(rfini3); i++) {
+       for (i = 0; i < nitems(rfini3); i++) {
                error = zyd_rfwrite(sc, rfini3[i]);
                if (error != 0)
                        goto fail;
        }
 fail:
        return (error);
-#undef N
 }
 
 static int
 zyd_al2230_fini(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error, i;
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phy[] = ZYD_AL2230_PHY_FINI_PART1;
 
-       for (i = 0; i < N(phy); i++)
+       for (i = 0; i < nitems(phy); i++)
                zyd_write16_m(sc, phy[i].reg, phy[i].val);
 
        if (sc->sc_newphy != 0)
@@ -1064,13 +1059,11 @@ zyd_al2230_fini(struct zyd_rf *rf)
        zyd_write16_m(sc, ZYD_CR203, 0x6);
 fail:
        return (error);
-#undef N
 }
 
 static int
 zyd_al2230_init_b(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phy1[] = ZYD_AL2230_PHY_PART1;
        static const struct zyd_phy_pair phy2[] = ZYD_AL2230_PHY_PART2;
@@ -1083,15 +1076,15 @@ zyd_al2230_init_b(struct zyd_rf *rf)
        static const uint32_t zyd_al2230_chtable[][3] = ZYD_AL2230_CHANTABLE;
        int i, error;
 
-       for (i = 0; i < N(phy1); i++)
+       for (i = 0; i < nitems(phy1); i++)
                zyd_write16_m(sc, phy1[i].reg, phy1[i].val);
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        if (sc->sc_rfrev == ZYD_RF_AL2230S || sc->sc_al2230s != 0) {
-               for (i = 0; i < N(phy2230s); i++)
+               for (i = 0; i < nitems(phy2230s); i++)
                        zyd_write16_m(sc, phy2230s[i].reg, phy2230s[i].val);
        }
 
@@ -1101,7 +1094,7 @@ zyd_al2230_init_b(struct zyd_rf *rf)
                        return (error);
        }
 
-       for (i = 0; i < N(rfini_part1); i++) {
+       for (i = 0; i < nitems(rfini_part1); i++) {
                error = zyd_rfwrite_cr(sc, rfini_part1[i]);
                if (error != 0)
                        return (error);
@@ -1114,28 +1107,27 @@ zyd_al2230_init_b(struct zyd_rf *rf)
        if (error != 0)
                goto fail;
 
-       for (i = 0; i < N(rfini_part2); i++) {
+       for (i = 0; i < nitems(rfini_part2); i++) {
                error = zyd_rfwrite_cr(sc, rfini_part2[i]);
                if (error != 0)
                        return (error);
        }
 
-       for (i = 0; i < N(phy2); i++)
+       for (i = 0; i < nitems(phy2); i++)
                zyd_write16_m(sc, phy2[i].reg, phy2[i].val);
 
-       for (i = 0; i < N(rfini_part3); i++) {
+       for (i = 0; i < nitems(rfini_part3); i++) {
                error = zyd_rfwrite_cr(sc, rfini_part3[i]);
                if (error != 0)
                        return (error);
        }
 
-       for (i = 0; i < N(phy3); i++)
+       for (i = 0; i < nitems(phy3); i++)
                zyd_write16_m(sc, phy3[i].reg, phy3[i].val);
 
        error = zyd_al2230_fini(rf);
 fail:
        return (error);
-#undef N
 }
 
 static int
@@ -1153,7 +1145,6 @@ fail:
 static int
 zyd_al2230_set_channel(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error, i;
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phy1[] = {
@@ -1173,17 +1164,15 @@ zyd_al2230_set_channel(struct zyd_rf *rf, uint8_t chan)
        if (error != 0)
                goto fail;
 
-       for (i = 0; i < N(phy1); i++)
+       for (i = 0; i < nitems(phy1); i++)
                zyd_write16_m(sc, phy1[i].reg, phy1[i].val);
 fail:
        return (error);
-#undef N
 }
 
 static int
 zyd_al2230_set_channel_b(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error, i;
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phy1[] = ZYD_AL2230_PHY_PART1;
@@ -1191,7 +1180,7 @@ zyd_al2230_set_channel_b(struct zyd_rf *rf, uint8_t chan)
                uint32_t        r1, r2, r3;
        } rfprog[] = ZYD_AL2230_CHANTABLE_B;
 
-       for (i = 0; i < N(phy1); i++)
+       for (i = 0; i < nitems(phy1); i++)
                zyd_write16_m(sc, phy1[i].reg, phy1[i].val);
 
        error = zyd_rfwrite_cr(sc, rfprog[chan - 1].r1);
@@ -1206,7 +1195,6 @@ zyd_al2230_set_channel_b(struct zyd_rf *rf, uint8_t chan)
        error = zyd_al2230_fini(rf);
 fail:
        return (error);
-#undef N
 }
 
 #define        ZYD_AL2230_PHY_BANDEDGE6                                        \
@@ -1218,7 +1206,6 @@ fail:
 static int
 zyd_al2230_bandedge6(struct zyd_rf *rf, struct ieee80211_channel *c)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error = 0, i;
        struct zyd_softc *sc = rf->rf_sc;
        struct ifnet *ifp = sc->sc_ifp;
@@ -1229,11 +1216,10 @@ zyd_al2230_bandedge6(struct zyd_rf *rf, struct ieee80211_channel *c)
        if (chan == 1 || chan == 11)
                r[0].val = 0x12;
 
-       for (i = 0; i < N(r); i++)
+       for (i = 0; i < nitems(r); i++)
                zyd_write16_m(sc, r[i].reg, r[i].val);
 fail:
        return (error);
-#undef N
 }
 
 /*
@@ -1242,7 +1228,6 @@ fail:
 static int
 zyd_al7230B_init(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini_1[] = ZYD_AL7230B_PHY_1;
        static const struct zyd_phy_pair phyini_2[] = ZYD_AL7230B_PHY_2;
@@ -1254,29 +1239,28 @@ zyd_al7230B_init(struct zyd_rf *rf)
        /* for AL7230B, PHY and RF need to be initialized in "phases" */
 
        /* init RF-dependent PHY registers, part one */
-       for (i = 0; i < N(phyini_1); i++)
+       for (i = 0; i < nitems(phyini_1); i++)
                zyd_write16_m(sc, phyini_1[i].reg, phyini_1[i].val);
 
        /* init AL7230B radio, part one */
-       for (i = 0; i < N(rfini_1); i++) {
+       for (i = 0; i < nitems(rfini_1); i++) {
                if ((error = zyd_rfwrite(sc, rfini_1[i])) != 0)
                        return (error);
        }
        /* init RF-dependent PHY registers, part two */
-       for (i = 0; i < N(phyini_2); i++)
+       for (i = 0; i < nitems(phyini_2); i++)
                zyd_write16_m(sc, phyini_2[i].reg, phyini_2[i].val);
 
        /* init AL7230B radio, part two */
-       for (i = 0; i < N(rfini_2); i++) {
+       for (i = 0; i < nitems(rfini_2); i++) {
                if ((error = zyd_rfwrite(sc, rfini_2[i])) != 0)
                        return (error);
        }
        /* init RF-dependent PHY registers, part three */
-       for (i = 0; i < N(phyini_3); i++)
+       for (i = 0; i < nitems(phyini_3); i++)
                zyd_write16_m(sc, phyini_3[i].reg, phyini_3[i].val);
 fail:
        return (error);
-#undef N
 }
 
 static int
@@ -1294,7 +1278,6 @@ fail:
 static int
 zyd_al7230B_set_channel(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct {
                uint32_t        r1, r2;
@@ -1305,7 +1288,7 @@ zyd_al7230B_set_channel(struct zyd_rf *rf, uint8_t chan)
        zyd_write16_m(sc, ZYD_CR240, 0x57);
        zyd_write16_m(sc, ZYD_CR251, 0x2f);
 
-       for (i = 0; i < N(rfsc); i++) {
+       for (i = 0; i < nitems(rfsc); i++) {
                if ((error = zyd_rfwrite(sc, rfsc[i])) != 0)
                        return (error);
        }
@@ -1331,7 +1314,6 @@ zyd_al7230B_set_channel(struct zyd_rf *rf, uint8_t chan)
        zyd_write16_m(sc, ZYD_CR240, 0x08);
 fail:
        return (error);
-#undef N
 }
 
 /*
@@ -1340,7 +1322,6 @@ fail:
 static int
 zyd_al2210_init(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_AL2210_PHY;
        static const uint32_t rfini[] = ZYD_AL2210_RF;
@@ -1350,11 +1331,11 @@ zyd_al2210_init(struct zyd_rf *rf)
        zyd_write32_m(sc, ZYD_CR18, 2);
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        /* init AL2210 radio */
-       for (i = 0; i < N(rfini); i++) {
+       for (i = 0; i < nitems(rfini); i++) {
                if ((error = zyd_rfwrite(sc, rfini[i])) != 0)
                        return (error);
        }
@@ -1368,7 +1349,6 @@ zyd_al2210_init(struct zyd_rf *rf)
        zyd_write32_m(sc, ZYD_CR18, 3);
 fail:
        return (error);
-#undef N
 }
 
 static int
@@ -1413,7 +1393,6 @@ static int
 zyd_gct_init(struct zyd_rf *rf)
 {
 #define        ZYD_GCT_INTR_REG        0x85c1
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_GCT_PHY;
        static const uint32_t rfini[] = ZYD_GCT_RF;
@@ -1422,11 +1401,11 @@ zyd_gct_init(struct zyd_rf *rf)
        uint16_t data;
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        /* init cgt radio */
-       for (i = 0; i < N(rfini); i++) {
+       for (i = 0; i < nitems(rfini); i++) {
                if ((error = zyd_rfwrite(sc, rfini[i])) != 0)
                        return (error);
        }
@@ -1435,7 +1414,7 @@ zyd_gct_init(struct zyd_rf *rf)
        if (error != 0)
                return (error);
 
-       for (i = 0; i < N(vco) - 1; i++) {
+       for (i = 0; i < nitems(vco) - 1; i++) {
                error = zyd_gct_set_channel_synth(rf, 1, 0);
                if (error != 0)
                        goto fail;
@@ -1462,26 +1441,23 @@ zyd_gct_init(struct zyd_rf *rf)
        zyd_write16_m(sc, ZYD_CR203, 0x6);
 fail:
        return (error);
-#undef N
 #undef ZYD_GCT_INTR_REG
 }
 
 static int
 zyd_gct_mode(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const uint32_t mode[] = {
                0x25f98, 0x25f9a, 0x25f94, 0x27fd4
        };
        int i, error;
 
-       for (i = 0; i < N(mode); i++) {
+       for (i = 0; i < nitems(mode); i++) {
                if ((error = zyd_rfwrite(sc, mode[i])) != 0)
                        break;
        }
        return (error);
-#undef N
 }
 
 static int
@@ -1511,7 +1487,6 @@ zyd_gct_write(struct zyd_rf *rf, uint16_t value)
 static int
 zyd_gct_switch_radio(struct zyd_rf *rf, int on)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error;
        struct zyd_softc *sc = rf->rf_sc;
 
@@ -1529,7 +1504,6 @@ fail:
 static int
 zyd_gct_set_channel(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        int error, i;
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair cmd[] = {
@@ -1548,7 +1522,7 @@ zyd_gct_set_channel(struct zyd_rf *rf, uint8_t chan)
        error = zyd_gct_mode(rf);
        if (error != 0)
                return (error);
-       for (i = 0; i < N(cmd); i++)
+       for (i = 0; i < nitems(cmd); i++)
                zyd_write16_m(sc, cmd[i].reg, cmd[i].val);
        error = zyd_gct_txgain(rf, chan);
        if (error != 0)
@@ -1556,25 +1530,22 @@ zyd_gct_set_channel(struct zyd_rf *rf, uint8_t chan)
        zyd_write16_m(sc, ZYD_CR203, 0x6);
 fail:
        return (error);
-#undef N
 }
 
 static int
 zyd_gct_txgain(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static uint32_t txgain[] = ZYD_GCT_TXGAIN;
        uint8_t idx = sc->sc_pwrint[chan - 1];
 
-       if (idx >= N(txgain)) {
+       if (idx >= nitems(txgain)) {
                device_printf(sc->sc_dev, "could not set TX gain (%d %#x)\n",
                    chan, idx);
                return 0;
        }
 
        return zyd_rfwrite(sc, 0x700000 | txgain[idx]);
-#undef N
 }
 
 /*
@@ -1583,7 +1554,6 @@ zyd_gct_txgain(struct zyd_rf *rf, uint8_t chan)
 static int
 zyd_maxim2_init(struct zyd_rf *rf)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_MAXIM2_PHY;
        static const uint32_t rfini[] = ZYD_MAXIM2_RF;
@@ -1591,14 +1561,14 @@ zyd_maxim2_init(struct zyd_rf *rf)
        int i, error;
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        zyd_read16_m(sc, ZYD_CR203, &tmp);
        zyd_write16_m(sc, ZYD_CR203, tmp & ~(1 << 4));
 
        /* init maxim2 radio */
-       for (i = 0; i < N(rfini); i++) {
+       for (i = 0; i < nitems(rfini); i++) {
                if ((error = zyd_rfwrite(sc, rfini[i])) != 0)
                        return (error);
        }
@@ -1606,7 +1576,6 @@ zyd_maxim2_init(struct zyd_rf *rf)
        zyd_write16_m(sc, ZYD_CR203, tmp | (1 << 4));
 fail:
        return (error);
-#undef N
 }
 
 static int
@@ -1620,7 +1589,6 @@ zyd_maxim2_switch_radio(struct zyd_rf *rf, int on)
 static int
 zyd_maxim2_set_channel(struct zyd_rf *rf, uint8_t chan)
 {
-#define N(a)   (sizeof(a) / sizeof((a)[0]))
        struct zyd_softc *sc = rf->rf_sc;
        static const struct zyd_phy_pair phyini[] = ZYD_MAXIM2_PHY;
        static const uint32_t rfini[] = ZYD_MAXIM2_RF;
@@ -1636,7 +1604,7 @@ zyd_maxim2_set_channel(struct zyd_rf *rf, uint8_t chan)
         */
 
        /* init RF-dependent PHY registers */
-       for (i = 0; i < N(phyini); i++)
+       for (i = 0; i < nitems(phyini); i++)
                zyd_write16_m(sc, phyini[i].reg, phyini[i].val);
 
        zyd_read16_m(sc, ZYD_CR203, &tmp);
@@ -1651,7 +1619,7 @@ zyd_maxim2_set_channel(struct zyd_rf *rf, uint8_t chan)
                goto fail;
 
        /* init maxim2 radio - skipping the two first values */
-       for (i = 2; i < N(rfini); i++) {
+       for (i = 2; i < nitems(rfini); i++) {
                if ((error = zyd_rfwrite(sc, rfini[i])) != 0)
                        return (error);
        }
@@ -1659,7 +1627,6 @@ zyd_maxim2_set_channel(struct zyd_rf *rf, uint8_t chan)
        zyd_write16_m(sc, ZYD_CR203, tmp | (1 << 4));
 fail:
        return (error);
-#undef N
 }
 
 static int
index 88c9920..ed84c8b 100644 (file)
@@ -242,7 +242,6 @@ ar9300_gpio_cfg_output_led_off(
     u_int32_t gpio,
     HAL_GPIO_MUX_TYPE halSignalType)
 {
-#define N(a)    (sizeof(a) / sizeof(a[0]))
     u_int32_t    ah_signal_type;
     u_int32_t    gpio_shift;
     u_int8_t    smart_ant = 0;
@@ -334,7 +333,6 @@ ar9300_gpio_cfg_output_led_off(
                (AR_GPIO_OE_OUT_DRV << gpio_shift));
 
     return AH_TRUE;
-#undef N
 }
 
 /*
index 6972200..a11963b 100644 (file)
@@ -1036,11 +1036,9 @@ ar9300_set_capability(struct ath_hal *ah, HAL_CAPABILITY_TYPE type,
                     HAL_ANI_FIRSTEP_LEVEL,
                     HAL_ANI_SPUR_IMMUNITY_LEVEL,
             };
-#define N(a)    (sizeof(a) / sizeof(a[0]))
-            return capability < N(cmds) ?
+            return capability < nitems(cmds) ?
                     ar9300_ani_control(ah, cmds[capability], setting) :
                     AH_FALSE;
-#undef N
     }
 
     case HAL_CAP_RXBUFSIZE:         /* set MAC receive buffer size */
@@ -2253,7 +2251,6 @@ ar9300_detect_mac_hang(struct ath_hal *ah)
 HAL_BOOL
 ar9300_detect_bb_hang(struct ath_hal *ah)
 {
-#define N(a) (sizeof(a) / sizeof(a[0]))
     struct ath_hal_9300 *ahp = AH9300(ah);
     u_int32_t hang_sig = 0;
     int i = 0;
@@ -2288,7 +2285,7 @@ ar9300_detect_bb_hang(struct ath_hal *ah)
         }
     }
 
-    for (i = 0; i < N(hang_list); i++) {
+    for (i = 0; i < nitems(hang_list); i++) {
         if ((hang_sig & hang_list[i].hang_mask) == hang_list[i].hang_val) {
             ahp->ah_hang[hang_list[i].hang_offset]++;
             HALDEBUG(ah, HAL_DEBUG_DFS, "%s sig1count=%d sig2count=%d "
@@ -2304,7 +2301,6 @@ ar9300_detect_bb_hang(struct ath_hal *ah)
 
     return AH_FALSE;
 
-#undef N
 } /* end - ar9300_detect_bb_hang () */
 
 #undef NUM_STATUS_READS
index f4f8705..ad5c72f 100644 (file)
@@ -634,7 +634,6 @@ safe_feed(struct safe_softc *sc, struct safe_ringentry *re)
        WRITE_REG(sc, SAFE_HI_RD_DESCR, 0);
 }
 
-#define        N(a)    (sizeof(a) / sizeof (a[0]))
 static void
 safe_setup_enckey(struct safe_session *ses, caddr_t key)
 {
@@ -643,7 +642,7 @@ safe_setup_enckey(struct safe_session *ses, caddr_t key)
        bcopy(key, ses->ses_key, ses->ses_klen / 8);
 
        /* PE is little-endian, insure proper byte order */
-       for (i = 0; i < N(ses->ses_key); i++)
+       for (i = 0; i < nitems(ses->ses_key); i++)
                ses->ses_key[i] = htole32(ses->ses_key[i]);
 }
 
@@ -691,12 +690,11 @@ safe_setup_mackey(struct safe_session *ses, int algo, caddr_t key, int klen)
                key[i] ^= HMAC_OPAD_VAL;
 
        /* PE is little-endian, insure proper byte order */
-       for (i = 0; i < N(ses->ses_hminner); i++) {
+       for (i = 0; i < nitems(ses->ses_hminner); i++) {
                ses->ses_hminner[i] = htole32(ses->ses_hminner[i]);
                ses->ses_hmouter[i] = htole32(ses->ses_hmouter[i]);
        }
 }
-#undef N
 
 /*
  * Allocate a new 'session' and return an encoded session id.  'sidp'
index 109c167..7210fa2 100644 (file)
@@ -88,7 +88,6 @@ struct siba_bwn_softc {
 
 #define        BS_BAR                          0x10
 #define        PCI_VENDOR_BROADCOM             0x14e4
-#define        N(a)                            (sizeof(a) / sizeof(a[0]))
 
 static const struct siba_dev {
        uint16_t        vid;
@@ -129,7 +128,7 @@ siba_bwn_probe(device_t dev)
        did = pci_get_device(dev);
        vid = pci_get_vendor(dev);
 
-       for (i = 0; i < N(siba_devices); i++) {
+       for (i = 0; i < nitems(siba_devices); i++) {
                if (siba_devices[i].did == did && siba_devices[i].vid == vid) {
                        device_set_desc(dev, siba_devices[i].desc);
                        return (BUS_PROBE_DEFAULT);
index 95014b5..7421ebd 100644 (file)
@@ -93,7 +93,6 @@ enum {
 #else
 #define DPRINTF(siba, m, ...) do { (void) siba; } while (0)
 #endif
-#define        N(a)                    (sizeof(a) / sizeof(a[0]))
 
 static void    siba_pci_gpio(struct siba_softc *, uint32_t, int);
 static void    siba_scan(struct siba_softc *);
@@ -963,9 +962,9 @@ siba_cc_pmu_init(struct siba_cc *scc)
                siba_cc_pmu1_pll0_init(scc, 0 /* use default */);
 
                updown = siba_cc_pmu_4325_updown;
-               updown_size = N(siba_cc_pmu_4325_updown);
+               updown_size = nitems(siba_cc_pmu_4325_updown);
                depend = siba_cc_pmu_4325_depend;
-               depend_size = N(siba_cc_pmu_4325_depend);
+               depend_size = nitems(siba_cc_pmu_4325_depend);
 
                min = (1 << SIBA_CC_PMU_4325_BURST) |
                    (1 << SIBA_CC_PMU_4325_LN);
@@ -978,9 +977,9 @@ siba_cc_pmu_init(struct siba_cc *scc)
                siba_cc_pmu0_pll0_init(scc, 0 /* use default */);
 
                updown = siba_cc_pmu_4328_updown;
-               updown_size = N(siba_cc_pmu_4328_updown);
+               updown_size = nitems(siba_cc_pmu_4328_updown);
                depend = siba_cc_pmu_4328_depend;
-               depend_size = N(siba_cc_pmu_4328_depend);
+               depend_size = nitems(siba_cc_pmu_4328_depend);
 
                min = (1 << SIBA_CC_PMU_4328_EXT_SWITCH_PWM) |
                          (1 << SIBA_CC_PMU_4328_BB_SWITCH_PWM) |
@@ -1331,7 +1330,7 @@ siba_cc_pmu1_plltab_find(uint32_t crystalfreq)
        const struct siba_cc_pmu1_plltab *e;
        unsigned int i;
 
-       for (i = 0; i < N(siba_cc_pmu1_plltab); i++) {
+       for (i = 0; i < nitems(siba_cc_pmu1_plltab); i++) {
                e = &siba_cc_pmu1_plltab[i];
                if (crystalfreq == e->freq)
                        return (e);
@@ -1362,7 +1361,7 @@ siba_cc_pmu0_plltab_findentry(uint32_t crystalfreq)
        const struct siba_cc_pmu0_plltab *e;
        unsigned int i;
 
-       for (i = 0; i < N(siba_cc_pmu0_plltab); i++) {
+       for (i = 0; i < nitems(siba_cc_pmu0_plltab); i++) {
                e = &siba_cc_pmu0_plltab[i];
                if (e->freq == crystalfreq)
                        return (e);