Module Name: src Committed By: tsutsui Date: Wed Jan 8 18:31:15 UTC 2025
Modified Files: src/sys/dev/qbus: if_qt.c Log Message: Fix misc style, especially weird indent, per KNF. To generate a diff of this commit: cvs rdiff -u -r1.27 -r1.28 src/sys/dev/qbus/if_qt.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/dev/qbus/if_qt.c diff -u src/sys/dev/qbus/if_qt.c:1.27 src/sys/dev/qbus/if_qt.c:1.28 --- src/sys/dev/qbus/if_qt.c:1.27 Fri Sep 2 23:48:10 2022 +++ src/sys/dev/qbus/if_qt.c Wed Jan 8 18:31:15 2025 @@ -1,4 +1,4 @@ -/* $NetBSD: if_qt.c,v 1.27 2022/09/02 23:48:10 thorpej Exp $ */ +/* $NetBSD: if_qt.c,v 1.28 2025/01/08 18:31:15 tsutsui Exp $ */ /* * Copyright (c) 1992 Steven M. Schultz * All rights reserved. @@ -43,7 +43,7 @@ * It works! Darned board couldn't handle "short" rings - those rings * where only half the entries were made available to the board (the * ring descriptors were the full size, merely half the entries were - * flagged as belonging always to the driver). Grrrr. Would have thought + * flagged as belonging always to the driver). Grrrr. Would have thought * the board could skip over those entries reserved by the driver. * Now to find a way not to have to allocated 32+12 times 1.5kb worth of * buffers... @@ -80,7 +80,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: if_qt.c,v 1.27 2022/09/02 23:48:10 thorpej Exp $"); +__KERNEL_RCSID(0, "$NetBSD: if_qt.c,v 1.28 2025/01/08 18:31:15 tsutsui Exp $"); #include "opt_inet.h" @@ -172,7 +172,9 @@ static void qtsrr(struct qt_softc *, int static void qtrint(struct qt_softc *sc); static void qttint(struct qt_softc *sc); -/* static void qtrestart(struct qt_softc *sc); */ +#ifdef notyet +static void qtreset(struct qt_softc *sc); +#endif CFATTACH_DECL_NEW(qt, sizeof(struct qt_softc), qtmatch, qtattach, NULL, NULL); @@ -180,7 +182,7 @@ CFATTACH_DECL_NEW(qt, sizeof(struct qt_s /* * Maximum packet size needs to include 4 bytes for the CRC * on received packets. -*/ + */ #define MAXPACKETSIZE (ETHERMTU + sizeof (struct ether_header) + 4) #define MINPACKETSIZE 64 @@ -243,7 +245,7 @@ qtmatch(device_t parent, cfdata_t cf, vo * the 'if_attach' call is skipped. For a -YM the START command is issued, * but the device is not marked as running|up - that happens at interrupt level * when the device interrupts to say it has started. -*/ + */ void qtattach(device_t parent, device_t self, void *aux) @@ -268,7 +270,7 @@ qtattach(device_t parent, device_t self, /* * Now allocate the buffers and initialize the buffers. This should _never_ * fail because main memory is allocated after the DMA pool is used up. -*/ + */ sc->is_addr[0] = QT_RCSR(0); sc->is_addr[1] = QT_RCSR(2); @@ -301,7 +303,7 @@ qtturbo(struct qt_softc *sc) * Issue the software reset. Delay 150us. The board should now be in * DELQA-Normal mode. Set ITB and DEQTA select. If both bits do not * stay turned on then the board is not a DELQA-YM. -*/ + */ QT_WCSR(CSR_ARQR, ARQR_SR); QT_WCSR(CSR_ARQR, 0); delay(150L); @@ -310,26 +312,24 @@ qtturbo(struct qt_softc *sc) i = QT_RCSR(CSR_SRR); QT_WCSR(CSR_SRR, 0x8000); /* Turn off ITB, set DELQA select */ if (i != 0x8001) - return(0); + return 0; /* * Board is a DELQA-YM. Send the commands to enable Turbo mode. Delay * 1 second, testing the SRR register every millisecond to see if the * board has shifted to Turbo mode. -*/ + */ QT_WCSR(CSR_XCR0, 0x0baf); QT_WCSR(CSR_XCR1, 0xff00); - for (i = 0; i < 1000; i++) - { - if ((QT_RCSR(CSR_SRR) & SRR_RESP) == 1) + for (i = 0; i < 1000; i++) { + if ((QT_RCSR(CSR_SRR) & SRR_RESP) == 1) break; delay(1000L); - } - if (i >= 1000) - { + } + if (i >= 1000) { printf("qt !Turbo\n"); - return(0); - } - return(1); + return 0; + } + return 1; } #define ETHER_CMP(a,b) memcmp((a), (b), 6) @@ -400,8 +400,8 @@ qtinit(struct ifnet *ifp) struct qt_init *iniblk; struct ifrw *ifrw; struct ifxmt *ifxp; - struct qt_rring *rp; - struct qt_tring *tp; + struct qt_rring *rp; + struct qt_tring *tp; int i, error; if (ifp->if_flags & IFF_RUNNING) { @@ -412,7 +412,8 @@ qtinit(struct ifnet *ifp) if (sc->sc_ib == NULL) { if (if_ubaminit(&sc->sc_ifuba, sc->sc_uh, MCLBYTES, sc->sc_ifr, NRCV, sc->sc_ifw, NXMT)) { - printf("%s: can't initialize\n", device_xname(sc->sc_dev)); + printf("%s: can't initialize\n", + device_xname(sc->sc_dev)); ifp->if_flags &= ~IFF_UP; return 0; } @@ -434,7 +435,7 @@ qtinit(struct ifnet *ifp) * (64 byte) alignment. Since the only time the INIT block is referenced is * at 'startup' or 'reset' time there is really no time penalty (and a modest * D space savings) involved. -*/ + */ memset(sc->sc_ib, 0, sizeof(struct qt_cdata)); iniblk = &sc->sc_ib->qc_init; @@ -459,7 +460,7 @@ qtinit(struct ifnet *ifp) * called with outstanding I/O operations we check the ring descriptors for * a non-zero 'rhost0' (or 'thost0') word and place those buffers back on * the free list. -*/ + */ for (i = 0; i < NRCV; i++) { rp = &sc->sc_ib->qc_r[i]; ifrw = &sc->sc_ifr[i]; @@ -467,14 +468,14 @@ qtinit(struct ifnet *ifp) rp->rmd4 = loint(ifrw->ifrw_info); rp->rmd5 = hiint(ifrw->ifrw_info); rp->rmd3 = 0; /* clear RMD3_OWN */ - } + } for (i = 0; i < NXMT; i++) { tp = &sc->sc_ib->qc_t[i]; ifxp = &sc->sc_ifw[i]; tp->tmd4 = loint(ifxp->ifw_info); tp->tmd5 = hiint(ifxp->ifw_info); tp->tmd3 = TMD3_OWN; - } + } sc->xnext = sc->xlast = sc->nxmit = 0; sc->rindex = 0; @@ -489,14 +490,14 @@ qtinit(struct ifnet *ifp) * full operations will proceed. If the timeout expires without an interrupt * being received an error is printed, the flags cleared and the device left * marked down. -*/ + */ QT_WCSR(CSR_IBAL, loint(&sc->sc_pib->qc_init)); QT_WCSR(CSR_IBAH, hiint(&sc->sc_pib->qc_init)); QT_WCSR(CSR_SRQR, 2); sc->is_if.if_flags |= IFF_RUNNING; return 0; - } +} /* * Start output on interface. @@ -504,15 +505,15 @@ qtinit(struct ifnet *ifp) void qtstart(struct ifnet *ifp) - { - int len, nxmit; +{ + int len, nxmit; struct qt_softc *sc = ifp->if_softc; struct qt_tring *rp; - struct mbuf *m = NULL; + struct mbuf *m = NULL; for (nxmit = sc->nxmit; nxmit < NXMT; nxmit++) { IF_DEQUEUE(&sc->is_if.if_snd, m); - if (m == 0) + if (m == NULL) break; rp = &sc->sc_ib->qc_t[sc->xnext]; @@ -527,7 +528,7 @@ qtstart(struct ifnet *ifp) rp->tmd3 = len & TMD3_BCT; /* set length,clear ownership */ QT_WCSR(CSR_ARQR, ARQR_TRQ); /* tell device it has buffer */ - if (++sc->xnext >= NXMT) + if (++sc->xnext >= NXMT) sc->xnext = 0; } if (sc->nxmit != nxmit) @@ -540,32 +541,32 @@ qtstart(struct ifnet *ifp) * interrupts and timeouts come here. Check for hard device errors and print a * message if any errors are found. If we are waiting for the device to * START then check if the device is now running. -*/ + */ void qtintr(void *arg) - { +{ struct qt_softc *sc = arg; struct ifnet *ifp = &sc->is_if; short status; - status = QT_RCSR(CSR_SRR); - if (status < 0) + if (status < 0) { /* should we reset the device after a bunch of these errs? */ qtsrr(sc, status); + } if ((ifp->if_flags & IFF_UP) == 0) return; /* Unwanted interrupt */ qtrint(sc); qttint(sc); qtstart(&sc->is_ec.ec_if); - } +} /* * Transmit interrupt service. Only called if there are outstanding transmit * requests which could have completed. The DELQA-YM doesn't provide the * status bits telling the kind (receive, transmit) of interrupt. -*/ + */ #define BBLMIS (TMD2_BBL|TMD2_MIS) @@ -582,7 +583,7 @@ qttint(struct qt_softc *sc) /* * Collisions don't count as output errors, but babbling and missing packets * do count as output errors. -*/ + */ if (rp->tmd2 & TMD2_CER) if_statinc(&sc->is_if, if_collisions); if ((rp->tmd0 & TMD0_ERR1) || @@ -604,7 +605,7 @@ qttint(struct qt_softc *sc) /* * Receive interrupt service. Pull packet off the interface and put into * a mbuf chain for processing later. -*/ + */ void qtrint(struct qt_softc *sc) @@ -612,13 +613,13 @@ qtrint(struct qt_softc *sc) struct qt_rring *rp; struct ifnet *ifp = &sc->is_ec.ec_if; struct mbuf *m; - int len; + int len; - while (sc->sc_ib->qc_r[(int)sc->rindex].rmd3 & RMD3_OWN) - { + while (sc->sc_ib->qc_r[(int)sc->rindex].rmd3 & RMD3_OWN) { rp = &sc->sc_ib->qc_r[(int)sc->rindex]; if ((rp->rmd0 & (RMD0_STP|RMD0_ENP)) != (RMD0_STP|RMD0_ENP)) { - printf("%s: chained packet\n", device_xname(sc->sc_dev)); + printf("%s: chained packet\n", + device_xname(sc->sc_dev)); if_statinc(&sc->is_if, if_ierrors); goto rnext; } @@ -637,7 +638,7 @@ qtrint(struct qt_softc *sc) } m = if_ubaget(&sc->sc_ifuba, &sc->sc_ifr[(int)sc->rindex], ifp, len); - if (m == 0) { + if (m == NULL) { if_statinc(&sc->is_if, if_ierrors); goto rnext; } @@ -646,11 +647,11 @@ rnext: --sc->nrcv; rp->rmd3 = 0; rp->rmd1 = MCLBYTES; - if (++sc->rindex >= NRCV) + if (++sc->rindex >= NRCV) sc->rindex = 0; - } - QT_WCSR(CSR_ARQR, ARQR_RRQ); /* tell device it has buffer */ } + QT_WCSR(CSR_ARQR, ARQR_RRQ); /* tell device it has buffer */ +} int qtioctl(struct ifnet *ifp, u_long cmd, void *data) @@ -667,13 +668,14 @@ qtioctl(struct ifnet *ifp, u_long cmd, v error = 0; } splx(s); - return (error); + return error; } void qtsrr(struct qt_softc *sc, int srrbits) { char buf[100]; + snprintb(buf, sizeof(buf), SRR_BITS, srrbits); printf("%s: srr=%s\n", device_xname(sc->sc_dev), buf); } @@ -689,9 +691,10 @@ qtstop(struct ifnet *ifp, int disable) int i; QT_WCSR(CSR_SRQR, 3); - for (i = 0; i < 100; i++) + for (i = 0; i < 100; i++) { if ((QT_RCSR(CSR_SRR) & SRR_RESP) == 3) break; + } if (QT_RCSR(CSR_SRR) & SRR_FES) qtsrr(sc, QT_RCSR(CSR_SRR)); /* Forget already queued transmit requests */ @@ -712,14 +715,13 @@ qtstop(struct ifnet *ifp, int disable) * After the reset put the device back in -T mode. Then call qtinit() to * reinitialize the ring structures and issue the 'timeout' for the "device * started interrupt". -*/ + */ void -qtreset(sc) - struct qt_softc *sc; - { +qtreset(struct qt_softc *sc) +{ qtturbo(sc); qtinit(&sc->is_ec.ec_if); - } +} #endif