diff options
| author | glebius <glebius@FreeBSD.org> | 2006-09-15 10:40:54 +0000 |
|---|---|---|
| committer | glebius <glebius@FreeBSD.org> | 2006-09-15 10:40:54 +0000 |
| commit | aeb9aeaa73fe735ddc4a606776072563dd9b6b43 (patch) | |
| tree | ae382d23f768d8bc206d445054d2cee99c94d544 /sys/pci/if_sf.c | |
| parent | 6b78b3a81dfd58cd5838b6fae2c4f76e95a5f930 (diff) | |
| download | FreeBSD-src-aeb9aeaa73fe735ddc4a606776072563dd9b6b43.zip FreeBSD-src-aeb9aeaa73fe735ddc4a606776072563dd9b6b43.tar.gz | |
Consistently use if_printf() only in interface methods: if_start,
if_ioctl, if_watchdog, etc, or in functions that are used by
these methods only. In all other cases use device_printf().
This also fixes several panics, when if_printf() is called before
softc->ifp was initialized.
Submitted by: Alex Lyashkov <umka sevcity.net>
Diffstat (limited to 'sys/pci/if_sf.c')
| -rw-r--r-- | sys/pci/if_sf.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sys/pci/if_sf.c b/sys/pci/if_sf.c index 498aba3..a308e40 100644 --- a/sys/pci/if_sf.c +++ b/sys/pci/if_sf.c @@ -616,7 +616,7 @@ sf_reset(sc) } if (i == SF_TIMEOUT) - if_printf(sc->sf_ifp, "reset never completed!\n"); + device_printf(sc->sf_dev, "reset never completed!\n"); /* Wait a little while for the chip to get its brains in order. */ DELAY(1000); @@ -690,6 +690,7 @@ sf_attach(dev) u_char eaddr[6]; sc = device_get_softc(dev); + sc->sf_dev = dev; mtx_init(&sc->sf_mtx, device_get_nameunit(dev), MTX_NETWORK_LOCK, MTX_DEF); @@ -1081,7 +1082,7 @@ sf_txthresh_adjust(sc) txfctl &= ~SF_TXFRMCTL_TXTHRESH; txfctl |= txthresh; #ifdef DIAGNOSTIC - if_printf(sc->sf_ifp, "tx underrun, increasing " + device_printf(sc->sf_dev, "tx underrun, increasing " "tx threshold to %d bytes\n", txthresh * 4); #endif @@ -1243,7 +1244,7 @@ sf_init_locked(sc) sf_setperf(sc, 0, IF_LLADDR(sc->sf_ifp)); if (sf_init_rx_ring(sc) == ENOBUFS) { - if_printf(sc->sf_ifp, + device_printf(sc->sf_dev, "initialization failed: no memory for rx buffers\n"); return; } |
