8390/8390p: Fix compat netdev ops handling.
David S. Miller [Mon, 15 Dec 2008 23:14:59 +0000 (15:14 -0800)]
Based upon a report from Randy Dunlap.

The compat netdev ops assignments need to happen in
8390.c and 8390p.c, not lib8390.c, as only the type
specific code can assign the correct function pointers.

Signed-off-by: David S. Miller <davem@davemloft.net>

drivers/net/8390.c
drivers/net/8390p.c
drivers/net/lib8390.c

index 029ad08..fbe609a 100644 (file)
@@ -72,7 +72,16 @@ EXPORT_SYMBOL(ei_netdev_ops);
 
 struct net_device *__alloc_ei_netdev(int size)
 {
-       return ____alloc_ei_netdev(size);
+       struct net_device *dev = ____alloc_ei_netdev(size);
+#ifdef CONFIG_COMPAT_NET_DEV_OPS
+       if (dev) {
+               dev->hard_start_xmit = ei_start_xmit;
+               dev->get_stats  = ei_get_stats;
+               dev->set_multicast_list = ei_set_multicast_list;
+               dev->tx_timeout = ei_tx_timeout;
+       }
+#endif
+       return dev;
 }
 EXPORT_SYMBOL(__alloc_ei_netdev);
 
index 9c916d4..ee70b35 100644 (file)
@@ -77,7 +77,16 @@ EXPORT_SYMBOL(eip_netdev_ops);
 
 struct net_device *__alloc_eip_netdev(int size)
 {
-       return ____alloc_ei_netdev(size);
+       struct net_device *dev = ____alloc_ei_netdev(size);
+#ifdef CONFIG_COMPAT_NET_DEV_OPS
+       if (dev) {
+               dev->hard_start_xmit = eip_start_xmit;
+               dev->get_stats  = eip_get_stats;
+               dev->set_multicast_list = eip_set_multicast_list;
+               dev->tx_timeout = eip_tx_timeout;
+       }
+#endif
+       return dev;
 }
 EXPORT_SYMBOL(__alloc_eip_netdev);
 
index 1d36ca4..789b6cb 100644 (file)
@@ -1010,12 +1010,6 @@ static void ethdev_setup(struct net_device *dev)
        if (ei_debug > 1)
                printk(version);
 
-#ifdef CONFIG_COMPAT_NET_DEV_OPS
-       dev->hard_start_xmit = ei_start_xmit;
-       dev->get_stats  = ei_get_stats;
-       dev->set_multicast_list = ei_set_multicast_list;
-       dev->tx_timeout = __ei_tx_timeout;
-#endif
        ether_setup(dev);
 
        spin_lock_init(&ei_local->page_lock);