Skip to content

Commit

Permalink
r8169: fix an if condition
Browse files Browse the repository at this point in the history
There is an extra semi-colon so __rtl8169_set_features() is called every
time.

Fixes: 929a031 ('r8169: adjust __rtl8169_set_features')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Hayes Wang <hayeswang@realtek.com>--
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Dan Carpenter authored and David S. Miller committed Sep 22, 2014
1 parent d70b113 commit 85911d7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/net/ethernet/realtek/r8169.c
Original file line number Diff line number Diff line change
Expand Up @@ -1818,7 +1818,7 @@ static int rtl8169_set_features(struct net_device *dev,
features &= NETIF_F_RXALL | NETIF_F_RXCSUM | NETIF_F_HW_VLAN_CTAG_RX;

rtl_lock_work(tp);
if (features ^ dev->features);
if (features ^ dev->features)
__rtl8169_set_features(dev, features);
rtl_unlock_work(tp);

Expand Down

0 comments on commit 85911d7

Please sign in to comment.