qlge: removing unreachable block of code
authorBreno Leitao <leitao@linux.vnet.ibm.com>
Thu, 4 Feb 2010 10:11:18 +0000 (10:11 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 4 Feb 2010 23:42:27 +0000 (15:42 -0800)
Currently the qlge_change_mtu() is never called if the new_mtu is
equal current MTU, due this condition on dev_set_mtu():

        if (new_mtu == dev->mtu)
                return 0;

So, this block of code is never reached and is being removed.

Signed-off-by: Breno Leitao <leitao@linux.vnet.ibm.com>
Signed-off-by: Ron Mercer <ron.mercer@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/qlge/qlge_main.c

index 87a40d168b3666033913d7897a682c6b51aabd49..0c9e00f36cd9692fdd6493e90cd3b10966e47bd1 100644 (file)
@@ -4123,9 +4123,6 @@ static int qlge_change_mtu(struct net_device *ndev, int new_mtu)
                QPRINTK(qdev, IFUP, ERR, "Changing to jumbo MTU.\n");
        } else if (ndev->mtu == 9000 && new_mtu == 1500) {
                QPRINTK(qdev, IFUP, ERR, "Changing to normal MTU.\n");
-       } else if ((ndev->mtu == 1500 && new_mtu == 1500) ||
-                  (ndev->mtu == 9000 && new_mtu == 9000)) {
-               return 0;
        } else
                return -EINVAL;