netdev
[Top] [All Lists]

Re: PATCH: sundance #2

To: Donald Becker <becker@xxxxxxxxx>
Subject: Re: PATCH: sundance #2
From: Jeff Garzik <jgarzik@xxxxxxxxxxxxxxxx>
Date: Thu, 19 Sep 2002 14:13:56 -0400
Cc: netdev@xxxxxxxxxxx, Jason Lunz <lunz@xxxxxxxxxxxx>, Richard Gooch <rgooch@xxxxxxxxxxxxxxx>, "Patrick R. McManus" <mcmanus@xxxxxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, edward_peng@xxxxxxxxxxxx
Organization: MandrakeSoft
References: <Pine.LNX.4.44.0209191316300.29420-100000@xxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1) Gecko/20020826
Donald Becker wrote:
+/* Set iff a MII transceiver on any interface requires mdio preamble.
+   This only set with older tranceivers, so the extra
+   code size of a per-interface flag is not worthwhile. */
+static int mii_preamble_required = 0;

You can get rid of this as a module option, and make it a per-interface
setting. The transceiver on the Kendin chip requires this (rather old-fashioned)
access method, while none of the previous Sundance-based boards with
external transceivers did.

I added it as a module parameter as a back-up over-ride, but I'm certain
that the automatic detection works.

Good enough for me...


                                Theory of Operation


Whoever changed the transmit path should update the TOO.

noted



-       {"Sundance Technology Alta", {0x020113F0, 0xffffffff,},
-        PCI_IOTYPE, 128, CanHaveMII},
+       {"D-Link DFE-550TX FAST Ethernet Adapter"},
+       {"D-Link DFE-550FX 100Mbps Fiber-optics Adapter"},

Yeah, you should probably throw away the rest of the changes.
You are probably going to want to keep the drv_flags field.  I know
that all of the current chips have the same flag (CanHaveMII), but...


That's probably a style area that you and I will disagree on... :)

        Jeff



<Prev in Thread] Current Thread [Next in Thread>