It's unnecessary and less readable after a clause ending in a goto.
Signed-off-by: Joe Perches <joe@perches.com>
Reviewed-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
 
                bond_dev_queue_xmit(bond, skb, tx_slave->dev);
                goto out;
-       } else {
-               if (tx_slave) {
-                       _lock_tx_hashtbl(bond);
-                       __tlb_clear_slave(bond, tx_slave, 0);
-                       _unlock_tx_hashtbl(bond);
-               }
+       }
+
+       if (tx_slave) {
+               _lock_tx_hashtbl(bond);
+               __tlb_clear_slave(bond, tx_slave, 0);
+               _unlock_tx_hashtbl(bond);
        }
 
        /* no suitable interface, frame not sent */