On 10/6/2001 5:00 PM Fred Gray fe...@np... wrote:
>On Sat, Oct 06, 2001 at 09:14:42AM -0400, Marty Connor wrote:
>> The routine that will be needed to fix this, I'm pretty sure is in
>> /usr/src/linux/drivers/net/tulip/pnic.c, and is called
>> pnic_check_duplex(). It will need to be modified a little for Etherboot,
>> but a call to it before returning from the reset routine will probably
>> help. I'll have to look at it more closely.
>Excellent! I went ahead and implemented this (patch attached), and it works
>for me. I don't know how this particular patch will affect other tulip-like
>cards, though. I suspect that I'm calling tulip_check_duplex from the wrong
>place in general.
Fred, great job! I've been wanting to fix this bug for a long time, but
since I couldn't reproduce the problem, I never got seriously into it. I
know there are a lot of folks with Netgear FA310TX cards and certain
picky switches that will surely appreciate this!
Your patch looks and compiles fine. Since this seems to be a peculiar
case, somewhat specific to certain cards, I've conditionalized the call
like this:
if (tp->chip_id == LC82C168)
tulip_check_duplex(nic);
Since the complete patch is small, I'll attach it for anyone who wants to
try it by hand. I'll also update the rom-o-matic.net site with a test
version for people who get their ROM images that way.
Will anyone with an FA310TX or other LC82C168/169 board test the
"5.0.4-mc1" release on rom-o-matic.net? I don't think the patch should
affect anything that works currently in 5.0.x, but better to test now.
>Thanks very much for your help,
And many thanks very much for yours! It's folks like you that make
Etherboot such a rewarding project.
Regards,
Marty
---
Try: http://rom-o-matic.net/ to make Etherboot images instantly.
Name: Marty Connor
US Mail: Entity Cyber, Inc.; P.O. Box 391827; Cambridge, MA 02139; USA
Voice: (617) 491-6935, Fax: (617) 491-7046
Email: md...@th...
Web: http://www.thinguin.org/
|