Module Name:    src
Committed By:   msaitoh
Date:           Thu May 23 10:30:36 UTC 2019

Modified Files:
        src/sys/arch/arm/ep93xx: epe.c
        src/sys/arch/mac68k/dev: if_mc.c
        src/sys/arch/macppc/dev: am79c950.c
        src/sys/dev/cadence: if_cemac.c
        src/sys/dev/ic: lance.c
        src/sys/dev/pci: if_txp.c
        src/sys/dev/scsipi: if_se.c

Log Message:
No functional change:
 - Change ac(was arpcom) to ec(ethercom)
 - Simplify MII structure initialization.
 - u_int*_t -> uint*_t.
 - KNF


To generate a diff of this commit:
cvs rdiff -u -r1.40 -r1.41 src/sys/arch/arm/ep93xx/epe.c
cvs rdiff -u -r1.49 -r1.50 src/sys/arch/mac68k/dev/if_mc.c
cvs rdiff -u -r1.43 -r1.44 src/sys/arch/macppc/dev/am79c950.c
cvs rdiff -u -r1.18 -r1.19 src/sys/dev/cadence/if_cemac.c
cvs rdiff -u -r1.55 -r1.56 src/sys/dev/ic/lance.c
cvs rdiff -u -r1.54 -r1.55 src/sys/dev/pci/if_txp.c
cvs rdiff -u -r1.99 -r1.100 src/sys/dev/scsipi/if_se.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/arch/arm/ep93xx/epe.c
diff -u src/sys/arch/arm/ep93xx/epe.c:1.40 src/sys/arch/arm/ep93xx/epe.c:1.41
--- src/sys/arch/arm/ep93xx/epe.c:1.40	Tue Feb  5 06:17:01 2019
+++ src/sys/arch/arm/ep93xx/epe.c	Thu May 23 10:30:35 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: epe.c,v 1.40 2019/02/05 06:17:01 msaitoh Exp $	*/
+/*	$NetBSD: epe.c,v 1.41 2019/05/23 10:30:35 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2004 Jesse Off
@@ -27,7 +27,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: epe.c,v 1.40 2019/02/05 06:17:01 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: epe.c,v 1.41 2019/05/23 10:30:35 msaitoh Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -45,7 +45,7 @@ __KERNEL_RCSID(0, "$NetBSD: epe.c,v 1.40
 
 #include <arm/cpufunc.h>
 
-#include <arm/ep93xx/epsocvar.h> 
+#include <arm/ep93xx/epsocvar.h>
 #include <arm/ep93xx/ep93xxvar.h>
 
 #include <net/if.h>
@@ -67,8 +67,8 @@ __KERNEL_RCSID(0, "$NetBSD: epe.c,v 1.40
 #endif
 
 #include <arm/ep93xx/ep93xxreg.h>
-#include <arm/ep93xx/epereg.h> 
-#include <arm/ep93xx/epevar.h> 
+#include <arm/ep93xx/epereg.h>
+#include <arm/ep93xx/epevar.h>
 
 #define DEFAULT_MDCDIV	32
 
@@ -131,7 +131,7 @@ epe_attach(device_t parent, device_t sel
 	sc->sc_intr = sa->sa_intr;
 	sc->sc_dmat = sa->sa_dmat;
 
-	if (bus_space_map(sa->sa_iot, sa->sa_addr, sa->sa_size, 
+	if (bus_space_map(sa->sa_iot, sa->sa_addr, sa->sa_size,
 		0, &sc->sc_ioh))
 		panic("%s: Cannot map registers", device_xname(self));
 
@@ -161,20 +161,19 @@ epe_gctx(struct epe_softc *sc)
 	cur = (uint32_t *)(EPE_READ(TXStsQCurAdd) -
 		sc->ctrlpage_dsaddr + (char*)sc->ctrlpage);
 
-	if (sc->TXStsQ_cur != cur) { 
-		CTRLPAGE_DMASYNC(TX_QLEN * 2 * sizeof(uint32_t), 
+	if (sc->TXStsQ_cur != cur) {
+		CTRLPAGE_DMASYNC(TX_QLEN * 2 * sizeof(uint32_t),
 			TX_QLEN * sizeof(uint32_t), BUS_DMASYNC_PREREAD);
-	} else {
+	} else
 		return 0;
-	}
 
 	do {
 		uint32_t tbi = *sc->TXStsQ_cur & 0x7fff;
 		struct mbuf *m = sc->txq[tbi].m;
 
-		if ((*sc->TXStsQ_cur & TXStsQ_TxWE) == 0) {
+		if ((*sc->TXStsQ_cur & TXStsQ_TxWE) == 0)
 			ifp->if_oerrors++;
-		}
+
 		bus_dmamap_unload(sc->sc_dmat, sc->txq[tbi].m_dmamap);
 		m_freem(m);
 		do {
@@ -188,7 +187,7 @@ epe_gctx(struct epe_softc *sc)
 		if (sc->TXStsQ_cur >= sc->TXStsQ + TX_QLEN) {
 			sc->TXStsQ_cur = sc->TXStsQ;
 		}
-	} while (sc->TXStsQ_cur != cur); 
+	} while (sc->TXStsQ_cur != cur);
 
 	sc->TXDQ_avail += ndq;
 	if (ifp->if_flags & IFF_OACTIVE) {
@@ -211,11 +210,11 @@ begin:
 	cur = (uint32_t *)(EPE_READ(RXStsQCurAdd) -
 		sc->ctrlpage_dsaddr + (char*)sc->ctrlpage);
 	CTRLPAGE_DMASYNC(TX_QLEN * 3 * sizeof(uint32_t),
-		RX_QLEN * 4 * sizeof(uint32_t), 
+		RX_QLEN * 4 * sizeof(uint32_t),
 		BUS_DMASYNC_PREREAD);
 	while (sc->RXStsQ_cur != cur) {
-		if ((sc->RXStsQ_cur[0] & (RXStsQ_RWE|RXStsQ_RFP|RXStsQ_EOB)) == 
-			(RXStsQ_RWE|RXStsQ_RFP|RXStsQ_EOB)) {
+		if ((sc->RXStsQ_cur[0] & (RXStsQ_RWE | RXStsQ_RFP |RXStsQ_EOB))
+		    == (RXStsQ_RWE | RXStsQ_RFP | RXStsQ_EOB)) {
 			uint32_t bi = (sc->RXStsQ_cur[1] >> 16) & 0x7fff;
 			uint32_t fl = sc->RXStsQ_cur[1] & 0xffff;
 			struct mbuf *m;
@@ -223,46 +222,44 @@ begin:
 			MGETHDR(m, M_DONTWAIT, MT_DATA);
 			if (m != NULL) MCLGET(m, M_DONTWAIT);
 			if (m != NULL && (m->m_flags & M_EXT)) {
-				bus_dmamap_unload(sc->sc_dmat, 
+				bus_dmamap_unload(sc->sc_dmat,
 					sc->rxq[bi].m_dmamap);
 				m_set_rcvif(sc->rxq[bi].m, ifp);
-				sc->rxq[bi].m->m_pkthdr.len = 
+				sc->rxq[bi].m->m_pkthdr.len =
 					sc->rxq[bi].m->m_len = fl;
 				if_percpuq_enqueue(ifp->if_percpuq,
 				    sc->rxq[bi].m);
 				sc->rxq[bi].m = m;
-				bus_dmamap_load(sc->sc_dmat, 
-					sc->rxq[bi].m_dmamap, 
+				bus_dmamap_load(sc->sc_dmat,
+					sc->rxq[bi].m_dmamap,
 					m->m_ext.ext_buf, MCLBYTES,
 					NULL, BUS_DMA_NOWAIT);
-				sc->RXDQ[bi * 2] = 
+				sc->RXDQ[bi * 2] =
 					sc->rxq[bi].m_dmamap->dm_segs[0].ds_addr;
 			} else {
 				/* Drop packets until we can get replacement
 				 * empty mbufs for the RXDQ.
 				 */
-				if (m != NULL) {
+				if (m != NULL)
 					m_freem(m);
-				}
+
 				ifp->if_ierrors++;
-			} 
-		} else {
+			}
+		} else
 			ifp->if_ierrors++;
