]> Pileus Git - ~andy/linux/commitdiff
ipv6: fix checkpatch errors in net/ipv6/addrconf.c
authordingtianhong <dingtianhong@huawei.com>
Sat, 17 Aug 2013 02:27:15 +0000 (10:27 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 20 Aug 2013 22:05:03 +0000 (15:05 -0700)
ERROR: code indent should use tabs where possible: fix 2.
ERROR: do not use assignment in if condition: fix 5.

Signed-off-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/addrconf.c

index 3ca23e89f517a6a2db31a3978957c69c305f4d43..21638d41501683c27fb05c9102e39b0c64d9adf0 100644 (file)
@@ -1110,8 +1110,8 @@ retry:
        spin_unlock_bh(&ifp->lock);
 
        regen_advance = idev->cnf.regen_max_retry *
-                       idev->cnf.dad_transmits *
-                       idev->nd_parms->retrans_time / HZ;
+                       idev->cnf.dad_transmits *
+                       idev->nd_parms->retrans_time / HZ;
        write_unlock(&idev->lock);
 
        /* A temporary address is created only if this calculated Preferred
@@ -2501,7 +2501,8 @@ static int inet6_addr_del(struct net *net, int ifindex, const struct in6_addr *p
        if (!dev)
                return -ENODEV;
 
-       if ((idev = __in6_dev_get(dev)) == NULL)
+       idev = __in6_dev_get(dev);
+       if (!idev)
                return -ENXIO;
 
        read_lock_bh(&idev->lock);
@@ -2640,7 +2641,8 @@ static void init_loopback(struct net_device *dev)
 
        ASSERT_RTNL();
 
-       if ((idev = ipv6_find_idev(dev)) == NULL) {
+       idev = ipv6_find_idev(dev);
+       if (!idev) {
                pr_debug("%s: add_dev failed\n", __func__);
                return;
        }
@@ -2738,7 +2740,8 @@ static void addrconf_sit_config(struct net_device *dev)
         * our v4 addrs in the tunnel
         */
 
-       if ((idev = ipv6_find_idev(dev)) == NULL) {
+       idev = ipv6_find_idev(dev);
+       if (!idev) {
                pr_debug("%s: add_dev failed\n", __func__);
                return;
        }
@@ -2770,7 +2773,8 @@ static void addrconf_gre_config(struct net_device *dev)
 
        ASSERT_RTNL();
 
-       if ((idev = ipv6_find_idev(dev)) == NULL) {
+       idev = ipv6_find_idev(dev);
+       if (!idev) {
                pr_debug("%s: add_dev failed\n", __func__);
                return;
        }
@@ -2801,11 +2805,11 @@ static void ip6_tnl_add_linklocal(struct inet6_dev *idev)
        struct net *net = dev_net(idev->dev);
 
        /* first try to inherit the link-local address from the link device */
-       if (idev->dev->iflink &&
-           (link_dev = __dev_get_by_index(net, idev->dev->iflink))) {
-               if (!ipv6_inherit_linklocal(idev, link_dev))
+       if (idev->dev->iflink)
+               link_dev = __dev_get_by_index(net, idev->dev->iflink);
+               if (link_dev && !ipv6_inherit_linklocal(idev, link_dev))
                        return;
-       }
+
        /* then try to inherit it from any device */
        for_each_netdev(net, link_dev) {
                if (!ipv6_inherit_linklocal(idev, link_dev))