Module Name:    src
Committed By:   msaitoh
Date:           Wed May 29 06:17:28 UTC 2019

Modified Files:
        src/sys/arch/arm/omap: if_cpsw.c
        src/sys/arch/arm/ti: if_cpsw.c
        src/sys/arch/mips/ralink: ralink_eth.c
        src/sys/arch/newsmips/apbus: if_tlp_ap.c
        src/sys/dev/ic: ax88190.c dl10019.c elink3.c
        src/sys/dev/pci: if_tlp_pci.c

Log Message:
No functional change:
 - Simplify MII structure initialization and reference.
 - KNF


To generate a diff of this commit:
cvs rdiff -u -r1.24 -r1.25 src/sys/arch/arm/omap/if_cpsw.c
cvs rdiff -u -r1.5 -r1.6 src/sys/arch/arm/ti/if_cpsw.c
cvs rdiff -u -r1.16 -r1.17 src/sys/arch/mips/ralink/ralink_eth.c
cvs rdiff -u -r1.14 -r1.15 src/sys/arch/newsmips/apbus/if_tlp_ap.c
cvs rdiff -u -r1.14 -r1.15 src/sys/dev/ic/ax88190.c
cvs rdiff -u -r1.13 -r1.14 src/sys/dev/ic/dl10019.c
cvs rdiff -u -r1.148 -r1.149 src/sys/dev/ic/elink3.c
cvs rdiff -u -r1.125 -r1.126 src/sys/dev/pci/if_tlp_pci.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/omap/if_cpsw.c
diff -u src/sys/arch/arm/omap/if_cpsw.c:1.24 src/sys/arch/arm/omap/if_cpsw.c:1.25
--- src/sys/arch/arm/omap/if_cpsw.c:1.24	Wed May 29 05:05:24 2019
+++ src/sys/arch/arm/omap/if_cpsw.c	Wed May 29 06:17:27 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_cpsw.c,v 1.24 2019/05/29 05:05:24 msaitoh Exp $	*/
+/*	$NetBSD: if_cpsw.c,v 1.25 2019/05/29 06:17:27 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2013 Jonathan A. Kollasch
@@ -53,7 +53,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(1, "$NetBSD: if_cpsw.c,v 1.24 2019/05/29 05:05:24 msaitoh Exp $");
+__KERNEL_RCSID(1, "$NetBSD: if_cpsw.c,v 1.25 2019/05/29 06:17:27 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -394,6 +394,7 @@ cpsw_attach(device_t parent, device_t se
 	prop_dictionary_t dict = device_properties(self);
 	struct ethercom * const ec = &sc->sc_ec;
 	struct ifnet * const ifp = &ec->ec_if;
+	struct mii_data * const mii = &sc->sc_mii;
 	int error;
 	u_int i;
 
@@ -508,7 +509,7 @@ cpsw_attach(device_t parent, device_t se
 	bus_dmamap_create(sc->sc_bdt, ETHER_MIN_LEN, 1, ETHER_MIN_LEN, 0,
 	    BUS_DMA_WAITOK, &sc->sc_txpad_dm);
 	bus_dmamap_load(sc->sc_bdt, sc->sc_txpad_dm, sc->sc_txpad,
-	    ETHER_MIN_LEN, NULL, BUS_DMA_WAITOK|BUS_DMA_WRITE);
+	    ETHER_MIN_LEN, NULL, BUS_DMA_WAITOK | BUS_DMA_WRITE);
 	bus_dmamap_sync(sc->sc_bdt, sc->sc_txpad_dm, 0, ETHER_MIN_LEN,
 	    BUS_DMASYNC_PREWRITE);
 
@@ -528,14 +529,13 @@ cpsw_attach(device_t parent, device_t se
 
 	cpsw_stop(ifp, 0);
 
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = cpsw_mii_readreg;
-	sc->sc_mii.mii_writereg = cpsw_mii_writereg;
-	sc->sc_mii.mii_statchg = cpsw_mii_statchg;
-
-	sc->sc_ec.ec_mii = &sc->sc_mii;
-	ifmedia_init(&sc->sc_mii.mii_media, 0, ether_mediachange,
-	    ether_mediastatus);
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = cpsw_mii_readreg;
+	mii->mii_writereg = cpsw_mii_writereg;
+	mii->mii_statchg = cpsw_mii_statchg;
+
+	sc->sc_ec.ec_mii = mii;
+	ifmedia_init(&mii->mii_media, 0, ether_mediachange, ether_mediastatus);
 
 	/* Initialize MDIO */
 	cpsw_write_4(sc, MDIOCONTROL,
@@ -543,13 +543,12 @@ cpsw_attach(device_t parent, device_t se
 	/* Clear ALE */
 	cpsw_write_4(sc, CPSW_ALE_CONTROL, ALECTL_CLEAR_TABLE);
 
-	mii_attach(self, &sc->sc_mii, 0xffffffff, MII_PHY_ANY, 0, 0);
-	if (LIST_FIRST(&sc->sc_mii.mii_phys) == NULL) {
+	mii_attach(self, mii, 0xffffffff, MII_PHY_ANY, 0, 0);
+	if (LIST_FIRST(&mii->mii_phys) == NULL) {
 		aprint_error_dev(self, "no PHY found!\n");
 		sc->sc_phy_has_1000t = false;
-		ifmedia_add(&sc->sc_mii.mii_media,
-		    IFM_ETHER|IFM_MANUAL, 0, NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_MANUAL);
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_MANUAL);
 	} else {
 		sc->sc_phy_has_1000t = cpsw_phy_has_1000t(sc);
 		if (sc->sc_phy_has_1000t) {
@@ -564,7 +563,7 @@ cpsw_attach(device_t parent, device_t se
 			    GMIISEL_GMII1_SEL(RGMII_MODE));
 		}
 
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 	}
 
 	if_attach(ifp);
@@ -597,7 +596,7 @@ cpsw_start(struct ifnet *ifp)
 	KERNHIST_FUNC(__func__);
 	KERNHIST_CALLED_5(cpswhist, (uintptr_t)sc, 0, 0, 0);
 
-	if (__predict_false((ifp->if_flags & (IFF_RUNNING|IFF_OACTIVE)) !=
+	if (__predict_false((ifp->if_flags & (IFF_RUNNING | IFF_OACTIVE)) !=
 	    IFF_RUNNING)) {
 		return;
 	}
@@ -838,7 +837,7 @@ cpsw_new_rxbuf(struct cpsw_softc * const
 	rdp->rx_mb[i] = m;
 
 	error = bus_dmamap_load_mbuf(sc->sc_bdt, rdp->rx_dm[i], rdp->rx_mb[i],
-	    BUS_DMA_READ|BUS_DMA_NOWAIT);
+	    BUS_DMA_READ | BUS_DMA_NOWAIT);
 	if (error) {
 		device_printf(sc->sc_dev, "can't load rx DMA map %d: %d\n",
 		    i, error);
@@ -1076,7 +1075,7 @@ cpsw_stop(struct ifnet *ifp, int disable
 		rdp->tx_mb[i] = NULL;
 	}
 
-	ifp->if_flags &= ~(IFF_RUNNING|IFF_OACTIVE);
+	ifp->if_flags &= ~(IFF_RUNNING | IFF_OACTIVE);
 	ifp->if_timer = 0;
 
 	if (!disable)
@@ -1152,8 +1151,8 @@ cpsw_rxintr(void *arg)
 			return 1;
 		}
 
-		if ((dw[3] & (CPDMA_BD_SOP|CPDMA_BD_EOP)) !=
-		    (CPDMA_BD_SOP|CPDMA_BD_EOP)) {
+		if ((dw[3] & (CPDMA_BD_SOP | CPDMA_BD_EOP)) !=
+		    (CPDMA_BD_SOP | CPDMA_BD_EOP)) {
 			//Debugger();
 		}
 

Index: src/sys/arch/arm/ti/if_cpsw.c
diff -u src/sys/arch/arm/ti/if_cpsw.c:1.5 src/sys/arch/arm/ti/if_cpsw.c:1.6
--- src/sys/arch/arm/ti/if_cpsw.c:1.5	Wed May 29 05:05:24 2019
+++ src/sys/arch/arm/ti/if_cpsw.c	Wed May 29 06:17:27 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_cpsw.c,v 1.5 2019/05/29 05:05:24 msaitoh Exp $	*/
+/*	$NetBSD: if_cpsw.c,v 1.6 2019/05/29 06:17:27 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2013 Jonathan A. Kollasch
@@ -53,7 +53,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(1, "$NetBSD: if_cpsw.c,v 1.5 2019/05/29 05:05:24 msaitoh Exp $");
+__KERNEL_RCSID(1, "$NetBSD: if_cpsw.c,v 1.6 2019/05/29 06:17:27 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -401,6 +401,7 @@ cpsw_attach(device_t parent, device_t se
 	prop_dictionary_t dict = device_properties(self);
 	struct ethercom * const ec = &sc->sc_ec;
 	struct ifnet * const ifp = &ec->ec_if;
+	struct mii_data * const mii = &sc->sc_mii;
 	const int phandle = faa->faa_phandle;
 	bus_addr_t addr;
 	bus_size_t size;
@@ -544,7 +545,7 @@ cpsw_attach(device_t parent, device_t se
 	bus_dmamap_create(sc->sc_bdt, ETHER_MIN_LEN, 1, ETHER_MIN_LEN, 0,
 	    BUS_DMA_WAITOK, &sc->sc_txpad_dm);
 	bus_dmamap_load(sc->sc_bdt, sc->sc_txpad_dm, sc->sc_txpad,
-	    ETHER_MIN_LEN, NULL, BUS_DMA_WAITOK|BUS_DMA_WRITE);
+	    ETHER_MIN_LEN, NULL, BUS_DMA_WAITOK | BUS_DMA_WRITE);
 	bus_dmamap_sync(sc->sc_bdt, sc->sc_txpad_dm, 0, ETHER_MIN_LEN,
 	    BUS_DMASYNC_PREWRITE);
 
@@ -564,14 +565,13 @@ cpsw_attach(device_t parent, device_t se
 
 	cpsw_stop(ifp, 0);
 
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = cpsw_mii_readreg;
-	sc->sc_mii.mii_writereg = cpsw_mii_writereg;
-	sc->sc_mii.mii_statchg = cpsw_mii_statchg;
-
-	sc->sc_ec.ec_mii = &sc->sc_mii;
-	ifmedia_init(&sc->sc_mii.mii_media, 0, ether_mediachange,
-	    ether_mediastatus);
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = cpsw_mii_readreg;
+	mii->mii_writereg = cpsw_mii_writereg;
+	mii->mii_statchg = cpsw_mii_statchg;
+
+	sc->sc_ec.ec_mii = mii;
+	ifmedia_init(&mii->mii_media, 0, ether_mediachange, ether_mediastatus);
 
 	/* Initialize MDIO */
 	cpsw_write_4(sc, MDIOCONTROL,
@@ -579,13 +579,12 @@ cpsw_attach(device_t parent, device_t se
 	/* Clear ALE */
 	cpsw_write_4(sc, CPSW_ALE_CONTROL, ALECTL_CLEAR_TABLE);
 
-	mii_attach(self, &sc->sc_mii, 0xffffffff, MII_PHY_ANY, 0, 0);
-	if (LIST_FIRST(&sc->sc_mii.mii_phys) == NULL) {
+	mii_attach(self, mii, 0xffffffff, MII_PHY_ANY, 0, 0);
+	if (LIST_FIRST(&mii->mii_phys) == NULL) {
 		aprint_error_dev(self, "no PHY found!\n");
 		sc->sc_phy_has_1000t = false;
-		ifmedia_add(&sc->sc_mii.mii_media,
-		    IFM_ETHER|IFM_MANUAL, 0, NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_MANUAL);
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_MANUAL);
 	} else {
 		sc->sc_phy_has_1000t = cpsw_phy_has_1000t(sc);
 		if (sc->sc_phy_has_1000t) {
@@ -602,7 +601,7 @@ cpsw_attach(device_t parent, device_t se
 #endif
 		}
 
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 	}
 
 	if_attach(ifp);
@@ -635,7 +634,7 @@ cpsw_start(struct ifnet *ifp)
 	KERNHIST_FUNC(__func__);
 	KERNHIST_CALLED_5(cpswhist, sc, 0, 0, 0);
 
-	if (__predict_false((ifp->if_flags & (IFF_RUNNING|IFF_OACTIVE)) !=
+	if (__predict_false((ifp->if_flags & (IFF_RUNNING | IFF_OACTIVE)) !=
 	    IFF_RUNNING)) {
 		return;
 	}
@@ -876,7 +875,7 @@ cpsw_new_rxbuf(struct cpsw_softc * const
 	rdp->rx_mb[i] = m;
 
 	error = bus_dmamap_load_mbuf(sc->sc_bdt, rdp->rx_dm[i], rdp->rx_mb[i],
-	    BUS_DMA_READ|BUS_DMA_NOWAIT);
+	    BUS_DMA_READ | BUS_DMA_NOWAIT);
 	if (error) {
 		device_printf(sc->sc_dev, "can't load rx DMA map %d: %d\n",
 		    i, error);
@@ -1114,7 +1113,7 @@ cpsw_stop(struct ifnet *ifp, int disable
 		rdp->tx_mb[i] = NULL;
 	}
 
-	ifp->if_flags &= ~(IFF_RUNNING|IFF_OACTIVE);
+	ifp->if_flags &= ~(IFF_RUNNING | IFF_OACTIVE);
 	ifp->if_timer = 0;
 
 	if (!disable)
@@ -1190,8 +1189,8 @@ cpsw_rxintr(void *arg)
 			return 1;
 		}
 
-		if ((dw[3] & (CPDMA_BD_SOP|CPDMA_BD_EOP)) !=
-		    (CPDMA_BD_SOP|CPDMA_BD_EOP)) {
+		if ((dw[3] & (CPDMA_BD_SOP | CPDMA_BD_EOP)) !=
+		    (CPDMA_BD_SOP | CPDMA_BD_EOP)) {
 			//Debugger();
 		}
 

Index: src/sys/arch/mips/ralink/ralink_eth.c
diff -u src/sys/arch/mips/ralink/ralink_eth.c:1.16 src/sys/arch/mips/ralink/ralink_eth.c:1.17
--- src/sys/arch/mips/ralink/ralink_eth.c:1.16	Wed May 29 05:06:39 2019
+++ src/sys/arch/mips/ralink/ralink_eth.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: ralink_eth.c,v 1.16 2019/05/29 05:06:39 msaitoh Exp $	*/
+/*	$NetBSD: ralink_eth.c,v 1.17 2019/05/29 06:17:28 msaitoh Exp $	*/
 /*-
  * Copyright (c) 2011 CradlePoint Technology, Inc.
  * All rights reserved.
@@ -29,7 +29,7 @@
 /* ralink_eth.c -- Ralink Ethernet Driver */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ralink_eth.c,v 1.16 2019/05/29 05:06:39 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ralink_eth.c,v 1.17 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/bus.h>
@@ -311,6 +311,7 @@ ralink_eth_attach(device_t parent, devic
 {
 	ralink_eth_softc_t * const sc = device_private(self);
 	const struct mainbus_attach_args *ma = aux;
+	struct mii_data *mii = &sc->sc_mii;
 	int error;
 	int i;
 
@@ -478,30 +479,28 @@ ralink_eth_attach(device_t parent, devic
 	 * Initialize our media structures.
 	 * This may probe the PHY, if present.
 	 */
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = ralink_eth_mii_read;
-	sc->sc_mii.mii_writereg = ralink_eth_mii_write;
-	sc->sc_mii.mii_statchg = ralink_eth_mii_statchg;
-	sc->sc_ethercom.ec_mii = &sc->sc_mii;
-	ifmedia_init(&sc->sc_mii.mii_media, 0, ether_mediachange,
-	    ether_mediastatus);
-	mii_attach(sc->sc_dev, &sc->sc_mii, ~0, MII_PHY_ANY, MII_OFFSET_ANY,
-	    MIIF_FORCEANEG|MIIF_DOPAUSE|MIIF_NOISOLATE);
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = ralink_eth_mii_read;
+	mii->mii_writereg = ralink_eth_mii_write;
+	mii->mii_statchg = ralink_eth_mii_statchg;
+	sc->sc_ethercom.ec_mii = mii;
+	ifmedia_init(mii->mii_media, 0, ether_mediachange, ether_mediastatus);
+	mii_attach(sc->sc_dev, mii, ~0, MII_PHY_ANY, MII_OFFSET_ANY,
+	    MIIF_FORCEANEG | MIIF_DOPAUSE | MIIF_NOISOLATE);
 
-	if (LIST_EMPTY(&sc->sc_mii.mii_phys)) {
+	if (LIST_EMPTY(&mii->mii_phys)) {
 #if 1
-		ifmedia_add(&sc->sc_mii.mii_media, IFM_ETHER|IFM_1000_T|
-		    IFM_FDX|IFM_ETH_RXPAUSE|IFM_ETH_TXPAUSE, 0, NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_1000_T|
-		    IFM_FDX|IFM_ETH_RXPAUSE|IFM_ETH_TXPAUSE);
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_1000_T |
+		    IFM_FDX | IFM_ETH_RXPAUSE | IFM_ETH_TXPAUSE, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_1000_T |
+		    IFM_FDX | IFM_ETH_RXPAUSE | IFM_ETH_TXPAUSE);
 #else
-		ifmedia_add(&sc->sc_mii.mii_media,
-		    IFM_ETHER|IFM_MANUAL, 0, NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_MANUAL);
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_MANUAL, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_MANUAL);
 #endif
 	} else {
 		/* Ensure we mask ok for the switch multiple phy's */
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 	}
 
 	ifp->if_softc = sc;
@@ -862,7 +861,7 @@ ralink_eth_hw_init(ralink_eth_softc_t *s
 	 */
 	bus_dmamap_sync(sc->sc_dmat, sc->sc_pdmamap,
 	    (int)&sc->sc_txdesc - (int)sc->sc_descs, sizeof(sc->sc_txdesc),
-	    BUS_DMASYNC_PREREAD|BUS_DMASYNC_PREWRITE);
+	    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 
 	/* Initialize the RX descriptor ring */
 	memset(sc->sc_rxdesc, 0, sizeof(sc->sc_rxdesc));
@@ -879,7 +878,7 @@ ralink_eth_hw_init(ralink_eth_softc_t *s
 	 */
 	bus_dmamap_sync(sc->sc_dmat, sc->sc_pdmamap,
 	    (int)&sc->sc_rxdesc - (int)sc->sc_descs, sizeof(sc->sc_rxdesc),
-	    BUS_DMASYNC_PREREAD|BUS_DMASYNC_PREWRITE);
+	    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 
 	/* Clear the PDMA state */
 	r = fe_read(sc, RA_FE_PDMA_GLOBAL_CFG);
@@ -1130,7 +1129,7 @@ ralink_eth_add_rxbuf(ralink_eth_softc_t 
 	rxs->rxs_mbuf = m;
 
 	error = bus_dmamap_load(sc->sc_dmat, rxs->rxs_dmamap, m->m_ext.ext_buf,
-	    m->m_ext.ext_size, NULL, BUS_DMA_READ|BUS_DMA_NOWAIT);
+	    m->m_ext.ext_size, NULL, BUS_DMA_READ | BUS_DMA_NOWAIT);
 	if (error) {
 		aprint_error_dev(sc->sc_dev, "can't load rx DMA map %d, "
 		    "error=%d\n", idx, error);
@@ -1163,7 +1162,7 @@ ralink_eth_start(struct ifnet *ifp)
 	int error;
 	int s;
 
-	if ((ifp->if_flags & (IFF_RUNNING|IFF_OACTIVE)) != IFF_RUNNING)
+	if ((ifp->if_flags & (IFF_RUNNING | IFF_OACTIVE)) != IFF_RUNNING)
 		return;
 
 	s = splnet();
@@ -1195,7 +1194,7 @@ ralink_eth_start(struct ifnet *ifp)
 		 * or does the DMA map load fail?
 		 */
 		if (bus_dmamap_load_mbuf(sc->sc_dmat, dmamap, m0,
-		    BUS_DMA_WRITE|BUS_DMA_NOWAIT) != 0) {
+		    BUS_DMA_WRITE | BUS_DMA_NOWAIT) != 0) {
 
 			/* Allocate a new mbuf for re-alignment */
 			MGETHDR(m, M_DONTWAIT, MT_DATA);
@@ -1217,7 +1216,7 @@ ralink_eth_start(struct ifnet *ifp)
 			m_copydata(m0, 0, m0->m_pkthdr.len, mtod(m, void *));
 			m->m_pkthdr.len = m->m_len = m0->m_pkthdr.len;
 			error = bus_dmamap_load_mbuf(sc->sc_dmat, dmamap, m,
-			    BUS_DMA_WRITE|BUS_DMA_NOWAIT);
+			    BUS_DMA_WRITE | BUS_DMA_NOWAIT);
 			if (error) {
 				aprint_error_dev(sc->sc_dev,
 				    "unable to load Tx buffer error=%d\n",
@@ -1267,7 +1266,7 @@ ralink_eth_start(struct ifnet *ifp)
 		bus_dmamap_sync(sc->sc_dmat, sc->sc_pdmamap,
 		    (int)&sc->sc_txdesc[tx_cpu_idx] - (int)sc->sc_descs,
 		    sizeof(struct ralink_tx_desc),
-		    BUS_DMASYNC_PREREAD|BUS_DMASYNC_PREWRITE);
+		    BUS_DMASYNC_PREREAD | BUS_DMASYNC_PREWRITE);
 
 		/*
 		 * Store a pointer to the packet so we can free it later,
@@ -1425,7 +1424,7 @@ ralink_eth_intr(void *arg)
 			return (n != 0);
 		}
 
-		if (status & (RA_FE_INT_RX_DONE_INT1|RA_FE_INT_RX_DONE_INT0))
+		if (status & (RA_FE_INT_RX_DONE_INT1 | RA_FE_INT_RX_DONE_INT0))
 			ralink_eth_rxintr(sc);
 
 		if (status & (RA_FE_INT_TX_DONE_INT3 | RA_FE_INT_TX_DONE_INT2 |
@@ -1477,7 +1476,7 @@ ralink_eth_rxintr(ralink_eth_softc_t *sc
 		bus_dmamap_sync(sc->sc_dmat, sc->sc_pdmamap,
 		    (int)&sc->sc_rxdesc[rx_cpu_idx] - (int)sc->sc_descs,
 		    sizeof(struct ralink_rx_desc),
-		    BUS_DMASYNC_POSTREAD|BUS_DMASYNC_POSTWRITE);
+		    BUS_DMASYNC_POSTREAD | BUS_DMASYNC_POSTWRITE);
 
 		RALINK_DEBUG(RALINK_DEBUG_REG,"rx(%d) 0x%08x: 0x%08x\n",
 		    rx_cpu_idx, (int)&sc->sc_rxdesc[rx_cpu_idx].data_ptr,
@@ -1593,7 +1592,7 @@ ralink_eth_txintr(ralink_eth_softc_t *sc
 		bus_dmamap_sync(sc->sc_dmat, sc->sc_pdmamap,
 		    (int)&sc->sc_txdesc[txs->txs_idx] - (int)sc->sc_descs,
 		    sizeof(struct ralink_tx_desc),
-		    BUS_DMASYNC_POSTREAD|BUS_DMASYNC_POSTWRITE);
+		    BUS_DMASYNC_POSTREAD | BUS_DMASYNC_POSTWRITE);
 
 		RALINK_DEBUG(RALINK_DEBUG_REG,"-tx(%d) 0x%08x: 0x%08x\n",
 		    txs->txs_idx, (int)&sc->sc_txdesc[txs->txs_idx].data_ptr0,

Index: src/sys/arch/newsmips/apbus/if_tlp_ap.c
diff -u src/sys/arch/newsmips/apbus/if_tlp_ap.c:1.14 src/sys/arch/newsmips/apbus/if_tlp_ap.c:1.15
--- src/sys/arch/newsmips/apbus/if_tlp_ap.c:1.14	Sun Sep 30 14:23:24 2018
+++ src/sys/arch/newsmips/apbus/if_tlp_ap.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_tlp_ap.c,v 1.14 2018/09/30 14:23:24 tsutsui Exp $	*/
+/*	$NetBSD: if_tlp_ap.c,v 1.15 2019/05/29 06:17:28 msaitoh Exp $	*/
 
 /*
  * Copyright (c) 2000 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_tlp_ap.c,v 1.14 2018/09/30 14:23:24 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_tlp_ap.c,v 1.15 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/device.h>
@@ -238,19 +238,19 @@ static void
 tlp_ap_tmsw_init(struct tulip_softc *sc)
 {
 	struct ifnet *ifp = &sc->sc_ethercom.ec_if;
+	struct mii_data *mii = &sc->sc_mii;
 
 	sc->sc_preinit = tlp_ap_preinit;
 
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = NULL;
-	sc->sc_mii.mii_writereg = NULL;
-	sc->sc_mii.mii_statchg = sc->sc_statchg;
-	ifmedia_init(&sc->sc_mii.mii_media, 0, tlp_mediachange,
-	    tlp_mediastatus);
-	ifmedia_add(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_TX, 0, NULL);
-	ifmedia_add(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_TX|IFM_FDX, 0,
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = NULL;
+	mii->mii_writereg = NULL;
+	mii->mii_statchg = sc->sc_statchg;
+	ifmedia_init(&mii->mii_media, 0, tlp_mediachange, tlp_mediastatus);
+	ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_100_TX, 0, NULL);
+	ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_100_TX | IFM_FDX, 0,
 	    NULL);
-	ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_TX);
+	ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_100_TX);
 }
 
 static void
@@ -272,7 +272,7 @@ tlp_ap_setmedia(struct tulip_softc *sc)
 	sc->sc_mii.mii_media_active = sc->sc_mii.mii_media.ifm_cur->ifm_media;
 
 	if (ifp->if_flags & IFF_UP)
-		tlp_idle(sc, OPMODE_ST|OPMODE_SR);
+		tlp_idle(sc, OPMODE_ST | OPMODE_SR);
 	if (sc->sc_mii.mii_media_active & IFM_FDX)
 		sc->sc_opmode |= OPMODE_FD;
 	else

Index: src/sys/dev/ic/ax88190.c
diff -u src/sys/dev/ic/ax88190.c:1.14 src/sys/dev/ic/ax88190.c:1.15
--- src/sys/dev/ic/ax88190.c:1.14	Wed May 29 05:06:40 2019
+++ src/sys/dev/ic/ax88190.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: ax88190.c,v 1.14 2019/05/29 05:06:40 msaitoh Exp $	*/
+/*	$NetBSD: ax88190.c,v 1.15 2019/05/29 06:17:28 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 2001 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ax88190.c,v 1.14 2019/05/29 05:06:40 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ax88190.c,v 1.15 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -84,23 +84,23 @@ void
 ax88190_media_init(struct dp8390_softc *sc)
 {
 	struct ifnet *ifp = &sc->sc_ec.ec_if;
+	struct mii_data *mii = &sc->sc_mii;
 
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = ax88190_mii_readreg;
-	sc->sc_mii.mii_writereg = ax88190_mii_writereg;
-	sc->sc_mii.mii_statchg = ax88190_mii_statchg;
-	ifmedia_init(&sc->sc_mii.mii_media, IFM_IMASK, dp8390_mediachange,
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = ax88190_mii_readreg;
+	mii->mii_writereg = ax88190_mii_writereg;
+	mii->mii_statchg = ax88190_mii_statchg;
+	ifmedia_init(&mii->mii_media, IFM_IMASK, dp8390_mediachange,
 	    dp8390_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);
 
-	if (LIST_FIRST(&sc->sc_mii.mii_phys) == NULL) {
-		ifmedia_add(&sc->sc_mii.mii_media, IFM_ETHER|IFM_NONE, 0,
-		    NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_NONE);
+	if (LIST_FIRST(&mii->mii_phys) == NULL) {
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_NONE, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_NONE);
 	} else
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 }
 
 void

Index: src/sys/dev/ic/dl10019.c
diff -u src/sys/dev/ic/dl10019.c:1.13 src/sys/dev/ic/dl10019.c:1.14
--- src/sys/dev/ic/dl10019.c:1.13	Tue Jan 22 03:42:26 2019
+++ src/sys/dev/ic/dl10019.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: dl10019.c,v 1.13 2019/01/22 03:42:26 msaitoh Exp $	*/
+/*	$NetBSD: dl10019.c,v 1.14 2019/05/29 06:17:28 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 2001 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: dl10019.c,v 1.13 2019/01/22 03:42:26 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: dl10019.c,v 1.14 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -117,24 +117,25 @@ void
 dl10019_media_init(struct dp8390_softc *sc)
 {
 	struct ifnet *ifp = &sc->sc_ec.ec_if;
+	struct mii_data *mii = &sc->sc_mii;
 
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = dl10019_mii_readreg;
-	sc->sc_mii.mii_writereg = dl10019_mii_writereg;
-	sc->sc_mii.mii_statchg = dl10019_mii_statchg;
-	ifmedia_init(&sc->sc_mii.mii_media, IFM_IMASK, dp8390_mediachange,
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = dl10019_mii_readreg;
+	mii->mii_writereg = dl10019_mii_writereg;
+	mii->mii_statchg = dl10019_mii_statchg;
+	ifmedia_init(&mii->mii_media, IFM_IMASK, dp8390_mediachange,
 	    dp8390_mediastatus);
 
 	dl10019_mii_reset(sc);
 
-	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);
 
-	if (LIST_FIRST(&sc->sc_mii.mii_phys) == NULL) {
-		ifmedia_add(&sc->sc_mii.mii_media, IFM_ETHER|IFM_NONE, 0, NULL);
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_NONE);
+	if (LIST_FIRST(&mii->mii_phys) == NULL) {
+		ifmedia_add(&mii->mii_media, IFM_ETHER | IFM_NONE, 0, NULL);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_NONE);
 	} else
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_AUTO);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_AUTO);
 }
 
 void

Index: src/sys/dev/ic/elink3.c
diff -u src/sys/dev/ic/elink3.c:1.148 src/sys/dev/ic/elink3.c:1.149
--- src/sys/dev/ic/elink3.c:1.148	Fri May 24 06:26:38 2019
+++ src/sys/dev/ic/elink3.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: elink3.c,v 1.148 2019/05/24 06:26:38 msaitoh Exp $	*/
+/*	$NetBSD: elink3.c,v 1.149 2019/05/29 06:17:28 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 1998, 2001 The NetBSD Foundation, Inc.
@@ -62,7 +62,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: elink3.c,v 1.148 2019/05/24 06:26:38 msaitoh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: elink3.c,v 1.149 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include "opt_inet.h"
 
@@ -135,23 +135,23 @@ struct ep_media {
  * MII and real PHYs attached; no `native' media.
  */
 const struct ep_media ep_vortex_media[] = {
-	{ ELINK_PCI_10BASE_T,	"10baseT",	IFM_ETHER|IFM_10_T,
+	{ ELINK_PCI_10BASE_T,	"10baseT",	IFM_ETHER | IFM_10_T,
 	  ELINKMEDIA_10BASE_T },
-	{ ELINK_PCI_10BASE_T,	"10baseT-FDX",	IFM_ETHER|IFM_10_T|IFM_FDX,
+	{ ELINK_PCI_10BASE_T,	"10baseT-FDX",	IFM_ETHER | IFM_10_T | IFM_FDX,
 	  ELINKMEDIA_10BASE_T },
-	{ ELINK_PCI_AUI,	"10base5",	IFM_ETHER|IFM_10_5,
+	{ ELINK_PCI_AUI,	"10base5",	IFM_ETHER | IFM_10_5,
 	  ELINKMEDIA_AUI },
-	{ ELINK_PCI_BNC,	"10base2",	IFM_ETHER|IFM_10_2,
+	{ ELINK_PCI_BNC,	"10base2",	IFM_ETHER | IFM_10_2,
 	  ELINKMEDIA_10BASE_2 },
-	{ ELINK_PCI_100BASE_TX,	"100baseTX",	IFM_ETHER|IFM_100_TX,
+	{ ELINK_PCI_100BASE_TX,	"100baseTX",	IFM_ETHER | IFM_100_TX,
 	  ELINKMEDIA_100BASE_TX },
-	{ ELINK_PCI_100BASE_TX,	"100baseTX-FDX",IFM_ETHER|IFM_100_TX|IFM_FDX,
+	{ ELINK_PCI_100BASE_TX,	"100baseTX-FDX",IFM_ETHER | IFM_100_TX|IFM_FDX,
 	  ELINKMEDIA_100BASE_TX },
-	{ ELINK_PCI_100BASE_FX,	"100baseFX",	IFM_ETHER|IFM_100_FX,
+	{ ELINK_PCI_100BASE_FX,	"100baseFX",	IFM_ETHER | IFM_100_FX,
 	  ELINKMEDIA_100BASE_FX },
-	{ ELINK_PCI_100BASE_MII,"manual",	IFM_ETHER|IFM_MANUAL,
+	{ ELINK_PCI_100BASE_MII,"manual",	IFM_ETHER | IFM_MANUAL,
 	  ELINKMEDIA_MII },
-	{ ELINK_PCI_100BASE_T4,	"100baseT4",	IFM_ETHER|IFM_100_T4,
+	{ ELINK_PCI_100BASE_T4,	"100baseT4",	IFM_ETHER | IFM_100_T4,
 	  ELINKMEDIA_100BASE_T4 },
 	{ 0,			NULL,		0,
 	  0 },
@@ -162,11 +162,11 @@ const struct ep_media ep_vortex_media[] 
  * in the 3c509, 3c579, and 3c589.
  */
 const struct ep_media ep_509_media[] = {
-	{ ELINK_W0_CC_UTP,	"10baseT",	IFM_ETHER|IFM_10_T,
+	{ ELINK_W0_CC_UTP,	"10baseT",	IFM_ETHER | IFM_10_T,
 	  ELINKMEDIA_10BASE_T },
-	{ ELINK_W0_CC_AUI,	"10base5",	IFM_ETHER|IFM_10_5,
+	{ ELINK_W0_CC_AUI,	"10base5",	IFM_ETHER | IFM_10_5,
 	  ELINKMEDIA_AUI },
-	{ ELINK_W0_CC_BNC,	"10base2",	IFM_ETHER|IFM_10_2,
+	{ ELINK_W0_CC_BNC,	"10base2",	IFM_ETHER | IFM_10_2,
 	  ELINKMEDIA_10BASE_2 },
 	{ 0,			NULL,		0,
 	  0 },
@@ -319,6 +319,7 @@ epconfig(struct ep_softc *sc, u_short ch
 	struct ifnet *ifp = &sc->sc_ethercom.ec_if;
 	bus_space_tag_t iot = sc->sc_iot;
 	bus_space_handle_t ioh = sc->sc_ioh;
+	struct mii_data *mii = &sc->sc_mii;
 	u_int16_t i;
 	u_int8_t myla[ETHER_ADDR_LEN];
 
@@ -426,12 +427,12 @@ epconfig(struct ep_softc *sc, u_short ch
 	 * Initialize our media structures and MII info.  We'll
 	 * probe the MII if we discover that we have one.
 	 */
-	sc->sc_mii.mii_ifp = ifp;
-	sc->sc_mii.mii_readreg = ep_mii_readreg;
-	sc->sc_mii.mii_writereg = ep_mii_writereg;
-	sc->sc_mii.mii_statchg = ep_statchg;
-	sc->sc_ethercom.ec_mii = &sc->sc_mii;
-	ifmedia_init(&sc->sc_mii.mii_media, IFM_IMASK, ep_media_change,
+	mii->mii_ifp = ifp;
+	mii->mii_readreg = ep_mii_readreg;
+	mii->mii_writereg = ep_mii_writereg;
+	mii->mii_statchg = ep_statchg;
+	sc->sc_ethercom.ec_mii = mii;
+	ifmedia_init(&mii->mii_media, IFM_IMASK, ep_media_change,
 	    ep_media_status);
 
 	/*
@@ -459,16 +460,16 @@ epconfig(struct ep_softc *sc, u_short ch
 		 * we don't, just treat the Boomerang like the Vortex.
 		 */
 		if (sc->ep_flags & ELINK_FLAGS_MII) {
-			mii_attach(sc->sc_dev, &sc->sc_mii, 0xffffffff,
+			mii_attach(sc->sc_dev, mii, 0xffffffff,
 			    MII_PHY_ANY, MII_OFFSET_ANY, 0);
-			if (LIST_FIRST(&sc->sc_mii.mii_phys) == NULL) {
-				ifmedia_add(&sc->sc_mii.mii_media,
-				    IFM_ETHER|IFM_NONE, 0, NULL);
-				ifmedia_set(&sc->sc_mii.mii_media,
-				    IFM_ETHER|IFM_NONE);
+			if (LIST_FIRST(&mii->mii_phys) == NULL) {
+				ifmedia_add(&mii->mii_media,
+				    IFM_ETHER | IFM_NONE, 0, NULL);
+				ifmedia_set(&mii->mii_media,
+				    IFM_ETHER | IFM_NONE);
 			} else {
-				ifmedia_set(&sc->sc_mii.mii_media,
-				    IFM_ETHER|IFM_AUTO);
+				ifmedia_set(&mii->mii_media,
+				    IFM_ETHER | IFM_AUTO);
 			}
 			break;
 		}
@@ -573,8 +574,8 @@ ep_509_probemedia(struct ep_softc *sc)
 	/* Sanity check that there are any media! */
 	if ((ep_w0_config & ELINK_W0_CC_MEDIAMASK) == 0) {
 		aprint_error("no media present!\n");
-		ifmedia_add(ifm, IFM_ETHER|IFM_NONE, 0, NULL);
-		ifmedia_set(ifm, IFM_ETHER|IFM_NONE);
+		ifmedia_add(ifm, IFM_ETHER | IFM_NONE, 0, NULL);
+		ifmedia_set(ifm, IFM_ETHER | IFM_NONE);
 		return;
 	}
 
@@ -644,8 +645,8 @@ ep_vortex_probemedia(struct ep_softc *sc
 	/* Sanity check that there are any media! */
 	if ((reset_options & ELINK_PCI_MEDIAMASK) == 0) {
 		aprint_error("no media present!\n");
-		ifmedia_add(ifm, IFM_ETHER|IFM_NONE, 0, NULL);
-		ifmedia_set(ifm, IFM_ETHER|IFM_NONE);
+		ifmedia_add(ifm, IFM_ETHER | IFM_NONE, 0, NULL);
+		ifmedia_set(ifm, IFM_ETHER | IFM_NONE);
 		return;
 	}
 
@@ -878,16 +879,16 @@ ep_roadrunner_mii_enable(struct ep_softc
 
 	GO_WINDOW(3);
 	bus_space_write_2(iot, ioh, ELINK_W3_RESET_OPTIONS,
-	    ELINK_PCI_100BASE_MII|ELINK_RUNNER_ENABLE_MII);
+	    ELINK_PCI_100BASE_MII | ELINK_RUNNER_ENABLE_MII);
 	delay(1000);
 	bus_space_write_2(iot, ioh, ELINK_W3_RESET_OPTIONS,
-	    ELINK_PCI_100BASE_MII|ELINK_RUNNER_MII_RESET|
+	    ELINK_PCI_100BASE_MII | ELINK_RUNNER_MII_RESET |
 	    ELINK_RUNNER_ENABLE_MII);
 	ep_reset_cmd(sc, ELINK_COMMAND, TX_RESET);
 	ep_reset_cmd(sc, ELINK_COMMAND, RX_RESET);
 	delay(1000);
 	bus_space_write_2(iot, ioh, ELINK_W3_RESET_OPTIONS,
-	    ELINK_PCI_100BASE_MII|ELINK_RUNNER_ENABLE_MII);
+	    ELINK_PCI_100BASE_MII | ELINK_RUNNER_ENABLE_MII);
 }
 
 /*
@@ -1046,7 +1047,7 @@ ep_media_status(struct ifnet *ifp, struc
 	bus_space_handle_t ioh = sc->sc_ioh;
 
 	if (sc->enabled == 0) {
-		req->ifm_active = IFM_ETHER|IFM_NONE;
+		req->ifm_active = IFM_ETHER | IFM_NONE;
 		req->ifm_status = 0;
 		return;
 	}

Index: src/sys/dev/pci/if_tlp_pci.c
diff -u src/sys/dev/pci/if_tlp_pci.c:1.125 src/sys/dev/pci/if_tlp_pci.c:1.126
--- src/sys/dev/pci/if_tlp_pci.c:1.125	Sun Dec  9 11:14:02 2018
+++ src/sys/dev/pci/if_tlp_pci.c	Wed May 29 06:17:28 2019
@@ -1,4 +1,4 @@
-/*	$NetBSD: if_tlp_pci.c,v 1.125 2018/12/09 11:14:02 jdolecek Exp $	*/
+/*	$NetBSD: if_tlp_pci.c,v 1.126 2019/05/29 06:17:28 msaitoh Exp $	*/
 
 /*-
  * Copyright (c) 1998, 1999, 2000, 2002 The NetBSD Foundation, Inc.
@@ -36,7 +36,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_tlp_pci.c,v 1.125 2018/12/09 11:14:02 jdolecek Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_tlp_pci.c,v 1.126 2019/05/29 06:17:28 msaitoh Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -509,9 +509,9 @@ tlp_pci_attach(device_t parent, device_t
 		 * Clear the "sleep mode" bit in the CFDA register.
 		 */
 		reg = pci_conf_read(pc, pa->pa_tag, TULIP_PCI_CFDA);
-		if (reg & (CFDA_SLEEP|CFDA_SNOOZE))
+		if (reg & (CFDA_SLEEP | CFDA_SNOOZE))
 			pci_conf_write(pc, pa->pa_tag, TULIP_PCI_CFDA,
-			    reg & ~(CFDA_SLEEP|CFDA_SNOOZE));
+			    reg & ~(CFDA_SLEEP | CFDA_SNOOZE));
 		break;
 
 	default:
@@ -534,7 +534,7 @@ tlp_pci_attach(device_t parent, device_t
 	    PCI_MAPREG_TYPE_IO, 0,
 	    &iot, &ioh, NULL, &iosize) == 0);
 	memh_valid = (pci_mapreg_map(pa, TULIP_PCI_MMBA,
-	    PCI_MAPREG_TYPE_MEM|PCI_MAPREG_MEM_TYPE_32BIT, 0,
+	    PCI_MAPREG_TYPE_MEM | PCI_MAPREG_MEM_TYPE_32BIT, 0,
 	    &memt, &memh, NULL, &memsize) == 0);
 	if (memh_valid) {
 		sc->sc_st = memt;
@@ -652,7 +652,7 @@ tlp_pci_attach(device_t parent, device_t
 
 			sc->sc_srom_addrbits = 6;
 			sc->sc_srom = malloc(TULIP_ROM_SIZE(6), M_DEVBUF,
-			    M_NOWAIT|M_ZERO);
+			    M_NOWAIT | M_ZERO);
 			memcpy(sc->sc_srom, enaddr, sizeof(enaddr));
 			if (tlp_srom_debug) {
 				aprint_normal("SROM CONTENTS:");
@@ -1274,7 +1274,7 @@ tlp_pci_cogent_21040_quirks(struct tulip
 {
 
 	strcpy(psc->sc_tulip.sc_name, "Cogent multi-port");
-	psc->sc_flags |= TULIP_PCI_SHAREDINTR|TULIP_PCI_SHAREDROM;
+	psc->sc_flags |= TULIP_PCI_SHAREDINTR | TULIP_PCI_SHAREDROM;
 }
 
 static void
@@ -1388,6 +1388,7 @@ static void
 tlp_smc9332dst_tmsw_init(struct tulip_softc *sc)
 {
 	struct tulip_21x4x_media *tm;
+	struct mii_data *mii = &sc->sc_mii;
 	const char *sep = "";
 	uint32_t reg;
 	int i, cnt;
@@ -1396,15 +1397,14 @@ tlp_smc9332dst_tmsw_init(struct tulip_so
 	sc->sc_opmode = OPMODE_MBO | OPMODE_PS;
 	TULIP_WRITE(sc, CSR_OPMODE, sc->sc_opmode);
 
-	ifmedia_init(&sc->sc_mii.mii_media, 0, tlp_mediachange,
-	    tlp_mediastatus);
+	ifmedia_init(&mii->mii_media, 0, tlp_mediachange, tlp_mediastatus);
 	aprint_normal_dev(sc->sc_dev, "");
 
 #define	ADD(m, c) \
-	tm = malloc(sizeof(*tm), M_DEVBUF, M_WAITOK|M_ZERO);		\
+	tm = malloc(sizeof(*tm), M_DEVBUF, M_WAITOK | M_ZERO);		\
 	tm->tm_opmode = (c);						\
 	tm->tm_gpdata = GPP_SMC9332DST_INIT;				\
-	ifmedia_add(&sc->sc_mii.mii_media, (m), 0, tm)
+	ifmedia_add(&mii->mii_media, (m), 0, tm)
 #define	PRINT(str)	aprint_normal("%s%s", sep, str); sep = ", "
 
 	ADD(IFM_MAKEWORD(IFM_ETHER, IFM_10_T, 0, 0), OPMODE_TTM);
@@ -1448,18 +1448,17 @@ tlp_smc9332dst_tmsw_init(struct tulip_so
 		}
 		delay(1000);
 	}
-	if (cnt > 100) {
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_TX);
-	} else {
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_10_T);
-	}
+	if (cnt > 100)
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_100_TX);
+	else
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_10_T);
 }
 
 static void
 tlp_pci_vpc_21140_quirks(struct tulip_pci_softc *psc, const uint8_t *enaddr)
 {
 	struct tulip_softc *sc = &psc->sc_tulip;
-	char *p1 = (char *) &sc->sc_srom[32];
+	char *p1 = (char *)&sc->sc_srom[32];
 	char *p2 = &sc->sc_name[0];
 
 	do {
@@ -1583,7 +1582,7 @@ tlp_pci_adaptec_quirks(struct tulip_pci_
 
 	case 0x2400:
 		strcpy(psc->sc_tulip.sc_name, "Adaptec ANA-6944A");
-		psc->sc_flags |= TULIP_PCI_SHAREDINTR|TULIP_PCI_SHAREDROM;
+		psc->sc_flags |= TULIP_PCI_SHAREDINTR | TULIP_PCI_SHAREDROM;
 		break;
 
 	case 0x2b00:
@@ -1592,7 +1591,7 @@ tlp_pci_adaptec_quirks(struct tulip_pci_
 
 	case 0x3000:
 		strcpy(psc->sc_tulip.sc_name, "Adaptec ANA-6922");
-		psc->sc_flags |= TULIP_PCI_SHAREDINTR|TULIP_PCI_SHAREDROM;
+		psc->sc_flags |= TULIP_PCI_SHAREDINTR | TULIP_PCI_SHAREDROM;
 		break;
 
 	default:
@@ -1606,21 +1605,21 @@ static void
 tlp_cogent_em1x0_tmsw_init(struct tulip_softc *sc)
 {
 	struct tulip_21x4x_media *tm;
+	struct mii_data *mii = &sc->sc_mii;
 	const char *sep = "";
 
 	sc->sc_gp_dir = GPP_COGENT_EM1x0_PINS;
 	sc->sc_opmode = OPMODE_MBO | OPMODE_PS;
 	TULIP_WRITE(sc, CSR_OPMODE, sc->sc_opmode);
 
-	ifmedia_init(&sc->sc_mii.mii_media, 0, tlp_mediachange,
-	    tlp_mediastatus);
+	ifmedia_init(&mii->mii_media, 0, tlp_mediachange, tlp_mediastatus);
 	aprint_normal_dev(sc->sc_dev, "");
 
 #define	ADD(m, c) \
-	tm = malloc(sizeof(*tm), M_DEVBUF, M_WAITOK|M_ZERO);		\
+	tm = malloc(sizeof(*tm), M_DEVBUF, M_WAITOK | M_ZERO);		\
 	tm->tm_opmode = (c);						\
 	tm->tm_gpdata = GPP_COGENT_EM1x0_INIT;				\
-	ifmedia_add(&sc->sc_mii.mii_media, (m), 0, tm)
+	ifmedia_add(&mii->mii_media, (m), 0, tm)
 #define	PRINT(str)	aprint_normal("%s%s", sep, str); sep = ", "
 
 	if (sc->sc_srom[32] == 0x15) {
@@ -1633,7 +1632,7 @@ tlp_cogent_em1x0_tmsw_init(struct tulip_
 		PRINT("100baseFX-FDX");
 		aprint_normal("\n");
 
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_FX);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_100_FX);
 	} else {
 		ADD(IFM_MAKEWORD(IFM_ETHER, IFM_100_TX, 0, 0),
 		    OPMODE_PS | OPMODE_PCS | OPMODE_SCR);
@@ -1644,7 +1643,7 @@ tlp_cogent_em1x0_tmsw_init(struct tulip_
 		PRINT("100baseTX-FDX");
 		aprint_normal("\n");
 
-		ifmedia_set(&sc->sc_mii.mii_media, IFM_ETHER|IFM_100_TX);
+		ifmedia_set(&mii->mii_media, IFM_ETHER | IFM_100_TX);
 	}
 
 #undef ADD

Reply via email to