summaryrefslogtreecommitdiffstats
path: root/drivers/net/ethernet/8390/ne.c
diff options
context:
space:
mode:
authorFinn Thain <fthain@telegraphics.com.au>2018-02-18 21:39:17 -0500
committerDavid S. Miller <davem@davemloft.net>2018-02-21 14:14:04 -0500
commit646fe03b0d5fb678f0901e25eb184ae8860744fa (patch)
tree18f8bb2053912c4743b226d55672c9568704ebdf /drivers/net/ethernet/8390/ne.c
parent73219de29486637c82925e963380bda020e4edfb (diff)
downloadlinux-646fe03b0d5fb678f0901e25eb184ae8860744fa.tar.bz2
net/8390: Fix msg_enable patch snafu
The lib8390 module parameter 'msg_enable' doesn't do anything useful: it causes an ancient version string to be logged. Remove redundant code that logs the same string. In ne.c and wd.c, the value of ei_local->msg_enable is used before being assigned. Use ne_msg_enable and wd_msg_enable, respectively. Most of the other 8390 drivers never assign ei_local->msg_enable. Use the 'msg_enable' module parameter from lib8390 as the default value. Eliminate the pointless static and local variables. Clean up an indentation mistake. All of these issues originated from the same patch. Cc: Russell King <linux@armlinux.org.uk> Fixes: c45f812f0280 ("8390 : Replace ei_debug with msg_enable/NETIF_MSG_* feature") Tested-by: Stan Johnson <userm57@yahoo.com> Signed-off-by: Finn Thain <fthain@telegraphics.com.au> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/8390/ne.c')
-rw-r--r--drivers/net/ethernet/8390/ne.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/8390/ne.c b/drivers/net/ethernet/8390/ne.c
index 66f47987e2a2..4cdff6e6af89 100644
--- a/drivers/net/ethernet/8390/ne.c
+++ b/drivers/net/ethernet/8390/ne.c
@@ -485,7 +485,7 @@ static int __init ne_probe1(struct net_device *dev, unsigned long ioaddr)
mdelay(10); /* wait 10ms for interrupt to propagate */
outb_p(0x00, ioaddr + EN0_IMR); /* Mask it again. */
dev->irq = probe_irq_off(cookie);
- if (netif_msg_probe(ei_local))
+ if (ne_msg_enable & NETIF_MSG_PROBE)
pr_cont(" autoirq is %d", dev->irq);
} else if (dev->irq == 2)
/* Fixup for users that don't know that IRQ 2 is really IRQ 9,