-		}
 
 		ndq++;
 
 		sc->RXStsQ_cur += 2;
-		if (sc->RXStsQ_cur >= sc->RXStsQ + (RX_QLEN * 2)) {
+		if (sc->RXStsQ_cur >= sc->RXStsQ + (RX_QLEN * 2))
 			sc->RXStsQ_cur = sc->RXStsQ;
-		}
 	}
 
 	if (ndq > 0) {
 		ifp->if_ipackets += ndq;
 		CTRLPAGE_DMASYNC(TX_QLEN * 3 * sizeof(uint32_t),
- 			RX_QLEN * 4 * sizeof(uint32_t), 
-			BUS_DMASYNC_PREWRITE|BUS_DMASYNC_PREREAD);
+ 			RX_QLEN * 4 * sizeof(uint32_t),
+			BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		EPE_WRITE(RXStsEnq, ndq);
 		EPE_WRITE(RXDEnq, ndq);
 		ndq = 0;
@@ -270,13 +267,13 @@ begin:
 
 	if (epe_gctx(sc) > 0 && IFQ_IS_EMPTY(&ifp->if_snd) == 0) {
 		if_schedule_deferred_start(ifp);
-	} 
+	}
 
 	irq = EPE_READ(IntStsC);
-	if ((irq & (IntSts_RxSQ|IntSts_ECI)) != 0)
+	if ((irq & (IntSts_RxSQ | IntSts_ECI)) != 0)
 		goto begin;
 
-	return (1);
+	return 1;
 }
 
 
@@ -296,12 +293,13 @@ epe_init(struct epe_softc *sc)
 	/* Read ethernet MAC, should already be set by bootrom */
 	bus_space_read_region_1(sc->sc_iot, sc->sc_ioh, EPE_IndAd,
 		sc->sc_enaddr, ETHER_ADDR_LEN);
