[incubator-nuttx] branch master updated: netdev/carrier: check the IF stautus before carrier

[xiaoxiang]

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new 252985fc1d netdev/carrier: check the IF stautus before carrier
252985fc1d is described below

commit 252985fc1d5f95cc70459ebe4bf31c332f3c707e
Author: chao.an <anc...@xiaomi.com>
AuthorDate: Sat Jun 25 02:30:10 2022 +0800

    netdev/carrier: check the IF stautus before carrier
    
    Signed-off-by: chao.an <anc...@xiaomi.com>
---
 net/netdev/netdev_carrier.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/netdev/netdev_carrier.c b/net/netdev/netdev_carrier.c
index fe087649d6..630bb76b0a 100644
--- a/net/netdev/netdev_carrier.c
+++ b/net/netdev/netdev_carrier.c
@@ -60,7 +60,7 @@
 
 int netdev_carrier_on(FAR struct net_driver_s *dev)
 {
-  if (dev)
+  if (dev && !IFF_IS_RUNNING(dev->d_flags))
     {
       dev->d_flags |= IFF_RUNNING;
       netlink_device_notify(dev);
@@ -87,7 +87,7 @@ int netdev_carrier_on(FAR struct net_driver_s *dev)
 
 int netdev_carrier_off(FAR struct net_driver_s *dev)
 {
-  if (dev)
+  if (dev && IFF_IS_RUNNING(dev->d_flags))
     {
       dev->d_flags &= ~IFF_RUNNING;
       netlink_device_notify(dev);