net/ariadne: delete unneeded call to netdev_priv
authorJulia Lawall <Julia.Lawall@lip6.fr>
Thu, 15 May 2014 03:43:19 +0000 (05:43 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 15 May 2014 20:57:47 +0000 (16:57 -0400)
Netdev_priv is an accessor function, and has no purpose if its result is
not used.

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

// <smpl>
@@ local idexpression x; @@
-x = netdev_priv(...);
... when != x
// </smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/amd/ariadne.c

index b08101b31b8bc547ffbfb486f87abc7a18f92686..968b7bfac8fcaa9af767c57c474a83b3229b09a0 100644 (file)
@@ -718,7 +718,6 @@ static int ariadne_init_one(struct zorro_dev *z,
        unsigned long mem_start = board + ARIADNE_RAM;
        struct resource *r1, *r2;
        struct net_device *dev;
-       struct ariadne_private *priv;
        u32 serial;
        int err;
 
@@ -738,8 +737,6 @@ static int ariadne_init_one(struct zorro_dev *z,
                return -ENOMEM;
        }
 
-       priv = netdev_priv(dev);
-
        r1->name = dev->name;
        r2->name = dev->name;