| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: BCM5704 performance questions., Jason Lunz |
|---|---|
| Next by Date: | RE: ipw2100: firmware problem, Lee Revell |
| Previous by Thread: | [no subject], Zoran Bosic (ZG/ETK) |
| Next by Thread: | Re: [patch 2.6.12-rc6] 3c59x: remove superfluous vortex_debug test from boomerang_start_xmit, Jeff Garzik |
| Indexes: | [Date] [Thread] [Top] [All Lists] |