diff options
author | Russell King <rmk@dyn-67.arm.linux.org.uk> | 2005-10-02 13:12:03 -0400 |
---|---|---|
committer | Russell King <rmk+kernel@arm.linux.org.uk> | 2005-10-02 13:12:03 -0400 |
commit | 487fd4eb1445407c9760af08b0b34c3f4cdb4afc (patch) | |
tree | d4fe20f822429352184159985b5b330bf2fd4c2d | |
parent | 0e3a64e2162f971180bf1fdd91c263dbdfcfd385 (diff) |
[ARM] Fix init printk for EBSA110 network driver, and link timer
Arrange for the initialisation printks to happen after we've
registered the network interface, so we know what name the
device is. Also, check the link every 500ms (and use
msecs_to_jiffies.)
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
-rw-r--r-- | drivers/net/arm/am79c961a.c | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/drivers/net/arm/am79c961a.c b/drivers/net/arm/am79c961a.c index 9f6eb39cf492..c56d86d371a9 100644 --- a/drivers/net/arm/am79c961a.c +++ b/drivers/net/arm/am79c961a.c | |||
@@ -15,16 +15,13 @@ | |||
15 | */ | 15 | */ |
16 | #include <linux/kernel.h> | 16 | #include <linux/kernel.h> |
17 | #include <linux/types.h> | 17 | #include <linux/types.h> |
18 | #include <linux/fcntl.h> | ||
19 | #include <linux/interrupt.h> | 18 | #include <linux/interrupt.h> |
20 | #include <linux/ioport.h> | 19 | #include <linux/ioport.h> |
21 | #include <linux/in.h> | ||
22 | #include <linux/slab.h> | 20 | #include <linux/slab.h> |
23 | #include <linux/string.h> | 21 | #include <linux/string.h> |
24 | #include <linux/errno.h> | 22 | #include <linux/errno.h> |
25 | #include <linux/netdevice.h> | 23 | #include <linux/netdevice.h> |
26 | #include <linux/etherdevice.h> | 24 | #include <linux/etherdevice.h> |
27 | #include <linux/skbuff.h> | ||
28 | #include <linux/delay.h> | 25 | #include <linux/delay.h> |
29 | #include <linux/init.h> | 26 | #include <linux/init.h> |
30 | #include <linux/crc32.h> | 27 | #include <linux/crc32.h> |
@@ -33,7 +30,6 @@ | |||
33 | #include <asm/system.h> | 30 | #include <asm/system.h> |
34 | #include <asm/irq.h> | 31 | #include <asm/irq.h> |
35 | #include <asm/io.h> | 32 | #include <asm/io.h> |
36 | #include <asm/dma.h> | ||
37 | 33 | ||
38 | #define TX_BUFFERS 15 | 34 | #define TX_BUFFERS 15 |
39 | #define RX_BUFFERS 25 | 35 | #define RX_BUFFERS 25 |
@@ -288,7 +284,7 @@ static void am79c961_timer(unsigned long data) | |||
288 | else if (!lnkstat && carrier) | 284 | else if (!lnkstat && carrier) |
289 | netif_carrier_off(dev); | 285 | netif_carrier_off(dev); |
290 | 286 | ||
291 | mod_timer(&priv->timer, jiffies + 5*HZ); | 287 | mod_timer(&priv->timer, jiffies + msecs_to_jiffies(500)); |
292 | } | 288 | } |
293 | 289 | ||
294 | /* | 290 | /* |
@@ -709,13 +705,9 @@ static int __init am79c961_init(void) | |||
709 | goto release; | 705 | goto release; |
710 | 706 | ||
711 | am79c961_banner(); | 707 | am79c961_banner(); |
712 | printk(KERN_INFO "%s: ether address ", dev->name); | ||
713 | 708 | ||
714 | /* Retrive and print the ethernet address. */ | 709 | for (i = 0; i < 6; i++) |
715 | for (i = 0; i < 6; i++) { | ||
716 | dev->dev_addr[i] = inb(dev->base_addr + i * 2) & 0xff; | 710 | dev->dev_addr[i] = inb(dev->base_addr + i * 2) & 0xff; |
717 | printk (i == 5 ? "%02x\n" : "%02x:", dev->dev_addr[i]); | ||
718 | } | ||
719 | 711 | ||
720 | spin_lock_init(&priv->chip_lock); | 712 | spin_lock_init(&priv->chip_lock); |
721 | init_timer(&priv->timer); | 713 | init_timer(&priv->timer); |
@@ -736,8 +728,14 @@ static int __init am79c961_init(void) | |||
736 | #endif | 728 | #endif |
737 | 729 | ||
738 | ret = register_netdev(dev); | 730 | ret = register_netdev(dev); |
739 | if (ret == 0) | 731 | if (ret == 0) { |
732 | printk(KERN_INFO "%s: ether address ", dev->name); | ||
733 | |||
734 | for (i = 0; i < 6; i++) | ||
735 | printk (i == 5 ? "%02x\n" : "%02x:", dev->dev_addr[i]); | ||
736 | |||
740 | return 0; | 737 | return 0; |
738 | } | ||
741 | 739 | ||
742 | release: | 740 | release: |
743 | release_region(dev->base_addr, 0x18); | 741 | release_region(dev->base_addr, 0x18); |