netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] 8139cp: fix eeprom read command length
@ 2006-06-14  6:43 Philip Craig
  2006-06-20  8:52 ` Jeff Garzik
  0 siblings, 1 reply; 5+ messages in thread
From: Philip Craig @ 2006-06-14  6:43 UTC (permalink / raw)
  To: Jeff Garzik; +Cc: netdev

The read command for the 93C46/93C56 EEPROMS should be 3 bits plus
the address.  This doesn't appear to affect the operation of the
read command, but similar errors for write commands do cause failures.

Signed-off-by: Philip Craig <philipc@snapgear.com>

Index: linux-2.6.17-rc6/drivers/net/8139cp.c
===================================================================
--- linux-2.6.17-rc6.orig/drivers/net/8139cp.c	2006-06-14 16:02:00.000000000 +1000
+++ linux-2.6.17-rc6/drivers/net/8139cp.c	2006-06-14 16:03:29.000000000 +1000
@@ -1628,7 +1628,7 @@ static int read_eeprom (void __iomem *io
 	eeprom_delay ();

 	/* Shift the read command bits out. */
-	for (i = 4 + addr_len; i >= 0; i--) {
+	for (i = 3 + addr_len - 1; i >= 0; i--) {
 		int dataval = (read_cmd & (1 << i)) ? EE_DATA_WRITE : 0;
 		writeb (EE_ENB | dataval, ee_addr);
 		eeprom_delay ();

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2006-06-23  3:20 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-14  6:43 [PATCH 1/2] 8139cp: fix eeprom read command length Philip Craig
2006-06-20  8:52 ` Jeff Garzik
2006-06-21  1:33   ` [PATCH 1/2 resend] " Philip Craig
2006-06-23  3:20     ` Jeff Garzik
2006-06-21  1:33   ` [PATCH 2/2 resend] 8139cp: add ethtool eeprom support Philip Craig

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).