netdev
[Top] [All Lists]

[patch 2.6.12-rc6] 3c59x: remove superfluous vortex_debug test from boom

To: netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
Subject: [patch 2.6.12-rc6] 3c59x: remove superfluous vortex_debug test from boomerang_start_xmit
From: "John W. Linville" <linville@xxxxxxxxxxxxx>
Date: Fri, 10 Jun 2005 10:27:02 -0400
Cc: akpm@xxxxxxxx, jgarzik@xxxxxxxxx
Mail-followup-to: netdev@xxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, akpm@xxxxxxxx, jgarzik@xxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.4.1i
Remove the superfluous test of "if (vortex_debug > 3)" inside the
"if (vortex_debug > 6)" clause early in boomerang_start_xmit.

Signed-off-by: John W. Linville <linville@xxxxxxxxxxxxx>
---
I stumbled across this while looking at something else...

 drivers/net/3c59x.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/net/3c59x.c b/drivers/net/3c59x.c
--- a/drivers/net/3c59x.c
+++ b/drivers/net/3c59x.c
@@ -2202,9 +2202,8 @@ boomerang_start_xmit(struct sk_buff *skb
 
        if (vortex_debug > 6) {
                printk(KERN_DEBUG "boomerang_start_xmit()\n");
-               if (vortex_debug > 3)
-                       printk(KERN_DEBUG "%s: Trying to send a packet, Tx 
index %d.\n",
-                                  dev->name, vp->cur_tx);
+               printk(KERN_DEBUG "%s: Trying to send a packet, Tx index %d.\n",
+                          dev->name, vp->cur_tx);
        }
 
        if (vp->cur_tx - vp->dirty_tx >= TX_RING_SIZE) {
-- 
John W. Linville
linville@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>