[PATCH] allow modular mv64340_eth

[PATCH] allow modular mv64340_eth

[Christoph Hellwig]

Only tested together with some additional patches to make the driver
work on PPC that need more work, but the changes are obvious.


--- a/drivers/net/Kconfig~	2004-07-06 14:32:32.380761120 +0200
+++ b/drivers/net/Kconfig	2004-07-06 14:32:40.494527640 +0200
@@ -2132,7 +2132,7 @@
 	  will be called tg3.  This is recommended.
 
 config MV64340_ETH
-	bool "MV-64340 Ethernet support"
+	tristate "MV-64340 Ethernet support"
 	depends on MOMENCO_OCELOT_C || MOMENCO_JAGUAR_ATX
 	help
 	  This driver supports the gigabit Ethernet on the Marvell MV64340
--- a/drivers/net/mv64340_eth.c~	2004-07-06 14:26:47.163242160 +0200
+++ b/drivers/net/mv64340_eth.c	2004-07-06 14:32:15.198373240 +0200
@@ -1316,7 +1316,7 @@
  * Input : number of port to initialize
  * Output : -ENONMEM if failed , 0 if success
  */
-static int mv64340_eth_init(int port_num)
+static struct net_device *mv64340_eth_init(int port_num)
 {
 	struct mv64340_private *mp;
 	struct net_device *dev;
@@ -1324,7 +1324,7 @@
 
 	dev = alloc_etherdev(sizeof(struct mv64340_private));
 	if (!dev)
-		return -ENOMEM;
+		return NULL;
 
 	mp = netdev_priv(dev);
 
@@ -1395,14 +1395,27 @@
 	printk("RX NAPI Enabled \n");
 #endif
 
-	return 0;
+	return dev;
 
 out_free_dev:
 	free_netdev(dev);
 
-	return err;
+	return NULL;
+}
+
+static void mv64340_eth_remove(struct net_device *dev)
+{
+	struct mv64340_private *mp = netdev_priv(dev);
+
+	unregister_netdev(dev);
+	flush_scheduled_work();
+	free_netdev(dev);
 }
 
+static struct net_device *mv64340_dev0;
+static struct net_device *mv64340_dev1;
+static struct net_device *mv64340_dev2;
+
 /*
  * mv64340_init_module
  *
@@ -1415,20 +1428,24 @@
 static int __init mv64340_init_module(void)
 {
 	printk(KERN_NOTICE "MV-64340 10/100/1000 Ethernet Driver\n");
+
 #ifdef CONFIG_MV64340_ETH_0
-	if (mv64340_eth_init(0)) {
+	mv64340_dev0 = mv64340_eth_init(0);
+	if (!mv64340_dev0) {
 		printk(KERN_ERR
 		       "Error registering MV-64360 ethernet port 0\n");
 	}
 #endif
 #ifdef CONFIG_MV64340_ETH_1
-	if (mv64340_eth_init(1)) {
+	mv64340_dev1 = mv64340_eth_init(1);
+	if (!mv64340_dev1) {
 		printk(KERN_ERR
 		       "Error registering MV-64360 ethernet port 1\n");
 	}
 #endif
 #ifdef CONFIG_MV64340_ETH_2
-	if (mv64340_eth_init(2)) {
+	mv64340_dev2 = mv64340_eth_init(2);
+	if (!mv64340_dev2) {
 		printk(KERN_ERR
 		       "Error registering MV-64360 ethernet port 2\n");
 	}
@@ -1445,9 +1462,14 @@
  *
  * Output : N/A
  */
-static void __init mv64340_cleanup_module(void)
+static void __exit mv64340_cleanup_module(void)
 {
-	/* Nothing to do here ! it's not a removable module */
+	if (mv64340_dev2)
+		mv64340_eth_remove(mv64340_dev2);
+	if (mv64340_dev1)
+		mv64340_eth_remove(mv64340_dev1);
+	if (mv64340_dev0)
+		mv64340_eth_remove(mv64340_dev0);
 }
 
 module_init(mv64340_init_module);

Re: [PATCH] allow modular mv64340_eth

[Jeff Garzik]

applied