Commit ad390d2d authored by Alan Cox's avatar Alan Cox Committed by Jeff Garzik

3c501: Further coding style fixes

Signed-off-by: default avatarAlan Cox <alan@redhat.com>
Signed-off-by: default avatarJeff Garzik <jeff@garzik.org>
parent cd7e9034
...@@ -485,9 +485,7 @@ static int el_start_xmit(struct sk_buff *skb, struct net_device *dev) ...@@ -485,9 +485,7 @@ static int el_start_xmit(struct sk_buff *skb, struct net_device *dev)
printk(KERN_DEBUG "%s: burped during tx load.\n", printk(KERN_DEBUG "%s: burped during tx load.\n",
dev->name); dev->name);
spin_lock_irqsave(&lp->lock, flags); spin_lock_irqsave(&lp->lock, flags);
} } while (1);
while (1);
} }
/** /**
...@@ -612,7 +610,8 @@ static irqreturn_t el_interrupt(int irq, void *dev_id) ...@@ -612,7 +610,8 @@ static irqreturn_t el_interrupt(int irq, void *dev_id)
dev->stats.tx_packets++; dev->stats.tx_packets++;
if (el_debug > 6) if (el_debug > 6)
printk(KERN_DEBUG " Tx succeeded %s\n", printk(KERN_DEBUG " Tx succeeded %s\n",
(txsr & TX_RDY) ? "." : "but tx is busy!"); (txsr & TX_RDY) ? "." :
"but tx is busy!");
/* /*
* This is safe the interrupt is atomic WRT itself. * This is safe the interrupt is atomic WRT itself.
*/ */
...@@ -693,7 +692,8 @@ static void el_receive(struct net_device *dev) ...@@ -693,7 +692,8 @@ static void el_receive(struct net_device *dev)
if (pkt_len < 60 || pkt_len > 1536) { if (pkt_len < 60 || pkt_len > 1536) {
if (el_debug) if (el_debug)
printk(KERN_DEBUG "%s: bogus packet, length=%d\n", dev->name, pkt_len); printk(KERN_DEBUG "%s: bogus packet, length=%d\n",
dev->name, pkt_len);
dev->stats.rx_over_errors++; dev->stats.rx_over_errors++;
return; return;
} }
...@@ -711,7 +711,8 @@ static void el_receive(struct net_device *dev) ...@@ -711,7 +711,8 @@ static void el_receive(struct net_device *dev)
outw(0x00, GP_LOW); outw(0x00, GP_LOW);
if (skb == NULL) { if (skb == NULL) {
printk(KERN_INFO "%s: Memory squeeze, dropping packet.\n", dev->name); printk(KERN_INFO "%s: Memory squeeze, dropping packet.\n",
dev->name);
dev->stats.rx_dropped++; dev->stats.rx_dropped++;
return; return;
} else { } else {
...@@ -748,7 +749,8 @@ static void el_reset(struct net_device *dev) ...@@ -748,7 +749,8 @@ static void el_reset(struct net_device *dev)
if (el_debug > 2) if (el_debug > 2)
printk(KERN_INFO "3c501 reset..."); printk(KERN_INFO "3c501 reset...");
outb(AX_RESET, AX_CMD); /* Reset the chip */ outb(AX_RESET, AX_CMD); /* Reset the chip */
outb(AX_LOOP, AX_CMD); /* Aux control, irq and loopback enabled */ /* Aux control, irq and loopback enabled */
outb(AX_LOOP, AX_CMD);
{ {
int i; int i;
for (i = 0; i < 6; i++) /* Set the station address. */ for (i = 0; i < 6; i++) /* Set the station address. */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment