summary refs log tree commit diff
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2010-01-18 00:29:19 -0800
committerDavid S. Miller <davem@davemloft.net>2010-01-18 00:29:19 -0800
commit2ae3111eafd797e24cc390bb5ce6a8fa0deacdb2 (patch)
tree14eeb14eb9a4d4e80a8714b88470ca9eaf8dde78
parent354fcd7774a00809ebefdba75e747364cb22a940 (diff)
downloadlinux-2ae3111eafd797e24cc390bb5ce6a8fa0deacdb2.tar.gz
drivers/net: Eliminate useless code
The variable qdev is initialized twice to the same (side effect-free)
expression.  Drop one initialization.

A simplified version of the semantic match that finds this problem is:
(http://coccinelle.lip6.fr/)

// <smpl>
@forall@
idexpression *x;
identifier f!=ERR_PTR;
@@

x = f(...)
... when != x
(
x = f(...,<+...x...+>,...)
|
* x = f(...)
)
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--drivers/net/qla3xxx.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/net/qla3xxx.c b/drivers/net/qla3xxx.c
index f922294fd349..4ef0afbcbe1b 100644
--- a/drivers/net/qla3xxx.c
+++ b/drivers/net/qla3xxx.c
@@ -4087,7 +4087,6 @@ static void __devexit ql3xxx_remove(struct pci_dev *pdev)
 	struct ql3_adapter *qdev = netdev_priv(ndev);
 
 	unregister_netdev(ndev);
-	qdev = netdev_priv(ndev);
 
 	ql_disable_interrupts(qdev);