| To: | Andrew Morton <akpm@xxxxxxxx> |
|---|---|
| Subject: | Re: [patch 12/13] pcnet32 79C975 fiber fix |
| From: | Jeff Garzik <jgarzik@xxxxxxxxx> |
| Date: | Tue, 22 Mar 2005 14:14:09 -0500 |
| Cc: | Don Fry <brazilnut@xxxxxxxxxx>, davem@xxxxxxxxxxxxx, netdev@xxxxxxxxxxx, steven.hardy@xxxxxxxxxxxxxxxx |
| In-reply-to: | <20050315170322.3398bd60.akpm@xxxxxxxx> |
| References: | <200503152222.j2FMMiLB016826@xxxxxxxxxxxxxxxxxxx> <20050316005630.GA9421@xxxxxxxxxx> <20050315170322.3398bd60.akpm@xxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.3) Gecko/20040922 |
Andrew Morton wrote: Don Fry <brazilnut@xxxxxxxxxx> wrote:I had not seen this problem until now. The patch looks ok and probably should have been written that way initially. I have been able to do a quick touch test with a 970A, 971, 972, 973, 975, 976, and 978 (some on ia32 and others on ppc64) without any adverse effects. The 975 I have is copper not fiber.Great, thanks.Since only bit 12 is needed to enable LED writes, the code could really be "a->write_bcr(ioaddr, 2, a->read_bcr(ioaddr, 2) | 0x1000);" During pcnet32_open the ASEL bit is changed anyway.OK, could you please prepare a final patch sometime and get it into Jeff? cc me as well please so I can update the patch I have.
Yes, please do Don, if you have the time.
Jeff
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [IPSEC] Too many SADs!, Stephen Frost |
|---|---|
| Next by Date: | Re: [patch 2.6.11] e1000: avoid sleeping in watchdog timer context, Jeff Garzik |
| Previous by Thread: | Re: [patch 12/13] pcnet32 79C975 fiber fix, Andrew Morton |
| Next by Thread: | [patch 02/13] ENI155P error handling fix, akpm |
| Indexes: | [Date] [Thread] [Top] [All Lists] |