diff options
author | Zhang Shengju <zhangshengju@cmss.chinamobile.com> | 2017-04-26 17:49:38 +0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-04-27 16:31:05 -0400 |
commit | 0575c86b5dd596253bdfc0365b570d67b1a12523 (patch) | |
tree | d8b5e683dfca962f0e495b919c283132165b6a1a /net/core | |
parent | 8ecbc40ada116f2f7d6b61cd646802c87b7c5c7d (diff) | |
download | op-kernel-dev-0575c86b5dd596253bdfc0365b570d67b1a12523.zip op-kernel-dev-0575c86b5dd596253bdfc0365b570d67b1a12523.tar.gz |
net: remove unnecessary carrier status check
Since netif_carrier_on() will do nothing if device's carrier is already
on, so it's unnecessary to do carrier status check.
It's the same for netif_carrier_off().
Signed-off-by: Zhang Shengju <zhangshengju@cmss.chinamobile.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core')
-rw-r--r-- | net/core/dev.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 3361ee8..8371a01 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -7245,13 +7245,10 @@ void netif_stacked_transfer_operstate(const struct net_device *rootdev, else netif_dormant_off(dev); - if (netif_carrier_ok(rootdev)) { - if (!netif_carrier_ok(dev)) - netif_carrier_on(dev); - } else { - if (netif_carrier_ok(dev)) - netif_carrier_off(dev); - } + if (netif_carrier_ok(rootdev)) + netif_carrier_on(dev); + else + netif_carrier_off(dev); } EXPORT_SYMBOL(netif_stacked_transfer_operstate); |