Messages in this thread Patch in this message | | | Date | Wed, 14 Oct 1998 13:56:52 +0300 (EEST) | From | Serguei Koubouchine <> | Subject | Re: Linux-2.1.125 ... pre-2.2 imminent |
| |
On Sun, 11 Oct 1998, Linus Torvalds wrote:
> > > On Sun, 11 Oct 1998, Serguei Koubouchine wrote: > > > > This effectively turns Linux into a toy system which can't be used into > > serious network environment :-(( > > Did you read what I said, or are you just out for blood?
Yes, I did read and I'm not for blood...
> I told you that the current driver is better than any of the alternatives. > If you have problems with that, then _fix_ it. > > > Furthermore, the eepro100 driver v1.0.3 DOES work with multicasting. > > I tried a newer eepro100 not too long ago. > > It broke even WITHOUT multicasting, and I could see the breakage myself. > > So I had a choice between a driver that works for me, and a driver that > breaks for me. Which one do _you_ think I should use? > > In short, don't complain. Fix the driver, and get back to me. Otherwise > you're just a pathetic whiner.
Please find the patch against 1.0.3 enclosed. I did never write ethernet drivers and I do not have any documentation on those cards, but knowing about that mii-diag voodoo the fix is rather obvious and I'm pretty sure I haven't done anything wrong. The patched driver does work here at my place like a charm with all the flavours of eepro100 cards in reach (AT-2560TX from Allied Telesyn inclusive). It does not have any problems with gated-3.5.10 and multicasting so far (kernel 2.1.125-ac2).
BTW, I did post the unresolved module symbols report for strip.o several times. Does anyone care to fix it? Is it mainained?
=== Cut === /lib/modules/2.1.125/net/strip.o: unresolved symbol(s) arp_tbl === Cut ===
======================================================================= Serguei Koubouchine aka the Tamer < > The impossible we do immediately. e-mail: ksi@gu.net SK320-RIPE < > Miracles require 24-hour notice. ======================================================================= diff -urN linux-2.1.125.orig/drivers/net/eepro100.c linux-2.1.125/drivers/net/eepro100.c --- linux-2.1.125.orig/drivers/net/eepro100.c Sat Sep 12 11:54:02 1998 +++ linux-2.1.125/drivers/net/eepro100.c Wed Oct 14 12:08:09 1998 @@ -106,7 +106,7 @@ The registers beyond 0x18 only exist on the i82558. */ #define SPEEDO3_TOTAL_SIZE 0x20 -int speedo_debug = 1; +int speedo_debug = 0; /* Theory of Operation @@ -437,7 +437,7 @@ static int full_duplex[] = {-1, -1, -1, -1, -1, -1, -1, -1}; static int options[] = {-1, -1, -1, -1, -1, -1, -1, -1}; #ifdef MODULE -static int debug = -1; /* The debug level */ +static int debug = 0; /* The debug level */ #endif /* A list of all installed Speedo devices, for removing the driver module. */ @@ -880,6 +880,8 @@ wait_for_cmd_done(ioaddr + SCBCmd); outw(CU_DUMPSTATS, ioaddr + SCBCmd); + wait_for_cmd_done(ioaddr + SCBCmd); + mdio_read(ioaddr, sp->phy[0] & 0x1f, 0); return 0; } | |