net: hdlc_fr: add braces {} to all arms of the statement
authorPeng Li <lipeng321@huawei.com>
Fri, 28 May 2021 00:12:47 +0000 (08:12 +0800)
committerJakub Kicinski <kuba@kernel.org>
Fri, 28 May 2021 21:08:11 +0000 (14:08 -0700)
Braces {} should be used on all arms of this statement.

Signed-off-by: Peng Li <lipeng321@huawei.com>
Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/wan/hdlc_fr.c

index 77b4f65..5c2a2ec 100644 (file)
@@ -613,10 +613,10 @@ static void fr_timer(struct timer_list *t)
                fr_set_link_state(reliable, dev);
        }
 
-       if (state(hdlc)->settings.dce)
+       if (state(hdlc)->settings.dce) {
                state(hdlc)->timer.expires = jiffies +
                        state(hdlc)->settings.t392 * HZ;
-       else {
+       else {
                if (state(hdlc)->n391cnt)
                        state(hdlc)->n391cnt--;
 
@@ -671,8 +671,9 @@ static int fr_lmi_recv(struct net_device *dev, struct sk_buff *skb)
                        return 1;
                }
                i = 7;
-       } else
+       } else {
                i = 6;
+       }
 
        if (skb->data[i] != (lmi == LMI_CCITT ? LMI_CCITT_REPTYPE :
                             LMI_ANSI_CISCO_REPTYPE)) {
@@ -1013,8 +1014,9 @@ static void fr_start(struct net_device *dev)
                /* First poll after 1 s */
                state(hdlc)->timer.expires = jiffies + HZ;
                add_timer(&state(hdlc)->timer);
-       } else
+       } else {
                fr_set_link_state(1, dev);
+       }
 }
 
 static void fr_stop(struct net_device *dev)