drivers/net: remove unnecessary flush_scheduled_work() calls
authorTejun Heo <tj@kernel.org>
Sun, 12 Dec 2010 15:45:14 +0000 (16:45 +0100)
committerTejun Heo <tj@kernel.org>
Sun, 12 Dec 2010 15:45:14 +0000 (16:45 +0100)
janz-ican3, sh_eth, skge and vxge don't use workqueue at all and there
is no reason to flush the system_wq.  Drop flush_scheduled_work()
calls and references to workqueue.

Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Wolfgang Grandegger <wg@grandegger.com>
Cc: Stephen Hemminger <shemminger@linux-foundation.org>
Cc: Ramkrishna Vepa <ramkrishna.vepa@exar.com>
Cc: Sivakumar Subramani <sivakumar.subramani@exar.com>
Cc: Sreenivasa Honnur <sreenivasa.honnur@exar.com>
Cc: Jon Mason <jon.mason@exar.com>
Cc: netdev@vger.kernel.org
drivers/net/can/janz-ican3.c
drivers/net/sh_eth.c
drivers/net/sh_eth.h
drivers/net/skge.c
drivers/net/vxge/vxge-main.c

index 6e533dcc36c0d64292b94330b7fe2cb3898c4a6a..b9a6d7a5a739453986ad73db51e8fa8155fa271f 100644 (file)
@@ -1114,11 +1114,6 @@ static bool ican3_txok(struct ican3_dev *mod)
 /*
  * Recieve one CAN frame from the hardware
  *
- * This works like the core of a NAPI function, but is intended to be called
- * from workqueue context instead. This driver already needs a workqueue to
- * process control messages, so we use the workqueue instead of using NAPI.
- * This was done to simplify locking.
- *
  * CONTEXT: must be called from user context
  */
 static int ican3_recv_skb(struct ican3_dev *mod)
@@ -1251,7 +1246,6 @@ static irqreturn_t ican3_irq(int irq, void *dev_id)
  * Reset an ICAN module to its power-on state
  *
  * CONTEXT: no network device registered
- * LOCKING: work function disabled
  */
 static int ican3_reset_module(struct ican3_dev *mod)
 {
@@ -1262,9 +1256,6 @@ static int ican3_reset_module(struct ican3_dev *mod)
        /* disable interrupts so no more work is scheduled */
        iowrite8(1 << mod->num, &mod->ctrl->int_disable);
 
-       /* flush any pending work */
-       flush_scheduled_work();
-
        /* the first unallocated page in the DPM is #9 */
        mod->free_page = DPM_FREE_START;
 
index b12660d7233839d4d37d547592883545e38a1b7b..819c1750e2abf750cbb73ad779dd695a1464326d 100644 (file)
@@ -1552,7 +1552,6 @@ static int sh_eth_drv_remove(struct platform_device *pdev)
 
        sh_mdio_release(ndev);
        unregister_netdev(ndev);
-       flush_scheduled_work();
        pm_runtime_disable(&pdev->dev);
        free_netdev(ndev);
        platform_set_drvdata(pdev, NULL);
index 8b47763958f2d516b7a7690f233091dd67850420..efa64221eedeab1e0cbfe98d8317c37a38381670 100644 (file)
@@ -26,7 +26,6 @@
 #include <linux/module.h>
 #include <linux/kernel.h>
 #include <linux/spinlock.h>
-#include <linux/workqueue.h>
 #include <linux/netdevice.h>
 #include <linux/phy.h>
 
index 220e0398f1d519130c63628b3e0461c45227fa7c..8c1404b58382b2fc644c32c095d7e62b21871bec 100644 (file)
@@ -4012,8 +4012,6 @@ static void __devexit skge_remove(struct pci_dev *pdev)
        if (!hw)
                return;
 
-       flush_scheduled_work();
-
        dev1 = hw->dev[1];
        if (dev1)
                unregister_netdev(dev1);
index b771e4b2ca9e8be1c25e5a3f741127e6bcedf124..537ad874f11cb2f91e6910ac2ea0a07893e49e92 100644 (file)
@@ -3439,8 +3439,6 @@ static void vxge_device_unregister(struct __vxge_hw_device *hldev)
 
        strncpy(buf, dev->name, IFNAMSIZ);
 
-       flush_scheduled_work();
-
        /* in 2.6 will call stop() if device is up */
        unregister_netdev(dev);