-	aprint_normal_dev(sc->sc_dev, "MAC address %s\n", 
+	aprint_normal_dev(sc->sc_dev, "MAC address %s\n",
 		ether_sprintf(sc->sc_enaddr));
 
 	/* Soft Reset the MAC */
 	EPE_WRITE(SelfCtl, SelfCtl_RESET);
-	while(EPE_READ(SelfCtl) & SelfCtl_RESET);
+	while (EPE_READ(SelfCtl) & SelfCtl_RESET)
+		;
 
 	/* suggested magic initialization values from datasheet */
 	EPE_WRITE(RXBufThrshld, 0x800040);
@@ -312,11 +310,11 @@ epe_init(struct epe_softc *sc)
 	EPE_WRITE(TXDThrshld, 0x40002);
 
 	/* Allocate a page of memory for descriptor and status queues */
-	err = bus_dmamem_alloc(sc->sc_dmat, PAGE_SIZE, 0, PAGE_SIZE, 
+	err = bus_dmamem_alloc(sc->sc_dmat, PAGE_SIZE, 0, PAGE_SIZE,
 		&segs, 1, &rsegs, BUS_DMA_WAITOK);
 	if (err == 0) {
-		err = bus_dmamem_map(sc->sc_dmat, &segs, 1, PAGE_SIZE, 
-			&sc->ctrlpage, (BUS_DMA_WAITOK|BUS_DMA_COHERENT));
+		err = bus_dmamem_map(sc->sc_dmat, &segs, 1, PAGE_SIZE,
+			&sc->ctrlpage, (BUS_DMA_WAITOK | BUS_DMA_COHERENT));
 	}
 	if (err == 0) {
 		err = bus_dmamap_create(sc->sc_dmat, PAGE_SIZE, 1, PAGE_SIZE,
@@ -331,7 +329,7 @@ epe_init(struct epe_softc *sc)
 	}
 	sc->ctrlpage_dsaddr = sc->ctrlpage_dmamap->dm_segs[0].ds_addr;
 	memset(sc->ctrlpage, 0, PAGE_SIZE);
-	
+
 	/* Set up pointers to start of each queue in kernel addr space.
 	 * Each descriptor queue or status queue entry uses 2 words
 	 */
@@ -345,12 +343,12 @@ epe_init(struct epe_softc *sc)
 	sc->RXStsQ_cur = sc->RXStsQ;
 
 	/* Program each queue's start addr, cur addr, and len registers
-	 * with the physical addresses. 
+	 * with the physical addresses.
 	 */
 	addr = (char *)sc->ctrlpage_dmamap->dm_segs[0].ds_addr;
 	EPE_WRITE(TXDQBAdd, (uint32_t)addr);
 	EPE_WRITE(TXDQCurAdd, (uint32_t)addr);
-	EPE_WRITE(TXDQBLen, TX_QLEN * 2 * sizeof(uint32_t)); 
+	EPE_WRITE(TXDQBLen, TX_QLEN * 2 * sizeof(uint32_t));
 
 	addr += (sc->TXStsQ - sc->TXDQ) * sizeof(uint32_t);
 	EPE_WRITE(TXStsQBAdd, (uint32_t)addr);
@@ -361,24 +359,23 @@ epe_init(struct epe_softc *sc)
 	EPE_WRITE(RXDQBAdd, (uint32_t)addr);
 	EPE_WRITE(RXDCurAdd, (uint32_t)addr);
 	EPE_WRITE(RXDQBLen, RX_QLEN * 2 * sizeof(uint32_t));
-	
+
 	addr += (sc->RXStsQ - sc->RXDQ) * sizeof(uint32_t);
 	EPE_WRITE(RXStsQBAdd, (uint32_t)addr);
 	EPE_WRITE(RXStsQCurAdd, (uint32_t)addr);
 	EPE_WRITE(RXStsQBLen, RX_QLEN * 2 * sizeof(uint32_t));
 
 	/* Populate the RXDQ with mbufs */
-	for(i = 0; i < RX_QLEN; i++) {
+	for (i = 0; i < RX_QLEN; i++) {
 		struct mbuf *m;
 
-		bus_dmamap_create(sc->sc_dmat, MCLBYTES, TX_QLEN/4, MCLBYTES, 0,
-			BUS_DMA_WAITOK, &sc->rxq[i].m_dmamap);
+		bus_dmamap_create(sc->sc_dmat, MCLBYTES, TX_QLEN/4, MCLBYTES,
+		    0, BUS_DMA_WAITOK, &sc->rxq[i].m_dmamap);
 		MGETHDR(m, M_WAIT, MT_DATA);
 		MCLGET(m, M_WAIT);
 		sc->rxq[i].m = m;
-		bus_dmamap_load(sc->sc_dmat, sc->rxq[i].m_dmamap, 
-			m->m_ext.ext_buf, MCLBYTES, NULL, 
-			BUS_DMA_WAITOK);
+		bus_dmamap_load(sc->sc_dmat, sc->rxq[i].m_dmamap,
+			m->m_ext.ext_buf, MCLBYTES, NULL, BUS_DMA_WAITOK);
 
 		sc->RXDQ[i * 2] = sc->rxq[i].m_dmamap->dm_segs[0].ds_addr;
 		sc->RXDQ[i * 2 + 1] = (i << 16) | MCLBYTES;
@@ -386,9 +383,9 @@ epe_init(struct epe_softc *sc)
 			MCLBYTES, BUS_DMASYNC_PREREAD);
 	}
 
-	for(i = 0; i < TX_QLEN; i++) {
+	for (i = 0; i < TX_QLEN; i++) {
 		bus_dmamap_create(sc->sc_dmat, MCLBYTES, 1, MCLBYTES, 0,
-			(BUS_DMA_WAITOK|BUS_DMA_ALLOCNOW),
+			(BUS_DMA_WAITOK | BUS_DMA_ALLOCNOW),
 			&sc->txq[i].m_dmamap);
 		sc->txq[i].m = NULL;
 		sc->TXDQ[i * 2 + 1] = (i << 16);
@@ -397,7 +394,7 @@ epe_init(struct epe_softc *sc)
 	/* Divide HCLK by 32 for MDC clock */
 	if (device_cfdata(sc->sc_dev)->cf_flags)
 		mdcdiv = device_cfdata(sc->sc_dev)->cf_flags;
-	EPE_WRITE(SelfCtl, (SelfCtl_MDCDIV(mdcdiv)|SelfCtl_PSPRS));
+	EPE_WRITE(SelfCtl, (SelfCtl_MDCDIV(mdcdiv) | SelfCtl_PSPRS));
 
 	sc->sc_mii.mii_ifp = ifp;
 	sc->sc_mii.mii_readreg = epe_mii_readreg;
@@ -408,18 +405,18 @@ epe_init(struct epe_softc *sc)
 		ether_mediastatus);
 	mii_attach(sc->sc_dev, &sc->sc_mii, 0xffffffff, MII_PHY_ANY,
 		MII_OFFSET_ANY, 0);
-	ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+	ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER | IFM_AUTO);
 
-	EPE_WRITE(BMCtl, BMCtl_RxEn|BMCtl_TxEn);
+	EPE_WRITE(BMCtl, BMCtl_RxEn | BMCtl_TxEn);
 	EPE_WRITE(IntEn, IntEn_REOFIE);
 	/* maximum valid max frame length */
-	EPE_WRITE(MaxFrmLen, (0x7ff << 16)|MHLEN);
+	EPE_WRITE(MaxFrmLen, (0x7ff << 16) | MHLEN);
 	/* wait for receiver ready */
-	while((EPE_READ(BMSts) & BMSts_RxAct) == 0)
+	while ((EPE_READ(BMSts) & BMSts_RxAct) == 0)
 		continue;
 	/* enqueue the entries in RXStsQ and RXDQ */
-	CTRLPAGE_DMASYNC(0, sc->ctrlpage_dmamap->dm_mapsize, 
-		BUS_DMASYNC_PREWRITE|BUS_DMASYNC_PREREAD);
+	CTRLPAGE_DMASYNC(0, sc->ctrlpage_dmamap->dm_mapsize,
+		BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 	EPE_WRITE(RXDEnq, RX_QLEN - 1);
 	EPE_WRITE(RXStsEnq, RX_QLEN - 1);
 
@@ -448,7 +445,7 @@ epe_mediachange(struct ifnet *ifp)
 {
 	if (ifp->if_flags & IFF_UP)
 		epe_ifinit(ifp);
-	return (0);
+	return 0;
 }
 
 int
@@ -459,7 +456,8 @@ epe_mii_readreg(device_t self, int phy, 
 	d = EPE_READ(SelfCtl);
 	EPE_WRITE(SelfCtl, d & ~SelfCtl_PSPRS); /* no preamble suppress */
 	EPE_WRITE(MIICmd, (MIICmd_READ | (phy << 5) | reg));
-	while(EPE_READ(MIISts) & MIISts_BUSY);
+	while (EPE_READ(MIISts) & MIISts_BUSY)
+		;
 	*val = EPE_READ(MIIData) & 0xffff;
 	EPE_WRITE(SelfCtl, d); /* restore old value */
 	return 0;
@@ -474,13 +472,13 @@ epe_mii_writereg(device_t self, int phy,
 	EPE_WRITE(SelfCtl, d & ~SelfCtl_PSPRS); /* no preamble suppress */
 	EPE_WRITE(MIIData, val);
 	EPE_WRITE(MIICmd, (MIICmd_WRITE | (phy << 5) | reg));
-	while(EPE_READ(MIISts) & MIISts_BUSY);
+	while (EPE_READ(MIISts) & MIISts_BUSY)
+		;
 	EPE_WRITE(SelfCtl, d); /* restore old value */
 
 	return 0;
 }
 
-	
 void
 epe_statchg(struct ifnet *ifp)
 {
@@ -510,9 +508,9 @@ epe_tick(void *arg)
 	ifp->if_collisions += EPE_READ(TXCollCnt);
 	/* These misses are ok, they will happen if the RAM/CPU can't keep up */
 	misses = EPE_READ(RXMissCnt);
-	if (misses > 0) 
+	if (misses > 0)
 		printf("%s: %d rx misses\n", device_xname(sc->sc_dev), misses);
-	
+
 	s = splnet();
 	if (epe_gctx(sc) > 0 && IFQ_IS_EMPTY(&ifp->if_snd) == 0) {
 		epe_ifstart(ifp);
@@ -548,21 +546,21 @@ epe_ifstart(struct ifnet *ifp)
 	bus_dma_segment_t *segs;
 	int s, bi, err, nsegs, ndq;
 
-	s = splnet();	
+	s = splnet();
 start:
 	ndq = 0;
 	if (sc->TXDQ_avail == 0) {
 		if (epe_gctx(sc) == 0) {
 			/* Enable End-Of-TX-Chain interrupt */
-			EPE_WRITE(IntEn, IntEn_REOFIE|IntEn_ECIE);
+			EPE_WRITE(IntEn, IntEn_REOFIE | IntEn_ECIE);
 			ifp->if_flags |= IFF_OACTIVE;
 			ifp->if_timer = 10;
 			splx(s);
 			return;
 		}
-	} 
+	}
 
-	bi = sc->TXDQ_cur - sc->TXDQ; 
+	bi = sc->TXDQ_cur - sc->TXDQ;
 
 	IFQ_POLL(&ifp->if_snd, m);
 	if (m == NULL) {
@@ -571,13 +569,13 @@ start:
 	}
 more:
 	if ((err = bus_dmamap_load_mbuf(sc->sc_dmat, sc->txq[bi].m_dmamap, m,
-		BUS_DMA_NOWAIT)) || 
+		BUS_DMA_NOWAIT)) ||
 		sc->txq[bi].m_dmamap->dm_segs[0].ds_addr & 0x3 ||
 		sc->txq[bi].m_dmamap->dm_nsegs > (sc->TXDQ_avail - ndq)) {
 		/* Copy entire mbuf chain to new and 32-bit aligned storage */
 		struct mbuf *mn;
 
-		if (err == 0) 
+		if (err == 0)
 			bus_dmamap_unload(sc->sc_dmat, sc->txq[bi].m_dmamap);
 
 		MGETHDR(mn, M_DONTWAIT, MT_DATA);
@@ -589,7 +587,7 @@ more:
 				goto stop;
 			}
 		}
-		mn->m_data = (void *)(((uint32_t)mn->m_data + 0x3) & (~0x3)); 
+		mn->m_data = (void *)(((uint32_t)mn->m_data + 0x3) & (~0x3));
 		m_copydata(m, 0, m->m_pkthdr.len, mtod(mn, void *));
 		mn->m_pkthdr.len = mn->m_len = m->m_pkthdr.len;
 		IFQ_DEQUEUE(&ifp->if_snd, m);
@@ -605,9 +603,9 @@ more:
 
 	nsegs = sc->txq[bi].m_dmamap->dm_nsegs;
 	segs = sc->txq[bi].m_dmamap->dm_segs;
-	bus_dmamap_sync(sc->sc_dmat, sc->txq[bi].m_dmamap, 0, 
-		sc->txq[bi].m_dmamap->dm_mapsize, 
-		BUS_DMASYNC_PREREAD|BUS_DMASYNC_PREWRITE);
+	bus_dmamap_sync(sc->sc_dmat, sc->txq[bi].m_dmamap, 0,
+		sc->txq[bi].m_dmamap->dm_mapsize,
+		BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 
 	/* XXX: This driver hasn't been tested w/nsegs > 1 */
 	while (nsegs > 0) {
@@ -631,16 +629,15 @@ more:
 	 */
 	if ((sc->TXDQ_avail - ndq) > 0 && ndq < TX_QLEN / 2) {
 		IFQ_POLL(&ifp->if_snd, m);
-		if (m != NULL) {
+		if (m != NULL)
 			goto more;
-		}
-	} 
+	}
 stop:
 	if (ndq > 0) {
 		sc->TXDQ_avail -= ndq;
 		sc->TXDQ_cur = &sc->TXDQ[bi];
 		CTRLPAGE_DMASYNC(0, TX_QLEN * 2 * sizeof(uint32_t),
-			BUS_DMASYNC_PREWRITE|BUS_DMASYNC_PREREAD);
+			BUS_DMASYNC_PREWRITE | BUS_DMASYNC_PREREAD);
 		EPE_WRITE(TXDEnq, ndq);
 	}
 
@@ -658,7 +655,7 @@ epe_ifwatchdog(struct ifnet *ifp)
 
 	if ((ifp->if_flags & IFF_RUNNING) == 0)
 		return;
-       	printf("%s: device timeout, BMCtl = 0x%08x, BMSts = 0x%08x\n", 
+       	printf("%s: device timeout, BMCtl = 0x%08x, BMSts = 0x%08x\n",
 		device_xname(sc->sc_dev), EPE_READ(BMCtl), EPE_READ(BMSts));
 }
 
@@ -669,7 +666,7 @@ epe_ifinit(struct ifnet *ifp)
 	int rc, s = splnet();
 
 	callout_stop(&sc->epe_tick_ch);
-	EPE_WRITE(RXCtl, RXCtl_IA0|RXCtl_BA|RXCtl_RCRCA|RXCtl_SRxON);
+	EPE_WRITE(RXCtl, RXCtl_IA0 | RXCtl_BA | RXCtl_RCRCA | RXCtl_SRxON);
 	EPE_WRITE(TXCtl, TXCtl_STxON);
 	EPE_WRITE(GIIntMsk, GIIntMsk_INT); /* start interrupting */
 
@@ -708,7 +705,7 @@ static void
 epe_setaddr(struct ifnet *ifp)
 {
 	struct epe_softc *sc = ifp->if_softc;
-	struct ethercom *ac = &sc->sc_ec;
+	struct ethercom *ec = &sc->sc_ec;
 	struct ether_multi *enm;
 	struct ether_multistep step;
 	uint8_t ias[2][ETHER_ADDR_LEN];
@@ -717,16 +714,15 @@ epe_setaddr(struct ifnet *ifp)
 
 	/* disable receiver temporarily */
 	EPE_WRITE(RXCtl, rxctl & ~RXCtl_SRxON);
-	
-	rxctl &= ~(RXCtl_MA|RXCtl_PA|RXCtl_IA2|RXCtl_IA3);
 
-	if (ifp->if_flags & IFF_PROMISC) {
+	rxctl &= ~(RXCtl_MA | RXCtl_PA | RXCtl_IA2 | RXCtl_IA3);
+
+	if (ifp->if_flags & IFF_PROMISC)
 		rxctl |= RXCtl_PA;
-	} 
 
 	ifp->if_flags &= ~IFF_ALLMULTI;
 
-	ETHER_FIRST_MULTI(step, ac, enm);
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (memcmp(enm->enm_addrlo, enm->enm_addrhi, ETHER_ADDR_LEN)) {
 			/*
@@ -737,7 +733,7 @@ epe_setaddr(struct ifnet *ifp)
 			 * ranges is for IP multicast routing, for which the
 			 * range is big enough to require all bits set.)
 			 */
-			rxctl &= ~(RXCtl_IA2|RXCtl_IA3);
+			rxctl &= ~(RXCtl_IA2 | RXCtl_IA3);
 			rxctl |= RXCtl_MA;
 			hashes[0] = 0xffffffffUL;
 			hashes[1] = 0xffffffffUL;
@@ -747,7 +743,7 @@ epe_setaddr(struct ifnet *ifp)
 
 		if (nma < 2) {
 			/* We can program 2 perfect address filters for mcast */
-			memcpy(ias[nma], enm->enm_addrlo, ETHER_ADDR_LEN);			
+			memcpy(ias[nma], enm->enm_addrlo, ETHER_ADDR_LEN);
 			rxctl |= (1 << (nma + 2));
 		} else {
 			/*
@@ -765,9 +761,9 @@ epe_setaddr(struct ifnet *ifp)
 		ETHER_NEXT_MULTI(step, enm);
 		nma++;
 	}
-	
+
 	EPE_WRITE(AFP, 0);
-	bus_space_write_region_1(sc->sc_iot, sc->sc_ioh, EPE_IndAd, 
+	bus_space_write_region_1(sc->sc_iot, sc->sc_ioh, EPE_IndAd,
 		sc->sc_enaddr, ETHER_ADDR_LEN);
 	if (rxctl & RXCtl_IA2) {
 		EPE_WRITE(AFP, 2);

Index: src/sys/arch/mac68k/dev/if_mc.c
diff -u src/sys/arch/mac68k/dev/if_mc.c:1.49 src/sys/arch/mac68k/dev/if_mc.c:1.50
--- src/sys/arch/mac68k/dev/if_mc.c:1.49	Tue Feb  5 06:17:01 2019
+++ src/sys/arch/mac68k/dev/if_mc.c	Thu May 23 10:30:35 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_mc.c,v 1.49 2019/02/05 06:17:01 msaitoh Exp $	*/
+/*	$NetBSD: if_mc.c,v 1.50 2019/05/23 10:30:35 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 1997 David Huang <k...@azeotrope.org>
@@ -35,7 +35,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_mc.c,v 1.49 2019/02/05 06:17:01 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_mc.c,v 1.50 2019/05/23 10:30:35 msaitoh Exp $");
 
 #include "opt_ddb.h"
 #include "opt_inet.h"
@@ -81,8 +81,8 @@ integrate u_int	maceput(struct mc_softc 
 integrate void	mc_tint(struct mc_softc *);
 integrate void	mace_read(struct mc_softc *, void *, int);
 integrate struct mbuf *mace_get(struct mc_softc *, void *, int);
-static void mace_calcladrf(struct ethercom *, u_int8_t *);
-static inline u_int16_t ether_cmp(void *, void *);
+static void mace_calcladrf(struct ethercom *, uint8_t *);
+static inline uint16_t ether_cmp(void *, void *);
 
 
 /*
@@ -99,12 +99,12 @@ static inline u_int16_t ether_cmp(void *
  * Please do NOT tweak this without looking at the actual
  * assembly code generated before and after your tweaks!
  */
-static inline u_int16_t
+static inline uint16_t
 ether_cmp(void *one, void *two)
 {
-	u_int16_t *a = (u_short *) one;
-	u_int16_t *b = (u_short *) two;
-	u_int16_t diff;
+	uint16_t *a = (u_short *) one;
+	uint16_t *b = (u_short *) two;
+	uint16_t diff;
 
 #ifdef	m68k
 	/*
@@ -124,7 +124,7 @@ ether_cmp(void *one, void *two)
 	diff = (a[0] - b[0]) | (a[1] - b[1]) | (a[2] - b[2]);
 #endif
 
-	return (diff);
+	return diff;
 }
 
 #define ETHER_CMP	ether_cmp
@@ -135,7 +135,7 @@ ether_cmp(void *one, void *two)
  * to accept packets.
  */
 int
-mcsetup(struct mc_softc	*sc, u_int8_t *lladdr)
+mcsetup(struct mc_softc	*sc, uint8_t *lladdr)
 {
 	struct ifnet *ifp = &sc->sc_if;
 
@@ -158,7 +158,7 @@ mcsetup(struct mc_softc	*sc, u_int8_t *l
 	if_deferred_start_init(ifp, NULL);
 	ether_ifattach(ifp, lladdr);
 
-	return (0);
+	return 0;
 }
 
 hide int
@@ -231,7 +231,7 @@ mcioctl(struct ifnet *ifp, u_long cmd, v
 		err = ether_ioctl(ifp, cmd, data);
 	}
 	splx(s);
-	return (err);
+	return err;
 }
 
 /*
@@ -285,11 +285,11 @@ hide int
 mcinit(struct mc_softc *sc)
 {
 	int s;
-	u_int8_t maccc, ladrf[8];
+	uint8_t maccc, ladrf[8];
 
 	if (sc->sc_if.if_flags & IFF_RUNNING)
 		/* already running */
-		return (0);
+		return 0;
 
 	s = splnet();
 
@@ -346,7 +346,7 @@ mcinit(struct mc_softc *sc)
 	sc->sc_if.if_flags &= ~IFF_OACTIVE;
 
 	splx(s);
-	return (0);
+	return 0;
 }
 
 /*
@@ -358,7 +358,7 @@ hide int
 mcstop(struct mc_softc *sc)
 {
 	int s;
-	
+
 	s = splnet();
 
 	NIC_PUT(sc, MACE_BIUCC, SWRST);
@@ -368,7 +368,7 @@ mcstop(struct mc_softc *sc)
 	sc->sc_if.if_flags &= ~IFF_RUNNING;
 
 	splx(s);
-	return (0);
+	return 0;
 }
 
 /*
@@ -420,14 +420,14 @@ maceput(struct mc_softc *sc, struct mbuf
 	(*sc->sc_putpacket)(sc, totlen);
 
 	sc->sc_if.if_timer = 5;	/* 5 seconds to watch for failing to transmit */
-	return (totlen);
+	return totlen;
 }
 
 void
 mcintr(void *arg)
 {
 struct mc_softc *sc = arg;
-	u_int8_t ir;
+	uint8_t ir;
 
 	ir = NIC_GET(sc, MACE_IR) & ~NIC_GET(sc, MACE_IMR);
 	if (ir & JAB) {
@@ -467,7 +467,7 @@ struct mc_softc *sc = arg;
 integrate void
 mc_tint(struct mc_softc *sc)
 {
-	u_int8_t /* xmtrc,*/ xmtfs;
+	uint8_t /* xmtrc,*/ xmtfs;
 
 	/* xmtrc = */ NIC_GET(sc, MACE_XMTRC);
 	xmtfs = NIC_GET(sc, MACE_XMTFS);
@@ -594,7 +594,7 @@ mace_get(struct mc_softc *sc, void *pkt,
 
 	MGETHDR(m, M_DONTWAIT, MT_DATA);
 	if (m == 0)
-		return (0);
+		return 0;
 	m_set_rcvif(m, &sc->sc_if);
 	m->m_pkthdr.len = totlen;
 	len = MHLEN;
@@ -627,7 +627,7 @@ mace_get(struct mc_softc *sc, void *pkt,
 		mp = &m->m_next;
 	}
 
-	return (top);
+	return top;
 }
 
 /*
@@ -635,13 +635,13 @@ mace_get(struct mc_softc *sc, void *pkt,
  * address filter.
  */
 void
-mace_calcladrf(struct ethercom *ac, u_int8_t *af)
+mace_calcladrf(struct ethercom *ec, uint8_t *af)
 {
-	struct ifnet *ifp = &ac->ec_if;
+	struct ifnet *ifp = &ec->ec_if;
 	struct ether_multi *enm;
 	u_char *cp;
-	u_int32_t crc;
-	static const u_int32_t crctab[] = {
+	uint32_t crc;
+	static const uint32_t crctab[] = {
 		0x00000000, 0x1db71064, 0x3b6e20c8, 0x26d930ac,
 		0x76dc4190, 0x6b6b51f4, 0x4db26158, 0x5005713c,
 		0xedb88320, 0xf00f9344, 0xd6d6a3e8, 0xcb61b38c,
@@ -658,8 +658,8 @@ mace_calcladrf(struct ethercom *ac, u_in
 	 * the word.
 	 */
 
-	*((u_int32_t *)af) = *((u_int32_t *)af + 1) = 0;
-	ETHER_FIRST_MULTI(step, ac, enm);
+	*((uint32_t *)af) = *((uint32_t *)af + 1) = 0;
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (ETHER_CMP(enm->enm_addrlo, enm->enm_addrhi)) {
 			/*
@@ -693,7 +693,7 @@ mace_calcladrf(struct ethercom *ac, u_in
 
 allmulti:
 	ifp->if_flags |= IFF_ALLMULTI;
-	*((u_int32_t *)af) = *((u_int32_t *)af + 1) = 0xffffffff;
+	*((uint32_t *)af) = *((uint32_t *)af + 1) = 0xffffffff;
 }
 
 static u_char bbr4[] = {0,8,4,12,2,10,6,14,1,9,5,13,3,11,7,15};

Index: src/sys/arch/macppc/dev/am79c950.c
diff -u src/sys/arch/macppc/dev/am79c950.c:1.43 src/sys/arch/macppc/dev/am79c950.c:1.44
--- src/sys/arch/macppc/dev/am79c950.c:1.43	Fri Apr 26 06:33:33 2019
+++ src/sys/arch/macppc/dev/am79c950.c	Thu May 23 10:30:35 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: am79c950.c,v 1.43 2019/04/26 06:33:33 msaitoh Exp $	*/
+/*	$NetBSD: am79c950.c,v 1.44 2019/05/23 10:30:35 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 1997 David Huang <k...@bga.com>
@@ -35,7 +35,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: am79c950.c,v 1.43 2019/04/26 06:33:33 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: am79c950.c,v 1.44 2019/05/23 10:30:35 msaitoh Exp $");
 
 #include "opt_inet.h"
 
@@ -155,8 +155,8 @@ mcsetup(struct mc_softc *sc, uint8_t *ll
 
 	/* Initialize ifmedia structures */
 	ifmedia_init(&sc->sc_media, 0, mc_mediachange, mc_mediastatus);
-	ifmedia_add(&sc->sc_media, IFM_ETHER|IFM_MANUAL, 0, NULL);
-	ifmedia_set(&sc->sc_media, IFM_ETHER|IFM_MANUAL);
+	ifmedia_add(&sc->sc_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
+	ifmedia_set(&sc->sc_media, IFM_ETHER | IFM_MANUAL);
 
 	if_attach(ifp);
 	if_deferred_start_init(ifp, NULL);
@@ -646,9 +646,9 @@ mace_get(struct mc_softc *sc, uint8_t *p
  * address filter.
  */
 void
-mace_calcladrf(struct ethercom *ac, uint8_t *af)
+mace_calcladrf(struct ethercom *ec, uint8_t *af)
 {
-	struct ifnet *ifp = &ac->ec_if;
+	struct ifnet *ifp = &ec->ec_if;
 	struct ether_multi *enm;
 	register u_char *cp, c;
 	register uint32_t crc;
@@ -665,7 +665,7 @@ mace_calcladrf(struct ethercom *ac, uint
 
 	*((uint32_t *)af) = *((uint32_t *)af + 1) = 0;
 
-	ETHER_FIRST_MULTI(step, ac, enm);
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (ETHER_CMP(enm->enm_addrlo, enm->enm_addrhi)) {
 			/*

Index: src/sys/dev/cadence/if_cemac.c
diff -u src/sys/dev/cadence/if_cemac.c:1.18 src/sys/dev/cadence/if_cemac.c:1.19
--- src/sys/dev/cadence/if_cemac.c:1.18	Sun May 19 13:32:35 2019
+++ src/sys/dev/cadence/if_cemac.c	Thu May 23 10:30:35 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_cemac.c,v 1.18 2019/05/19 13:32:35 alnsn Exp $	*/
+/*	$NetBSD: if_cemac.c,v 1.19 2019/05/23 10:30:35 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2015  Genetec Corporation.  All rights reserved.
@@ -40,7 +40,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_cemac.c,v 1.18 2019/05/19 13:32:35 alnsn Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_cemac.c,v 1.19 2019/05/23 10:30:35 msaitoh Exp $");
 
 #include <sys/types.h>
 #include <sys/param.h>
@@ -147,9 +147,9 @@ static void	cemac_setaddr(struct ifnet *
 
 #ifdef	CEMAC_DEBUG
 int cemac_debug = CEMAC_DEBUG;
-#define	DPRINTFN(n,fmt)	if (cemac_debug >= (n)) printf fmt
+#define	DPRINTFN(n, fmt)	if (cemac_debug >= (n)) printf fmt
 #else
-#define	DPRINTFN(n,fmt)
+#define	DPRINTFN(n, fmt)
 #endif
 
 CFATTACH_DECL_NEW(cemac, sizeof(struct cemac_softc),
@@ -200,9 +200,9 @@ cemac_attach_common(device_t self, bus_s
 	u = CEMAC_READ(ETH_TSR);
 	CEMAC_WRITE(ETH_TSR, (u & (ETH_TSR_UND | ETH_TSR_COMP | ETH_TSR_BNQ
 				  | ETH_TSR_IDLE | ETH_TSR_RLE
-				  | ETH_TSR_COL|ETH_TSR_OVR)));
+				  | ETH_TSR_COL | ETH_TSR_OVR)));
 	u = CEMAC_READ(ETH_RSR);
-	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR|ETH_RSR_REC|ETH_RSR_BNA)));
+	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR | ETH_RSR_REC | ETH_RSR_BNA)));
 
 	/* Fetch the Ethernet address from property if set. */
 	enaddr = prop_dictionary_get(device_properties(self), "mac-address");
@@ -282,7 +282,8 @@ cemac_intr(void *arg)
 	int bi;
 
 	imr = ~CEMAC_READ(ETH_IMR);
-	if (!(imr & (ETH_ISR_RCOM|ETH_ISR_TBRE|ETH_ISR_TIDLE|ETH_ISR_RBNA|ETH_ISR_ROVR|ETH_ISR_TCOM))) {
+	if (!(imr & (ETH_ISR_RCOM | ETH_ISR_TBRE | ETH_ISR_TIDLE |
+	    ETH_ISR_RBNA | ETH_ISR_ROVR | ETH_ISR_TCOM))) {
 		// interrupt not enabled, can't be us
 		return 0;
 	}
@@ -382,7 +383,7 @@ cemac_intr(void *arg)
 		if_schedule_deferred_start(ifp);
 #if 0 // reloop
 	irq = CEMAC_READ(IntStsC);
-	if ((irq & (IntSts_RxSQ|IntSts_ECI)) != 0)
+	if ((irq & (IntSts_RxSQ | IntSts_ECI)) != 0)
 		goto begin;
 #endif
 
@@ -396,6 +397,7 @@ cemac_init(struct cemac_softc *sc)
 	bus_dma_segment_t segs;
 	int rsegs, err, i;
 	struct ifnet * ifp = &sc->sc_ethercom.ec_if;
+	struct mii_data * const mii = &sc->sc_mii;
 	uint32_t u;
 #if 0
 	int mdcdiv = DEFAULT_MDCDIV;
@@ -427,9 +429,9 @@ cemac_init(struct cemac_softc *sc)
 	u = CEMAC_READ(ETH_TSR);
 	CEMAC_WRITE(ETH_TSR, (u & (ETH_TSR_UND | ETH_TSR_COMP | ETH_TSR_BNQ
 		    | ETH_TSR_IDLE | ETH_TSR_RLE
-		    | ETH_TSR_COL|ETH_TSR_OVR)));
+		    | ETH_TSR_COL | ETH_TSR_OVR)));
 	u = CEMAC_READ(ETH_RSR);
-	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR|ETH_RSR_REC|ETH_RSR_BNA)));
+	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR | ETH_RSR_REC | ETH_RSR_BNA)));
 
 #if 0
 	if (device_cfdata(sc->sc_dev)->cf_flags)
@@ -459,7 +461,7 @@ cemac_init(struct cemac_softc *sc)
 	if (err == 0) {
 		DPRINTFN(1,("%s: -> bus_dmamem_map\n", __FUNCTION__));
 		err = bus_dmamem_map(sc->sc_dmat, &segs, 1, sc->rbqlen,
-		    &sc->rbqpage, (BUS_DMA_WAITOK|BUS_DMA_COHERENT));
+		    &sc->rbqpage, (BUS_DMA_WAITOK | BUS_DMA_COHERENT));
 	}
 	if (err == 0) {
 		DPRINTFN(1,("%s: -> bus_dmamap_create\n", __FUNCTION__));
@@ -489,7 +491,7 @@ cemac_init(struct cemac_softc *sc)
 	if (err == 0) {
 		DPRINTFN(1,("%s: -> bus_dmamem_map\n", __FUNCTION__));
 		err = bus_dmamem_map(sc->sc_dmat, &segs, 1, sc->tbqlen,
-		    &sc->tbqpage, (BUS_DMA_WAITOK|BUS_DMA_COHERENT));
+		    &sc->tbqpage, (BUS_DMA_WAITOK | BUS_DMA_COHERENT));
 	}
 	if (err == 0) {
 		DPRINTFN(1,("%s: -> bus_dmamap_create\n", __FUNCTION__));
@@ -523,7 +525,7 @@ cemac_init(struct cemac_softc *sc)
 
 	/* Populate the RXQ with mbufs */
 	sc->rxqi = 0;
-	for(i = 0; i < RX_QLEN; i++) {
+	for (i = 0; i < RX_QLEN; i++) {
 		struct mbuf *m;
 
 		err = bus_dmamap_create(sc->sc_dmat, MCLBYTES, 1, MCLBYTES, PAGE_SIZE,
@@ -567,16 +569,16 @@ cemac_init(struct cemac_softc *sc)
 	CEMAC_WRITE(ETH_TBQP, (uint32_t)sc->tbqpage_dsaddr);
 
 	/* Divide HCLK by 32 for MDC clock */
-	sc->sc_ethercom.ec_mii = &sc->sc_mii;
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = cemac_mii_readreg;
-	sc->sc_mii.mii_writereg = cemac_mii_writereg;
-	sc->sc_mii.mii_statchg = cemac_statchg;
-	ifmedia_init(&sc->sc_mii.mii_media, IFM_IMASK, cemac_mediachange,
+	sc->sc_ethercom.ec_mii = mii;
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = cemac_mii_readreg;
+	mii->mii_writereg = cemac_mii_writereg;
+	mii->mii_statchg = cemac_statchg;
+	ifmedia_init(&mii->mii_media, IFM_IMASK, cemac_mediachange,
 	    cemac_mediastatus);
-	mii_attach(sc->sc_dev, &sc->sc_mii, 0xffffffff, MII_PHY_ANY,
+	mii_attach(sc->sc_dev, mii, 0xffffffff, MII_PHY_ANY,
 	    MII_OFFSET_ANY, 0);
-	ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+	ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 
 #if 0
 	// enable / disable interrupts
@@ -593,7 +595,7 @@ cemac_init(struct cemac_softc *sc)
 	 * We can support hardware checksumming.
 	 */
 	ifp->if_capabilities |=
-	    IFCAP_CSUM_IPv4_Tx | IFCAP_CSUM_IPv4_Rx |  
+	    IFCAP_CSUM_IPv4_Tx | IFCAP_CSUM_IPv4_Rx |
 	    IFCAP_CSUM_TCPv4_Tx | IFCAP_CSUM_TCPv4_Rx |
 	    IFCAP_CSUM_UDPv4_Tx | IFCAP_CSUM_UDPv4_Rx |
 	    IFCAP_CSUM_TCPv6_Tx | IFCAP_CSUM_TCPv6_Rx |
@@ -649,7 +651,8 @@ cemac_mii_readreg(device_t self, int phy
 			     | ((phy << ETH_MAN_PHYA_SHIFT) & ETH_MAN_PHYA)
 			     | ((reg << ETH_MAN_REGA_SHIFT) & ETH_MAN_REGA)
 			     | ETH_MAN_CODE_IEEE802_3));
-	while (!(CEMAC_READ(ETH_SR) & ETH_SR_IDLE));
+	while (!(CEMAC_READ(ETH_SR) & ETH_SR_IDLE))
+		;
 
 	*val = CEMAC_READ(ETH_MAN) & ETH_MAN_DATA;
 	return 0;
@@ -667,7 +670,8 @@ cemac_mii_writereg(device_t self, int ph
 			     | ((reg << ETH_MAN_REGA_SHIFT) & ETH_MAN_REGA)
 			     | ETH_MAN_CODE_IEEE802_3
 			     | (val & ETH_MAN_DATA)));
-	while (!(CEMAC_READ(ETH_SR) & ETH_SR_IDLE)) ;
+	while (!(CEMAC_READ(ETH_SR) & ETH_SR_IDLE))
+		;
 
 	return 0;
 }
@@ -742,7 +746,7 @@ cemac_ifioctl(struct ifnet *ifp, u_long 
 	int s, error;
 
 	s = splnet();
-	switch(cmd) {
+	switch (cmd) {
 	default:
 		error = ether_ioctl(ifp, cmd, data);
 		if (error != ENETRESET)
@@ -943,9 +947,9 @@ cemac_ifstop(struct ifnet *ifp, int disa
 	u = CEMAC_READ(ETH_TSR);
 	CEMAC_WRITE(ETH_TSR, (u & (ETH_TSR_UND | ETH_TSR_COMP | ETH_TSR_BNQ
 				  | ETH_TSR_IDLE | ETH_TSR_RLE
-				  | ETH_TSR_COL|ETH_TSR_OVR)));
+				  | ETH_TSR_COL | ETH_TSR_OVR)));
 	u = CEMAC_READ(ETH_RSR);
-	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR|ETH_RSR_REC|ETH_RSR_BNA)));
+	CEMAC_WRITE(ETH_RSR, (u & (ETH_RSR_OVR | ETH_RSR_REC | ETH_RSR_BNA)));
 #endif
 	callout_stop(&sc->cemac_tick_ch);
 
@@ -961,7 +965,7 @@ static void
 cemac_setaddr(struct ifnet *ifp)
 {
 	struct cemac_softc *sc = ifp->if_softc;
-	struct ethercom *ac = &sc->sc_ethercom;
+	struct ethercom *ec = &sc->sc_ethercom;
 	struct ether_multi *enm;
 	struct ether_multistep step;
 	uint8_t ias[3][ETHER_ADDR_LEN];
@@ -984,7 +988,7 @@ cemac_setaddr(struct ifnet *ifp)
 
 	ifp->if_flags &= ~IFF_ALLMULTI;
 
-	ETHER_FIRST_MULTI(step, ac, enm);
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (memcmp(enm->enm_addrlo, enm->enm_addrhi, ETHER_ADDR_LEN)) {
 			/*

Index: src/sys/dev/ic/lance.c
diff -u src/sys/dev/ic/lance.c:1.55 src/sys/dev/ic/lance.c:1.56
--- src/sys/dev/ic/lance.c:1.55	Tue Feb  5 06:17:02 2019
+++ src/sys/dev/ic/lance.c	Thu May 23 10:30:36 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: lance.c,v 1.55 2019/02/05 06:17:02 msaitoh Exp $	*/
+/*	$NetBSD: lance.c,v 1.56 2019/05/23 10:30:36 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 1997, 1998 The NetBSD Foundation, Inc.
@@ -65,7 +65,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: lance.c,v 1.55 2019/02/05 06:17:02 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: lance.c,v 1.56 2019/05/23 10:30:36 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -106,7 +106,7 @@ hide bool lance_shutdown(device_t, int);
 int lance_mediachange(struct ifnet *);
 void lance_mediastatus(struct ifnet *, struct ifmediareq *);
 
-static inline u_int16_t ether_cmp(void *, void *);
+static inline uint16_t ether_cmp(void *, void *);
 
 void lance_stop(struct ifnet *, int);
 int lance_ioctl(struct ifnet *, u_long, void *);
@@ -189,8 +189,8 @@ lance_config(struct lance_softc *sc)
 			   0, NULL);
 		ifmedia_set(&sc->sc_media, sc->sc_defaultmedia);
 	} else {
-		ifmedia_add(&sc->sc_media, IFM_ETHER|IFM_MANUAL, 0, NULL);
-		ifmedia_set(&sc->sc_media, IFM_ETHER|IFM_MANUAL);
+		ifmedia_add(&sc->sc_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
+		ifmedia_set(&sc->sc_media, IFM_ETHER | IFM_MANUAL);
 	}
 
 	switch (sc->sc_memsize) {
@@ -562,9 +562,9 @@ lance_shutdown(device_t self, int howto)
  * Set up the logical address filter.
  */
 void
-lance_setladrf(struct ethercom *ac, uint16_t *af)
+lance_setladrf(struct ethercom *ec, uint16_t *af)
 {
-	struct ifnet *ifp = &ac->ec_if;
+	struct ifnet *ifp = &ec->ec_if;
 	struct ether_multi *enm;
 	uint32_t crc;
 	struct ether_multistep step;
@@ -581,7 +581,7 @@ lance_setladrf(struct ethercom *ac, uint
 		goto allmulti;
 
 	af[0] = af[1] = af[2] = af[3] = 0x0000;
-	ETHER_FIRST_MULTI(step, ac, enm);
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (ETHER_CMP(enm->enm_addrlo, enm->enm_addrhi)) {
 			/*

Index: src/sys/dev/pci/if_txp.c
diff -u src/sys/dev/pci/if_txp.c:1.54 src/sys/dev/pci/if_txp.c:1.55
--- src/sys/dev/pci/if_txp.c:1.54	Tue May  7 15:23:32 2019
+++ src/sys/dev/pci/if_txp.c	Thu May 23 10:30:36 2019
@@ -1,4 +1,4 @@
-/* $NetBSD: if_txp.c,v 1.54 2019/05/07 15:23:32 msaitoh Exp $ */
+/* $NetBSD: if_txp.c,v 1.55 2019/05/23 10:30:36 msaitoh Exp $ */
 
 /*
  * Copyright (c) 2001
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_txp.c,v 1.54 2019/05/07 15:23:32 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_txp.c,v 1.55 2019/05/23 10:30:36 msaitoh Exp $");
 
 #include "opt_inet.h"
 
@@ -313,7 +313,7 @@ txp_attach(device_t parent, device_t sel
 		ifmedia_add(&sc->sc_ifmedia, IFM_ETHER | IFM_100_TX | IFM_FDX,
 			    0, NULL);
 	}
-	ifmedia_add(&sc->sc_ifmedia, IFM_ETHER|IFM_AUTO, 0, NULL);
+	ifmedia_add(&sc->sc_ifmedia, IFM_ETHER | IFM_AUTO, 0, NULL);
 
 	sc->sc_xcvr = TXP_XCVR_AUTO;
 	txp_command(sc, TXP_CMD_XCVR_SELECT, TXP_XCVR_AUTO, 0, 0,
@@ -352,7 +352,7 @@ txp_attach(device_t parent, device_t sel
 	return;
 
 cleanupintr:
-	pci_intr_disestablish(pc,sc->sc_ih);
+	pci_intr_disestablish(pc, sc->sc_ih);
 
 	return;
 
@@ -1262,7 +1262,7 @@ txp_ioctl(struct ifnet *ifp, u_long comm
 	}
 #endif
 
-	switch(command) {
+	switch (command) {
 	case SIOCINITIFADDR:
 		ifp->if_flags |= IFF_UP;
 		txp_init(sc);
@@ -1314,7 +1314,7 @@ txp_ioctl(struct ifnet *ifp, u_long comm
 
 	splx(s);
 
-	return(error);
+	return (error);
 }
 
 void
@@ -1927,7 +1927,7 @@ txp_show_descriptor(void *d)
 void
 txp_set_filter(struct txp_softc *sc)
 {
-	struct ethercom *ac = &sc->sc_arpcom;
+	struct ethercom *ec = &sc->sc_arpcom;
 	struct ifnet *ifp = &sc->sc_arpcom.ec_if;
 	uint32_t crc, carry, hashbit, hash[2];
 	uint16_t filter;
@@ -1952,7 +1952,7 @@ again:
 	else {
 		hash[0] = hash[1] = 0;
 
-		ETHER_FIRST_MULTI(step, ac, enm);
+		ETHER_FIRST_MULTI(step, ec, enm);
 		while (enm != NULL) {
 			if (memcmp(enm->enm_addrlo, enm->enm_addrhi,
 			    ETHER_ADDR_LEN)) {

Index: src/sys/dev/scsipi/if_se.c
diff -u src/sys/dev/scsipi/if_se.c:1.99 src/sys/dev/scsipi/if_se.c:1.100
--- src/sys/dev/scsipi/if_se.c:1.99	Tue Feb  5 06:17:03 2019
+++ src/sys/dev/scsipi/if_se.c	Thu May 23 10:30:36 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_se.c,v 1.99 2019/02/05 06:17:03 msaitoh Exp $	*/
+/*	$NetBSD: if_se.c,v 1.100 2019/05/23 10:30:36 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 1997 Ian W. Dall <ian.d...@dsto.defence.gov.au>
@@ -59,7 +59,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_se.c,v 1.99 2019/02/05 06:17:03 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_se.c,v 1.100 2019/05/23 10:30:36 msaitoh Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_inet.h"
@@ -151,7 +151,7 @@ int se_max_received = 0;	/* Instrumentat
 	static const struct scsi_ctron_ether_generic name
 
 #define	PROTOCMD_DECL_SPECIAL(name) \
-	static const struct __CONCAT(scsi_,name) name
+	static const struct __CONCAT(scsi_, name) name
 
 /* Command initializers for commands using scsi_ctron_ether_generic */
 PROTOCMD_DECL(ctron_ether_send)  = {CTRON_ETHER_SEND, 0, {0,0}, 0};
@@ -202,18 +202,18 @@ static int	se_ioctl(struct ifnet *, u_lo
 static void	sewatchdog(struct ifnet *);
 
 #if 0
-static inline u_int16_t ether_cmp(void *, void *);
+static inline uint16_t ether_cmp(void *, void *);
 #endif
 static void	se_recv(void *);
 static struct mbuf *se_get(struct se_softc *, char *, int);
 static int	se_read(struct se_softc *, char *, int);
 static int	se_reset(struct se_softc *);
 static int	se_add_proto(struct se_softc *, int);
-static int	se_get_addr(struct se_softc *, u_int8_t *);
+static int	se_get_addr(struct se_softc *, uint8_t *);
 static int	se_set_media(struct se_softc *, int);
 static int	se_init(struct se_softc *);
-static int	se_set_multi(struct se_softc *, u_int8_t *);
-static int	se_remove_multi(struct se_softc *, u_int8_t *);
+static int	se_set_multi(struct se_softc *, uint8_t *);
+static int	se_remove_multi(struct se_softc *, uint8_t *);
 #if 0
 static int	sc_set_all_multi(struct se_softc *, int);
 #endif
@@ -273,12 +273,12 @@ const struct scsipi_inquiry_pattern se_p
  * unrolled for speed.
  * Note: use this like memcmp()
  */
-static inline u_int16_t
+static inline uint16_t
 ether_cmp(void *one, void *two)
 {
-	u_int16_t *a = (u_int16_t *) one;
-	u_int16_t *b = (u_int16_t *) two;
-	u_int16_t diff;
+	uint16_t *a = (uint16_t *) one;
+	uint16_t *b = (uint16_t *) two;
+	uint16_t diff;
 
 	diff = (a[0] - b[0]) | (a[1] - b[1]) | (a[2] - b[2]);
 
@@ -311,7 +311,7 @@ seattach(device_t parent, device_t self,
 	struct scsipibus_attach_args *sa = aux;
 	struct scsipi_periph *periph = sa->sa_periph;
 	struct ifnet *ifp = &sc->sc_ethercom.ec_if;
-	u_int8_t myaddr[ETHER_ADDR_LEN];
+	uint8_t myaddr[ETHER_ADDR_LEN];
 	int rv;
 
 	sc->sc_dev = self;
@@ -322,7 +322,6 @@ seattach(device_t parent, device_t self,
 	callout_init(&sc->sc_ifstart_ch, 0);
 	callout_init(&sc->sc_recv_ch, 0);
 
-
 	/*
 	 * Store information needed to contact our base driver
 	 */
@@ -426,7 +425,7 @@ se_ifstart(struct ifnet *ifp)
 	u_char *cp;
 
 	/* Don't transmit if interface is busy or not running */
-	if ((ifp->if_flags & (IFF_RUNNING|IFF_OACTIVE)) != IFF_RUNNING)
+	if ((ifp->if_flags & (IFF_RUNNING | IFF_OACTIVE)) != IFF_RUNNING)
 		return;
 
 	IFQ_DEQUEUE(&ifp->if_snd, m0);
@@ -470,7 +469,7 @@ se_ifstart(struct ifnet *ifp)
 	error = se_scsipi_cmd(sc->sc_periph,
 	    (void *)&send_cmd, sizeof(send_cmd),
 	    sc->sc_tbuf, len, SERETRIES,
-	    SETIMEOUT, NULL, XS_CTL_NOSLEEP|XS_CTL_ASYNC|XS_CTL_DATA_OUT);
+	    SETIMEOUT, NULL, XS_CTL_NOSLEEP | XS_CTL_ASYNC | XS_CTL_DATA_OUT);
 	if (error) {
 		aprint_error_dev(sc->sc_dev, "not queued, error %d\n", error);
 		ifp->if_oerrors++;
@@ -496,7 +495,7 @@ sedone(struct scsipi_xfer *xs, int error
 	int s;
 
 	s = splnet();
-	if(IS_SEND(cmd)) {
+	if (IS_SEND(cmd)) {
 		if (xs->error == XS_BUSY) {
 			printf("se: busy, retry txmit\n");
 			callout_reset(&sc->sc_ifstart_ch, hz,
@@ -507,7 +506,7 @@ sedone(struct scsipi_xfer *xs, int error
 			 * sestart (through scsipi_free_xs).
 			 */
 		}
-	} else if(IS_RECV(cmd)) {
+	} else if (IS_RECV(cmd)) {
 		/* RECV complete */
 		/* pass data up. reschedule a recv */
 		/* scsipi_free_xs will call start. Harmless. */
@@ -563,7 +562,7 @@ se_recv(void *v)
 	error = se_scsipi_cmd(sc->sc_periph,
 	    (void *)&recv_cmd, sizeof(recv_cmd),
 	    sc->sc_rbuf, RBUF_LEN, SERETRIES, SETIMEOUT, NULL,
-	    XS_CTL_NOSLEEP|XS_CTL_ASYNC|XS_CTL_DATA_IN);
+	    XS_CTL_NOSLEEP | XS_CTL_ASYNC | XS_CTL_DATA_IN);
 	if (error)
 		callout_reset(&sc->sc_recv_ch, se_poll, se_recv, (void *)sc);
 }
@@ -719,7 +718,7 @@ se_add_proto(struct se_softc *sc, int pr
 {
 	int error;
 	struct scsi_ctron_ether_generic add_proto_cmd;
-	u_int8_t data[2];
+	uint8_t data[2];
 	_lto2b(proto, data);
 #ifdef SEDEBUG
 	if (sc->sc_debug)
@@ -736,7 +735,7 @@ se_add_proto(struct se_softc *sc, int pr
 }
 
 static int
-se_get_addr(struct se_softc *sc, u_int8_t *myaddr)
+se_get_addr(struct se_softc *sc, uint8_t *myaddr)
 {
 	int error;
 	struct scsi_ctron_ether_generic get_addr_cmd;
@@ -828,7 +827,7 @@ se_init(struct se_softc *sc)
 		return (error);
 #endif
 
-	if ((ifp->if_flags & (IFF_RUNNING|IFF_UP)) == IFF_UP) {
+	if ((ifp->if_flags & (IFF_RUNNING | IFF_UP)) == IFF_UP) {
 		ifp->if_flags |= IFF_RUNNING;
 		se_recv(sc);
 		ifp->if_flags &= ~IFF_OACTIVE;
@@ -838,7 +837,7 @@ se_init(struct se_softc *sc)
 }
 
 static int
-se_set_multi(struct se_softc *sc, u_int8_t *addr)
+se_set_multi(struct se_softc *sc, uint8_t *addr)
 {
 	struct scsi_ctron_ether_generic set_multi_cmd;
 	int error;
@@ -859,7 +858,7 @@ se_set_multi(struct se_softc *sc, u_int8
 }
 
 static int
-se_remove_multi(struct se_softc *sc, u_int8_t *addr)
+se_remove_multi(struct se_softc *sc, uint8_t *addr)
 {
 	struct scsi_ctron_ether_generic remove_multi_cmd;
 	int error;
@@ -884,12 +883,12 @@ static int
 sc_set_all_multi(struct se_softc *sc, int set)
 {
 	int error = 0;
-	u_int8_t *addr;
-	struct ethercom *ac = &sc->sc_ethercom;
+	uint8_t *addr;
+	struct ethercom *ec = &sc->sc_ethercom;
 	struct ether_multi *enm;
 	struct ether_multistep step;
 
-	ETHER_FIRST_MULTI(step, ac, enm);
+	ETHER_FIRST_MULTI(step, ec, enm);
 	while (enm != NULL) {
 		if (ETHER_CMP(enm->enm_addrlo, enm->enm_addrhi)) {
 			/*
@@ -981,7 +980,7 @@ se_ioctl(struct ifnet *ifp, u_long cmd, 
 		if ((error = ifioctl_common(ifp, cmd, data)) != 0)
 			break;
 		/* XXX re-use ether_ioctl() */
-		switch (ifp->if_flags & (IFF_UP|IFF_RUNNING)) {
+		switch (ifp->if_flags & (IFF_UP | IFF_RUNNING)) {
 		case IFF_RUNNING:
 			/*
 			 * If interface is marked down and it is running, then

Reply via email to