From owner-freebsd-hardware Wed Oct 7 11:20:11 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id LAA15489 for freebsd-hardware-outgoing; Wed, 7 Oct 1998 11:20:11 -0700 (PDT) (envelope-from owner-freebsd-hardware@FreeBSD.ORG) Received: from dingo.cdrom.com (dingo.cdrom.com [204.216.28.145]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id LAA15441 for ; Wed, 7 Oct 1998 11:20:07 -0700 (PDT) (envelope-from mike@dingo.cdrom.com) Received: from dingo.cdrom.com (localhost.cdrom.com [127.0.0.1]) by dingo.cdrom.com (8.9.1/8.8.8) with ESMTP id LAA01419; Wed, 7 Oct 1998 11:21:35 -0700 (PDT) (envelope-from mike@dingo.cdrom.com) Message-Id: <199810071821.LAA01419@dingo.cdrom.com> X-Mailer: exmh version 2.0.2 2/24/98 To: "Matthew N. Dodd" cc: Mike Smith , Marc Slemko , hardware@FreeBSD.ORG, wpaul@ctr.columbia.edu Subject: Re: Lite-On PNIC (fwd) In-reply-to: Your message of "Wed, 07 Oct 1998 14:02:22 EDT." Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Wed, 07 Oct 1998 11:21:35 -0700 From: Mike Smith Sender: owner-freebsd-hardware@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org > > [Added Bill Paul to the CC:] > > On Wed, 7 Oct 1998, Mike Smith wrote: > > Bill Paul has more or less inadvertently rewritten the 'de' driver > > while writing support for the Winbond 100Mbps chips. We might want to > > go with something that's slightly less of an ifdef tangle (have you > > tried looking at the 'de' sources?), especially if the maintainer > > continues to be too busy to maintain it. > > > > It would be interesting to test these changes though. > > You're proposing to provide Bill with examples of -all- the different > varients of the Tulip cards and enough hard drugs to keep in in the state > of mind required to do the work? I was actually planning on just feeding the non-Digital-based cards to Bill, as most of them are cut-downs. We're certainly more than happy to facilitate providing hardware to developers, especially ones with track records like Bill's. 8) > The PNIC changes appear to be straight forward (though they could probably > use some un-grunging) and in line with existing code to support other > Tulip varients. Though I would speak to Bill about this first, I'm > betting the best course would be to apply some flavor of this patch and > move on to better things. I'm pretty sure that more duplicated code would > be a step backwards. It almost certainly would be. The problem is that the 'de' driver is (meant to be) maintained by Matt Thomas, who has been extremely quiet of late (due to a move and other things). We've supplied Matt with hardware in the past (eg. a Macronix card a month or so back), but his responses have been kinda patchy, and as you've seen the net result is an almost unmaintainable mess. > (I would really fear a Bill-written Tulip driver; his other code is really > clean and it would be interesting to see his solution to the Tulip mess.) Having spent some time headfirst in the 'de' dungheap, I'd be inclined to say that it really needs to be a pile of well-specified function vectors with a large heap of access and personality macros. There's just far too much "if tulip_chipid == foo" logic in there, not to mention more #ifdefs than anyone wants to see. -- \\ Sometimes you're ahead, \\ Mike Smith \\ sometimes you're behind. \\ mike@smith.net.au \\ The race is long, and in the \\ msmith@freebsd.org \\ end it's only with yourself. \\ msmith@cdrom.com To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hardware" in the body of the message