diff options
| author | Bruce Allan <bruce.w.allan@intel.com> | 2012-01-31 06:37:54 +0000 |
|---|---|---|
| committer | Jeff Kirsher <jeffrey.t.kirsher@intel.com> | 2012-02-10 00:06:30 -0800 |
| commit | fe1e980f24697edb7d4e17cd74bbeae4a0388525 (patch) | |
| tree | 085f1120c8a08573a33991ac2c299f4872ef77be /drivers/net/ethernet/intel/e1000e/netdev.c | |
| parent | 4bcf053baf6b255e8c82c7ecd0d15954adb0379b (diff) | |
| download | lwn-fe1e980f24697edb7d4e17cd74bbeae4a0388525.tar.gz lwn-fe1e980f24697edb7d4e17cd74bbeae4a0388525.zip | |
e1000e: remove unnecessary returns from void functions
...and convert some goto's which simply return to just return.
Signed-off-by: Bruce Allan <bruce.w.allan@intel.com>
Tested-by: Aaron Brown <aaron.f.brown@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Diffstat (limited to 'drivers/net/ethernet/intel/e1000e/netdev.c')
| -rw-r--r-- | drivers/net/ethernet/intel/e1000e/netdev.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/drivers/net/ethernet/intel/e1000e/netdev.c b/drivers/net/ethernet/intel/e1000e/netdev.c index 293a7606e0f4..7ccd5f3cef61 100644 --- a/drivers/net/ethernet/intel/e1000e/netdev.c +++ b/drivers/net/ethernet/intel/e1000e/netdev.c @@ -221,7 +221,7 @@ static void e1000e_dump(struct e1000_adapter *adapter) /* Print Tx Ring Summary */ if (!netdev || !netif_running(netdev)) - goto exit; + return; dev_info(&adapter->pdev->dev, "Tx Ring Summary\n"); pr_info("Queue [NTU] [NTC] [bi(ntc)->dma ] leng ntw timestamp\n"); @@ -308,7 +308,7 @@ rx_ring_summary: /* Print Rx Ring */ if (!netif_msg_rx_status(adapter)) - goto exit; + return; dev_info(&adapter->pdev->dev, "Rx Ring Dump\n"); switch (adapter->rx_ps_pages) { @@ -449,9 +449,6 @@ rx_ring_summary: } } } - -exit: - return; } /** |