Module Name: src Committed By: msaitoh Date: Tue Apr 9 06:15:21 UTC 2019
Modified Files: src/sys/dev/isa: if_ix.c Log Message: KNF. No functional change. To generate a diff of this commit: cvs rdiff -u -r1.35 -r1.36 src/sys/dev/isa/if_ix.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/isa/if_ix.c diff -u src/sys/dev/isa/if_ix.c:1.35 src/sys/dev/isa/if_ix.c:1.36 --- src/sys/dev/isa/if_ix.c:1.35 Thu Jul 14 04:19:27 2016 +++ src/sys/dev/isa/if_ix.c Tue Apr 9 06:15:21 2019 @@ -1,4 +1,4 @@ -/* $NetBSD: if_ix.c,v 1.35 2016/07/14 04:19:27 msaitoh Exp $ */ +/* $NetBSD: if_ix.c,v 1.36 2019/04/09 06:15:21 msaitoh Exp $ */ /*- * Copyright (c) 1998 The NetBSD Foundation, Inc. @@ -30,7 +30,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: if_ix.c,v 1.35 2016/07/14 04:19:27 msaitoh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: if_ix.c,v 1.36 2019/04/09 06:15:21 msaitoh Exp $"); #include <sys/param.h> #include <sys/systm.h> @@ -63,7 +63,7 @@ __KERNEL_RCSID(0, "$NetBSD: if_ix.c,v 1. #define DPRINTF(x) #endif -int ix_media[] = { +static int ix_media[] = { IFM_ETHER | IFM_10_5, IFM_ETHER | IFM_10_2, IFM_ETHER | IFM_10_T, @@ -76,8 +76,8 @@ struct ix_softc { bus_space_tag_t sc_regt; /* space tag for registers */ bus_space_handle_t sc_regh; /* space handle for registers */ - u_int8_t use_pio; /* use PIO rather than shared mem */ - u_int16_t irq_encoded; /* encoded IRQ */ + uint8_t use_pio; /* use PIO rather than shared mem */ + uint16_t irq_encoded; /* encoded IRQ */ void *sc_ih; /* interrupt handle */ }; @@ -90,21 +90,21 @@ static void ix_copyout(struct ie_sof static void ix_bus_barrier(struct ie_softc *, int, int, int); -static u_int16_t ix_read_16(struct ie_softc *, int); -static void ix_write_16(struct ie_softc *, int, u_int16_t); +static uint16_t ix_read_16(struct ie_softc *, int); +static void ix_write_16(struct ie_softc *, int, uint16_t); static void ix_write_24(struct ie_softc *, int, int); -static void ix_zeromem (struct ie_softc *, int, int); +static void ix_zeromem(struct ie_softc *, int, int); static void ix_mediastatus(struct ie_softc *, struct ifmediareq *); -static u_int16_t ix_read_eeprom(bus_space_tag_t, bus_space_handle_t, int); +static uint16_t ix_read_eeprom(bus_space_tag_t, bus_space_handle_t, int); static void ix_eeprom_outbits(bus_space_tag_t, bus_space_handle_t, int, int); -static int ix_eeprom_inbits (bus_space_tag_t, bus_space_handle_t); -static void ix_eeprom_clock (bus_space_tag_t, bus_space_handle_t, int); +static int ix_eeprom_inbits(bus_space_tag_t, bus_space_handle_t); +static void ix_eeprom_clock(bus_space_tag_t, bus_space_handle_t, int); -int ix_match(device_t, cfdata_t, void *); -void ix_attach(device_t, device_t, void *); +static int ix_match(device_t, cfdata_t, void *); +static void ix_attach(device_t, device_t, void *); /* * EtherExpress/16 support routines @@ -112,12 +112,12 @@ void ix_attach(device_t, device_t, void static void ix_reset(struct ie_softc *sc, int why) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; switch (why) { case CHIP_PROBE: bus_space_write_1(isc->sc_regt, isc->sc_regh, IX_ECTRL, - IX_RESET_586); + IX_RESET_586); delay(100); bus_space_write_1(isc->sc_regt, isc->sc_regh, IX_ECTRL, 0); delay(100); @@ -125,17 +125,18 @@ ix_reset(struct ie_softc *sc, int why) case CARD_RESET: break; - } + } } static void ix_atten(struct ie_softc *sc, int why) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; + bus_space_write_1(isc->sc_regt, isc->sc_regh, IX_ATTN, 0); } -static u_int16_t +static uint16_t ix_read_eeprom(bus_space_tag_t iot, bus_space_handle_t ioh, int location) { int ectrl, edata; @@ -153,7 +154,7 @@ ix_read_eeprom(bus_space_tag_t iot, bus_ bus_space_write_1(iot, ioh, IX_ECTRL, ectrl); ix_eeprom_clock(iot, ioh, 1); ix_eeprom_clock(iot, ioh, 0); - return (edata); + return edata; } static void @@ -195,7 +196,7 @@ ix_eeprom_inbits(bus_space_tag_t iot, bu } ix_eeprom_clock(iot, ioh, 0); } - return (edata); + return edata; } static void @@ -215,7 +216,7 @@ ix_eeprom_clock(bus_space_tag_t iot, bus static int ix_intrhook(struct ie_softc *sc, int where) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; switch (where) { case INTR_ENTER: @@ -239,21 +240,20 @@ static void ix_copyin(struct ie_softc *sc, void *dst, int offset, size_t size) { int i, dribble; - u_int8_t* bptr = dst; - u_int16_t* wptr = dst; - struct ix_softc* isc = (struct ix_softc *) sc; + uint8_t *bptr = dst; + uint16_t *wptr = dst; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) { /* Reset read pointer to the specified offset */ bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_READ); + BUS_SPACE_BARRIER_READ); bus_space_write_2(sc->bt, sc->bh, IX_READPTR, offset); bus_space_barrier(sc->bt, sc->bh, IX_READPTR, 2, - BUS_SPACE_BARRIER_WRITE); - } else { + BUS_SPACE_BARRIER_WRITE); + } else bus_space_barrier(sc->bt, sc->bh, offset, size, - BUS_SPACE_BARRIER_READ); - } + BUS_SPACE_BARRIER_READ); if (offset % 2) { if (isc->use_pio) @@ -264,17 +264,16 @@ ix_copyin(struct ie_softc *sc, void *dst } dribble = size % 2; - wptr = (u_int16_t*) bptr; + wptr = (uint16_t*)bptr; if (isc->use_pio) { - for(i = 0; i < size / 2; i++) { + for (i = 0; i < size / 2; i++) { *wptr = bus_space_read_2(sc->bt, sc->bh, IX_DATAPORT); wptr++; } - } else { + } else bus_space_read_region_2(sc->bt, sc->bh, offset, - (u_int16_t *) bptr, size / 2); - } + (uint16_t *)bptr, size / 2); if (dribble) { bptr += size - 1; @@ -293,9 +292,9 @@ ix_copyout(struct ie_softc *sc, const vo int i, dribble; int osize = size; int ooffset = offset; - const u_int8_t* bptr = src; - const u_int16_t* wptr = src; - struct ix_softc* isc = (struct ix_softc *) sc; + const uint8_t *bptr = src; + const uint16_t *wptr = src; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) { /* Reset write pointer to the specified offset */ @@ -313,17 +312,16 @@ ix_copyout(struct ie_softc *sc, const vo } dribble = size % 2; - wptr = (const u_int16_t*) bptr; + wptr = (const uint16_t*)bptr; if (isc->use_pio) { - for(i = 0; i < size / 2; i++) { + for (i = 0; i < size / 2; i++) { bus_space_write_2(sc->bt, sc->bh, IX_DATAPORT, *wptr); wptr++; } - } else { + } else bus_space_write_region_2(sc->bt, sc->bh, offset, - (const u_int16_t *)bptr, size / 2); - } + (const uint16_t *)bptr, size / 2); if (dribble) { bptr += size - 1; @@ -337,16 +335,16 @@ ix_copyout(struct ie_softc *sc, const vo if (isc->use_pio) bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); else bus_space_barrier(sc->bt, sc->bh, ooffset, osize, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } static void ix_bus_barrier(struct ie_softc *sc, int offset, int length, int flags) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, flags); @@ -354,73 +352,73 @@ ix_bus_barrier(struct ie_softc *sc, int bus_space_barrier(sc->bt, sc->bh, offset, length, flags); } -static u_int16_t -ix_read_16 (struct ie_softc *sc, int offset) +static uint16_t +ix_read_16(struct ie_softc *sc, int offset) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) { bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_READ); + BUS_SPACE_BARRIER_READ); /* Reset read pointer to the specified offset */ bus_space_write_2(sc->bt, sc->bh, IX_READPTR, offset); bus_space_barrier(sc->bt, sc->bh, IX_READPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); return bus_space_read_2(sc->bt, sc->bh, IX_DATAPORT); } else { bus_space_barrier(sc->bt, sc->bh, offset, 2, - BUS_SPACE_BARRIER_READ); - return bus_space_read_2(sc->bt, sc->bh, offset); + BUS_SPACE_BARRIER_READ); + return bus_space_read_2(sc->bt, sc->bh, offset); } } static void -ix_write_16 (struct ie_softc *sc, int offset, u_int16_t value) +ix_write_16(struct ie_softc *sc, int offset, uint16_t value) { - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) { /* Reset write pointer to the specified offset */ bus_space_write_2(sc->bt, sc->bh, IX_WRITEPTR, offset); bus_space_barrier(sc->bt, sc->bh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); bus_space_write_2(sc->bt, sc->bh, IX_DATAPORT, value); bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } else { bus_space_write_2(sc->bt, sc->bh, offset, value); bus_space_barrier(sc->bt, sc->bh, offset, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } } static void ix_write_24 (struct ie_softc *sc, int offset, int addr) { - char* ptr; - struct ix_softc* isc = (struct ix_softc *) sc; - int val = addr + (u_long) sc->sc_maddr - (u_long) sc->sc_iobase; + char *ptr; + struct ix_softc *isc = (struct ix_softc *)sc; + int val = addr + (u_long)sc->sc_maddr - (u_long)sc->sc_iobase; if (isc->use_pio) { /* Reset write pointer to the specified offset */ bus_space_write_2(sc->bt, sc->bh, IX_WRITEPTR, offset); bus_space_barrier(sc->bt, sc->bh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); - ptr = (char*) &val; + ptr = (char*)&val; bus_space_write_2(sc->bt, sc->bh, IX_DATAPORT, - *((u_int16_t *)ptr)); + *((uint16_t *)ptr)); bus_space_write_2(sc->bt, sc->bh, IX_DATAPORT, - *((u_int16_t *)(ptr + 2))); + *((uint16_t *)(ptr + 2))); bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } else { bus_space_write_4(sc->bt, sc->bh, offset, val); bus_space_barrier(sc->bt, sc->bh, offset, 4, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } } @@ -429,13 +427,13 @@ ix_zeromem(struct ie_softc *sc, int offs { int i; int dribble; - struct ix_softc* isc = (struct ix_softc *) sc; + struct ix_softc *isc = (struct ix_softc *)sc; if (isc->use_pio) { /* Reset write pointer to the specified offset */ bus_space_write_2(sc->bt, sc->bh, IX_WRITEPTR, offset); bus_space_barrier(sc->bt, sc->bh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); if (offset % 2) { bus_space_write_1(sc->bt, sc->bh, IX_DATAPORT, 0); @@ -443,18 +441,18 @@ ix_zeromem(struct ie_softc *sc, int offs } dribble = count % 2; - for(i = 0; i < count / 2; i++) + for (i = 0; i < count / 2; i++) bus_space_write_2(sc->bt, sc->bh, IX_DATAPORT, 0); if (dribble) bus_space_write_1(sc->bt, sc->bh, IX_DATAPORT, 0); bus_space_barrier(sc->bt, sc->bh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } else { bus_space_set_region_1(sc->bt, sc->bh, offset, 0, count); bus_space_barrier(sc->bt, sc->bh, offset, count, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); } } @@ -463,9 +461,7 @@ ix_mediastatus(struct ie_softc *sc, stru { struct ifmedia *ifm = &sc->sc_media; - /* - * The currently selected media is always the active media. - */ + /* The currently selected media is always the active media. */ ifmr->ifm_active = ifm->ifm_cur->ifm_media; } @@ -479,31 +475,31 @@ ix_match(device_t parent, cfdata_t cf, v u_short checksum = 0; bus_space_handle_t ioh; bus_space_tag_t iot; - u_int8_t val, bart_config; - u_short pg, adjust, decode, edecode; - u_short board_id, id_var1, id_var2, irq, irq_encoded; + uint8_t val, bart_config; + uint16_t pg, adjust, decode, edecode; + uint16_t board_id, id_var1, id_var2, irq, irq_encoded; struct isa_attach_args * const ia = aux; short irq_translate[] = {0, 0x09, 0x03, 0x04, 0x05, 0x0a, 0x0b, 0}; if (ia->ia_nio < 1) - return (0); + return 0; if (ia->ia_niomem < 1) - return (0); + return 0; if (ia->ia_nirq < 1) - return (0); + return 0; if (ISA_DIRECT_CONFIG(ia)) - return (0); + return 0; iot = ia->ia_iot; if (ia->ia_io[0].ir_addr == ISA_UNKNOWN_PORT) - return (0); + return 0; if (bus_space_map(iot, ia->ia_io[0].ir_addr, IX_IOSIZE, 0, &ioh) != 0) { DPRINTF(("Can't map io space at 0x%x\n", ia->ia_iobase)); - return (0); + return 0; } /* XXX: reset any ee16 at the current iobase */ @@ -511,7 +507,7 @@ ix_match(device_t parent, cfdata_t cf, v bus_space_write_1(iot, ioh, IX_ECTRL, 0); delay(240); - /* now look for ee16. */ + /* Now look for ee16. */ board_id = id_var1 = id_var2 = 0; for (i = 0; i < 4 ; i++) { id_var1 = bus_space_read_1(iot, ioh, IX_ID_PORT); @@ -594,11 +590,11 @@ ix_match(device_t parent, cfdata_t cf, v goto out; } - /* need to put the 586 in RESET, and leave it */ + /* Need to put the 586 in RESET, and leave it */ bus_space_write_1(iot, ioh, IX_ECTRL, IX_RESET_586); - /* read the eeprom and checksum it, should == IX_ID */ - for(i = 0; i < 0x40; i++) + /* Read the eeprom and checksum it, should == IX_ID */ + for (i = 0; i < 0x40; i++) checksum += ix_read_eeprom(iot, ioh, i); if (checksum != IX_ID) { @@ -642,7 +638,7 @@ ix_match(device_t parent, cfdata_t cf, v goto out; } - /* disable the board interrupts */ + /* Disable the board interrupts */ bus_space_write_1(iot, ioh, IX_IRQ, irq_encoded); bart_config = bus_space_read_1(iot, ioh, IX_CONFIG); @@ -670,10 +666,10 @@ ix_match(device_t parent, cfdata_t cf, v out: bus_space_unmap(iot, ioh, IX_IOSIZE); - return (rv); + return rv; } -void +static void ix_attach(device_t parent, device_t self, void *aux) { struct ix_softc *isc = device_private(self); @@ -682,13 +678,13 @@ ix_attach(device_t parent, device_t self int media; int i, memsize; - u_int8_t bart_config; + uint8_t bart_config; bus_space_tag_t iot; - u_int8_t bpat, bval; - u_int16_t wpat, wval; + uint8_t bpat, bval; + uint16_t wpat, wval; bus_space_handle_t ioh, memh; - u_short irq_encoded; - u_int8_t ethaddr[ETHER_ADDR_LEN]; + uint16_t irq_encoded; + uint8_t ethaddr[ETHER_ADDR_LEN]; sc->sc_dev = self; iot = ia->ia_iot; @@ -701,7 +697,7 @@ ix_attach(device_t parent, device_t self isc->use_pio = (ia->ia_iomem[0].ir_size <= (16 * 1024)); if (bus_space_map(iot, ia->ia_io[0].ir_addr, - ia->ia_io[0].ir_size, 0, &ioh) != 0) { + ia->ia_io[0].ir_size, 0, &ioh) != 0) { DPRINTF(("\n%s: can't map i/o space 0x%x-0x%x\n", device_xname(self), ia->ia_[0].ir_addr, @@ -711,14 +707,15 @@ ix_attach(device_t parent, device_t self /* We map memory even if using PIO so something else doesn't grab it */ if (ia->ia_iomem[0].ir_size) { - if (bus_space_map(ia->ia_memt, ia->ia_iomem[0].ir_addr, - ia->ia_iomem[0].ir_size, 0, &memh) != 0) { - DPRINTF(("\n%s: can't map iomem space 0x%x-0x%x\n", - device_xname(self), ia->ia_iomem[0].ir_addr, - ia->ia_iomem[0].ir_addr + ia->ia_iomem[0].ir_size - 1)); - bus_space_unmap(iot, ioh, ia->ia_io[0].ir_size); - return; - } + if (bus_space_map(ia->ia_memt, ia->ia_iomem[0].ir_addr, + ia->ia_iomem[0].ir_size, 0, &memh) != 0) { + DPRINTF(("\n%s: can't map iomem space 0x%x-0x%x\n", + device_xname(self), ia->ia_iomem[0].ir_addr, + ia->ia_iomem[0].ir_addr + + ia->ia_iomem[0].ir_size - 1)); + bus_space_unmap(iot, ioh, ia->ia_io[0].ir_size); + return; + } } isc->sc_regt = iot; @@ -771,7 +768,7 @@ ix_attach(device_t parent, device_t self * not by how much is mapped into the memory-mapped region, so * determine how much total memory we have to play with here. */ - for(memsize = 64 * 1024; memsize; memsize -= 16 * 1024) { + for (memsize = 64 * 1024; memsize; memsize -= 16 * 1024) { /* warm up shared memory, the zero it all out */ ix_zeromem(sc, 0, 32); ix_zeromem(sc, 0, memsize); @@ -779,26 +776,25 @@ ix_attach(device_t parent, device_t self /* Reset write pointer to the start of RAM */ bus_space_write_2(iot, ioh, IX_WRITEPTR, 0); bus_space_barrier(iot, ioh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); - /* write test pattern */ - for(i = 0, wpat = 1; i < memsize; i += 2) { + /* Write test pattern */ + for (i = 0, wpat = 1; i < memsize; i += 2) { bus_space_write_2(iot, ioh, IX_DATAPORT, wpat); wpat += 3; } /* Flush all reads & writes to data port */ bus_space_barrier(iot, ioh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_READ | - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_READ | BUS_SPACE_BARRIER_WRITE); /* Reset read pointer to beginning of card RAM */ bus_space_write_2(iot, ioh, IX_READPTR, 0); bus_space_barrier(iot, ioh, IX_READPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); - /* read and verify test pattern */ - for(i = 0, wpat = 1; i < memsize; i += 2) { + /* Read and verify test pattern */ + for (i = 0, wpat = 1; i < memsize; i += 2) { wval = bus_space_read_2(iot, ioh, IX_DATAPORT); if (wval != wpat) @@ -818,26 +814,25 @@ ix_attach(device_t parent, device_t self /* Reset write pointer to start of card RAM */ bus_space_write_2(iot, ioh, IX_WRITEPTR, 0); bus_space_barrier(iot, ioh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); - /* write out test pattern */ - for(i = 0, bpat = 1; i < memsize; i++) { + /* Write out test pattern */ + for (i = 0, bpat = 1; i < memsize; i++) { bus_space_write_1(iot, ioh, IX_DATAPORT, bpat); bpat += 3; } /* Flush all reads & writes to data port */ bus_space_barrier(iot, ioh, IX_DATAPORT, 2, - BUS_SPACE_BARRIER_READ | - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_READ | BUS_SPACE_BARRIER_WRITE); /* Reset read pointer to beginning of card RAM */ bus_space_write_2(iot, ioh, IX_READPTR, 0); bus_space_barrier(iot, ioh, IX_READPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); - /* read and verify test pattern */ - for(i = 0, bpat = 1; i < memsize; i++) { + /* Read and verify test pattern */ + for (i = 0, bpat = 1; i < memsize; i++) { bval = bus_space_read_1(iot, ioh, IX_DATAPORT); if (bval != bpat) @@ -861,7 +856,7 @@ ix_attach(device_t parent, device_t self sc->bh = ioh; sc->sc_msize = memsize; - sc->sc_maddr = (void*) 0; + sc->sc_maddr = (void*)0; } else { sc->bt = ia->ia_memt; sc->bh = memh; @@ -873,7 +868,7 @@ ix_attach(device_t parent, device_t self /* Map i/o space. */ sc->sc_iobase = (char *)sc->sc_maddr + sc->sc_msize - (1 << 24); - /* set up pointers to important on-card control structures */ + /* Set up pointers to important on-card control structures */ sc->iscp = 0; sc->scb = IE_ISCP_SZ; sc->scp = sc->sc_msize + IE_SCP_ADDR - (1 << 24); @@ -881,38 +876,35 @@ ix_attach(device_t parent, device_t self sc->buf_area = sc->scb + IE_SCB_SZ; sc->buf_area_sz = sc->sc_msize - IE_ISCP_SZ - IE_SCB_SZ - IE_SCP_SZ; - /* zero card memory */ + /* Zero card memory */ ix_zeromem(sc, 0, 32); ix_zeromem(sc, 0, sc->sc_msize); - /* set card to 16-bit bus mode */ + /* Set card to 16-bit bus mode */ if (isc->use_pio) { bus_space_write_2(sc->bt, sc->bh, IX_WRITEPTR, - IE_SCP_BUS_USE((u_long)sc->scp)); + IE_SCP_BUS_USE((u_long)sc->scp)); bus_space_barrier(sc->bt, sc->bh, IX_WRITEPTR, 2, - BUS_SPACE_BARRIER_WRITE); + BUS_SPACE_BARRIER_WRITE); bus_space_write_1(sc->bt, sc->bh, IX_DATAPORT, - IE_SYSBUS_16BIT); - } else { + IE_SYSBUS_16BIT); + } else bus_space_write_1(sc->bt, sc->bh, - IE_SCP_BUS_USE((u_long)sc->scp), - IE_SYSBUS_16BIT); - } + IE_SCP_BUS_USE((u_long)sc->scp), IE_SYSBUS_16BIT); - /* set up pointers to key structures */ - ix_write_24(sc, IE_SCP_ISCP((u_long)sc->scp), (u_long) sc->iscp); - ix_write_16(sc, IE_ISCP_SCB((u_long)sc->iscp), (u_long) sc->scb); - ix_write_24(sc, IE_ISCP_BASE((u_long)sc->iscp), (u_long) sc->iscp); + /* Set up pointers to key structures */ + ix_write_24(sc, IE_SCP_ISCP((u_long)sc->scp), (u_long)sc->iscp); + ix_write_16(sc, IE_ISCP_SCB((u_long)sc->iscp), (u_long)sc->scb); + ix_write_24(sc, IE_ISCP_BASE((u_long)sc->iscp), (u_long)sc->iscp); - /* flush setup of pointers, check if chip answers */ + /* Flush setup of pointers, check if chip answers */ if (isc->use_pio) { bus_space_barrier(sc->bt, sc->bh, 0, IX_IOSIZE, - BUS_SPACE_BARRIER_WRITE); - } else { + BUS_SPACE_BARRIER_WRITE); + } else bus_space_barrier(sc->bt, sc->bh, 0, sc->sc_msize, - BUS_SPACE_BARRIER_WRITE); - } + BUS_SPACE_BARRIER_WRITE); if (!i82586_proberam(sc)) { DPRINTF(("\n%s: Can't talk to i82586!\n", @@ -927,7 +919,7 @@ ix_attach(device_t parent, device_t self /* Figure out which media is being used... */ if (ix_read_eeprom(iot, ioh, IX_EEPROM_CONFIG1) & - IX_EEPROM_MEDIA_EXT) { + IX_EEPROM_MEDIA_EXT) { if (ix_read_eeprom(iot, ioh, IX_EEPROM_MEDIA) & IX_EEPROM_MEDIA_TP) media = IFM_ETHER | IFM_10_T; @@ -943,13 +935,11 @@ ix_attach(device_t parent, device_t self bus_space_write_1(iot, ioh, IX_CONFIG, bart_config); bart_config = bus_space_read_1(iot, ioh, IX_CONFIG); - irq_encoded = ix_read_eeprom(iot, ioh, - IX_EEPROM_CONFIG1); + irq_encoded = ix_read_eeprom(iot, ioh, IX_EEPROM_CONFIG1); irq_encoded = (irq_encoded & IX_EEPROM_IRQ) >> IX_EEPROM_IRQ_SHIFT; /* Enable interrupts */ - bus_space_write_1(iot, ioh, IX_IRQ, - irq_encoded | IX_IRQ_ENABLE); + bus_space_write_1(iot, ioh, IX_IRQ, irq_encoded | IX_IRQ_ENABLE); /* Flush all writes to registers */ bus_space_barrier(iot, ioh, 0, ia->ia_io[0].ir_size, @@ -958,7 +948,7 @@ ix_attach(device_t parent, device_t self isc->irq_encoded = irq_encoded; i82586_attach(sc, "EtherExpress/16", ethaddr, - ix_media, NIX_MEDIA, media); + ix_media, NIX_MEDIA, media); if (isc->use_pio) aprint_error_dev(self, "unsupported memory config, using PIO "