All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.