sh_eth: do not call netif_start_queue() from sh_eth_dev_init()
authorSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Sun, 13 Mar 2016 22:09:53 +0000 (01:09 +0300)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Mar 2016 23:24:58 +0000 (19:24 -0400)
Iff  sh_eth_phy_start() call fails in sh_eth_open(), the netif_start_queue()
call done by sh_eth_dev_init()  is not undone.  In order to deal with that,
stop calling netif_start_queue()  from there, so that it can be called only
when the device is fully opened and sh_eth_dev_init() only deals with the
hardware initialization, symmetrically to sh_eth_dev_exit()...

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/renesas/sh_eth.c

index 8385d927adc30450ca70bc2978dfc518831d5a34..004e2d7560fd86e289980a2526a8e6e9f4fb77c3 100644 (file)
@@ -1312,8 +1312,6 @@ static int sh_eth_dev_init(struct net_device *ndev, bool start)
        if (start) {
                /* Setting the Rx mode will start the Rx process. */
                sh_eth_write(ndev, EDRRR_R, EDRRR);
-
-               netif_start_queue(ndev);
        }
 
        return ret;
@@ -2261,6 +2259,8 @@ static int sh_eth_open(struct net_device *ndev)
        if (ret)
                goto out_free_irq;
 
+       netif_start_queue(ndev);
+
        mdp->is_opened = 1;
 
        return ret;
@@ -2304,6 +2304,8 @@ static void sh_eth_tx_timeout(struct net_device *ndev)
 
        /* device init */
        sh_eth_dev_init(ndev, true);
+
+       netif_start_queue(ndev);
 }
 
 /* Packet transmit function */