mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Grant Grundler <grundler@parisc-linux.org>
To: Grant Grundler <grundler@parisc-linux.org>,
	frahm@irsamc.ups-tlse.fr, linux-kernel@vger.kernel.org,
	jgarzik@pobox.org
Subject: Re: [Fwd: 2.6.10-rc3: tulip-driver: tulip_stop_rxtx() failed]
Date: Wed, 15 Dec 2004 10:21:28 -0700	[thread overview]
Message-ID: <20041215172128.GB27818@colo.lackof.org> (raw)
In-Reply-To: <20041215005722.GE4357@tuxdriver.com>

On Tue, Dec 14, 2004 at 07:57:22PM -0500, John W. Linville wrote:
> On Sun, Dec 12, 2004 at 08:59:36PM -0700, Grant Grundler wrote:
> 
> > But still, I'm hopeing for two code changes as a result:
> > 1) include CSR5 and CSR6 in the printk output
> > 2) the date of the tulip driver revision needs to be updated (or dropped).
> 
> Patches?

Sorry...appended below.

> If you don't want to post them publicly yourself, send them to me
> and I'll be happy to test/package/post them...

Thanks!
But I don't mind posting them...I just spaced out and send the first bit
to jgarzik directly instead of "reply all".

Commit Log:
	add CSR5 and CSR6 output to debug tulip_stop_rxtx failures.
	Update version release date

Signed-off-by:
	Grant Grundler <grundler@parisc-linux.org>

thanks,
grant

Index: drivers/net/tulip/tulip.h
===================================================================
RCS file: /var/cvs/linux-2.6/drivers/net/tulip/tulip.h,v
retrieving revision 1.11
diff -u -p -r1.11 tulip.h
--- drivers/net/tulip/tulip.h	4 Dec 2004 07:02:42 -0000	1.11
+++ drivers/net/tulip/tulip.h	12 Dec 2004 21:51:43 -0000
@@ -474,8 +474,11 @@ static inline void tulip_stop_rxtx(struc
 			udelay(10);
 
 		if (!i)
-			printk(KERN_DEBUG "%s: tulip_stop_rxtx() failed\n",
-					tp->pdev->slot_name);
+			printk(KERN_DEBUG "%s: tulip_stop_rxtx() failed"
+					" (CSR5 0x%x CSR6 0x%x)\n",
+					tp->pdev->slot_name,
+					ioread32(ioaddr + CSR5),
+					ioread32(ioaddr + CSR6));
 	}
 }
 

Index: drivers/net/tulip/tulip_core.c
===================================================================
RCS file: /var/cvs/linux-2.6/drivers/net/tulip/tulip_core.c,v
retrieving revision 1.24
diff -u -p -r1.24 tulip_core.c
--- drivers/net/tulip/tulip_core.c	4 Dec 2004 07:02:42 -0000	1.24
+++ drivers/net/tulip/tulip_core.c	15 Dec 2004 17:18:39 -0000
@@ -22,7 +22,7 @@
 #else
 #define DRV_VERSION	"1.1.13"
 #endif
-#define DRV_RELDATE	"May 11, 2002"
+#define DRV_RELDATE	"December 15, 2004"
 
 
 #include <linux/module.h>

  reply	other threads:[~2004-12-15 17:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <41BAAC04.6090706@pobox.com>
2004-12-12 21:48 ` Grant Grundler
2004-12-13  2:19   ` frahm
2004-12-13  3:36     ` Grant Grundler
     [not found]   ` <200412130313.iBD3DAF4004365@albireo.free.fr>
2004-12-13  3:59     ` Grant Grundler
2004-12-13 11:52       ` FRAHM Klaus
2004-12-15  0:57       ` John W. Linville
2004-12-15 17:21         ` Grant Grundler [this message]
2004-12-13  3:14 frahm

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20041215172128.GB27818@colo.lackof.org \
    --to=grundler@parisc-linux.org \
    --cc=frahm@irsamc.ups-tlse.fr \
    --cc=jgarzik@pobox.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®