ksz884x: delete useless variable
authorJulia Lawall <Julia.Lawall@lip6.fr>
Mon, 13 Jan 2014 16:17:24 +0000 (17:17 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 15 Jan 2014 21:43:03 +0000 (13:43 -0800)
Delete a variable that is at most only assigned to a constant, but never
used otherwise.  In this code, it is the variable result that is used for
the return code, not rc.

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

// <smpl>
@@
type T;
identifier i;
constant c;
@@

-T i;
<... when != i
-i = c;
...>
// </smpl>

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

index 8e9dad7..ce84dc2 100644 (file)
@@ -5853,15 +5853,12 @@ static int netdev_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
        struct dev_info *hw_priv = priv->adapter;
        struct ksz_hw *hw = &hw_priv->hw;
        struct ksz_port *port = &priv->port;
-       int rc;
        int result = 0;
        struct mii_ioctl_data *data = if_mii(ifr);
 
        if (down_interruptible(&priv->proc_sem))
                return -ERESTARTSYS;
 
-       /* assume success */
-       rc = 0;
        switch (cmd) {
        /* Get address of MII PHY in use. */
        case SIOCGMIIPHY: