Aren/NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove work-around for Lite-On PNIC in tlp_filter_setup() that is

Browse Source 
apparently no longer necessary (probably due to other fixes that
have happened in the filter setup path) and is in fact harmful.

Based on a patch from Krister Walfridsson, who claims this makes
his newer NetGear FA-310TX work (although, my older NetGear FA-310TX
still does not, but fails in an entirely different way).
This commit is contained in:
thorpej 2000-07-04 04:18:17 +00:00
parent 33d6ee7269
commit a02e08bf4f
1 changed files with 3 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
sys/dev/ic/tulip.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: tulip.c,v 1.69 2000/06/28 17:13:05 mrg Exp $ */
/* $NetBSD: tulip.c,v 1.70 2000/07/04 04:18:17 thorpej Exp $ */
/*-
* Copyright (c) 1998, 1999, 2000 The NetBSD Foundation, Inc.
@ -2555,25 +2555,8 @@ tlp_filter_setup(sc)
* writing OPMODE will start the transmit and receive processes
* in motion.
*/
if (ifp->if_flags & IFF_RUNNING) {
/*
* Actually, some chips seem to need a really hard
* kick in the head for this to work. The genuine
* DEC chips can just be idled, but some of the
* clones seem to REALLY want a reset here. Doing
* the reset will end up here again, but with
* IFF_RUNNING cleared.
*/
switch (sc->sc_chip) {
case TULIP_CHIP_82C168:
case TULIP_CHIP_82C169:
tlp_init(sc);
return;
default:
tlp_idle(sc, OPMODE_ST|OPMODE_SR);
}
}
if (ifp->if_flags & IFF_RUNNING)
tlp_idle(sc, OPMODE_ST|OPMODE_SR);
sc->sc_opmode &= ~(OPMODE_PR|OPMODE_PM);