net: ethernet: Remove superfluous ether_setup after alloc_etherdev
authorTobias Klauser <tklauser@distanz.ch>
Thu, 2 Oct 2014 08:15:30 +0000 (10:15 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 3 Oct 2014 22:31:40 +0000 (15:31 -0700)
There is no need to call ether_setup after alloc_ethdev since it was
already called there.

Follow commits c706471b2601 ("net: axienet: remove unnecessary
ether_setup after alloc_etherdev") and 3c87dcbfb36c ("net: ll_temac:
Remove unnecessary ether_setup after alloc_etherdev") and fix the
pattern in all remaining ethernet drivers.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
19 files changed:
drivers/net/ethernet/adi/bfin_mac.c
drivers/net/ethernet/allwinner/sun4i-emac.c
drivers/net/ethernet/altera/altera_tse_main.c
drivers/net/ethernet/cadence/at91_ether.c
drivers/net/ethernet/calxeda/xgmac.c
drivers/net/ethernet/davicom/dm9000.c
drivers/net/ethernet/ethoc.c
drivers/net/ethernet/lantiq_etop.c
drivers/net/ethernet/moxa/moxart_ether.c
drivers/net/ethernet/netx-eth.c
drivers/net/ethernet/nuvoton/w90p910_ether.c
drivers/net/ethernet/nxp/lpc_eth.c
drivers/net/ethernet/qualcomm/qca_spi.c
drivers/net/ethernet/smsc/smc911x.c
drivers/net/ethernet/smsc/smc91x.c
drivers/net/ethernet/smsc/smsc911x.c
drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
drivers/net/ethernet/wiznet/w5100.c
drivers/net/ethernet/wiznet/w5300.c

index afa66847e10b60e09dd8c25776bd6e020cbfd7e9..8ed4d3408ef6d30512f4f6ca8f5ff3ee2146f457 100644 (file)
@@ -1692,9 +1692,6 @@ static int bfin_mac_probe(struct platform_device *pdev)
        lp->vlan1_mask = ETH_P_8021Q | mii_bus_data->vlan1_mask;
        lp->vlan2_mask = ETH_P_8021Q | mii_bus_data->vlan2_mask;
 
-       /* Fill in the fields of the device structure with ethernet values. */
-       ether_setup(ndev);
-
        ndev->netdev_ops = &bfin_mac_netdev_ops;
        ndev->ethtool_ops = &bfin_mac_ethtool_ops;
 
index 29b9f082475de10b290f579d72ef5240a133be78..1fcd5568a3520981fd6ac03b57a3a599c2876a6c 100644 (file)
@@ -878,8 +878,6 @@ static int emac_probe(struct platform_device *pdev)
        emac_powerup(ndev);
        emac_reset(db);
 
-       ether_setup(ndev);
-
        ndev->netdev_ops = &emac_netdev_ops;
        ndev->watchdog_timeo = msecs_to_jiffies(watchdog);
        ndev->ethtool_ops = &emac_ethtool_ops;
index 7330681574d20ccd02855beabedc9b01c8ae2b48..fc2d5556b71564b20ab12313de0e5b6769541d62 100644 (file)
@@ -1433,7 +1433,6 @@ static int altera_tse_probe(struct platform_device *pdev)
                goto err_free_netdev;
 
        /* initialize netdev */
-       ether_setup(ndev);
        ndev->mem_start = control_port->start;
        ndev->mem_end = control_port->end;
        ndev->netdev_ops = &altera_tse_netdev_ops;
index 4a79edaf388547d0cb5f7ca594d8ab20841efefc..4a24b9a6ad759255aeeaa3b4b18d56c475cd568a 100644 (file)
@@ -351,7 +351,6 @@ static int __init at91ether_probe(struct platform_device *pdev)
        if (res)
                goto err_disable_clock;
 
-       ether_setup(dev);
        dev->netdev_ops = &at91ether_netdev_ops;
        dev->ethtool_ops = &macb_ethtool_ops;
        platform_set_drvdata(pdev, dev);
index 25d6b2a10e4e6f7fb6b09d25706e11e2783bbea4..47bfea24b9e1b1bce64f07d5eb0424fc2a9b7e0f 100644 (file)
@@ -1735,7 +1735,6 @@ static int xgmac_probe(struct platform_device *pdev)
        SET_NETDEV_DEV(ndev, &pdev->dev);
        priv = netdev_priv(ndev);
        platform_set_drvdata(pdev, ndev);
-       ether_setup(ndev);
        ndev->netdev_ops = &xgmac_netdev_ops;
        ndev->ethtool_ops = &xgmac_ethtool_ops;
        spin_lock_init(&priv->stats_lock);
index 70089c29d307a43643222959202228403c31e97f..f3ba840cbf7b065fa4b73828e75d60af20069dbb 100644 (file)
@@ -1613,9 +1613,6 @@ dm9000_probe(struct platform_device *pdev)
 
        /* from this point we assume that we have found a DM9000 */
 
-       /* driver system function */
-       ether_setup(ndev);
-
        ndev->netdev_ops        = &dm9000_netdev_ops;
        ndev->watchdog_timeo    = msecs_to_jiffies(watchdog);
        ndev->ethtool_ops       = &dm9000_ethtool_ops;
index f3658bdb64cc4265de972cfc2bbd25157e95a680..0bc6c102f3ac12ea41770e90c195e28e1927667b 100644 (file)
@@ -1222,8 +1222,6 @@ static int ethoc_probe(struct platform_device *pdev)
                goto error;
        }
 
-       ether_setup(netdev);
-
        /* setup the net_device structure */
        netdev->netdev_ops = &ethoc_netdev_ops;
        netdev->watchdog_timeo = ETHOC_TIMEOUT;
index fd4b6aecf6ee85d8f135cd6491582b417e1f3f1a..2dad4d5047bae724510dbd94d72d0fe4dbb006d8 100644 (file)
@@ -633,7 +633,6 @@ ltq_etop_init(struct net_device *dev)
        int err;
        bool random_mac = false;
 
-       ether_setup(dev);
        dev->watchdog_timeo = 10 * HZ;
        err = ltq_etop_hw_init(dev);
        if (err)
index 2f12c88c66abfa1cd0bb0e099e4a33a761327fd6..bde1b70f473b31efd8c66dfeaccb28a77f1ab239 100644 (file)
@@ -511,7 +511,6 @@ static int moxart_mac_probe(struct platform_device *pdev)
                goto init_fail;
        }
 
-       ether_setup(ndev);
        ndev->netdev_ops = &moxart_netdev_ops;
        netif_napi_add(ndev, &priv->napi, moxart_rx_poll, RX_DESC_NUM);
        ndev->priv_flags |= IFF_UNICAST_FLT;
index 31eb911e4763c691bb288912a8e48900cf986552..8176c8a1cc6a93b146724b62377b2f059c608306 100644 (file)
@@ -315,8 +315,6 @@ static int netx_eth_enable(struct net_device *ndev)
        unsigned int mac4321, mac65;
        int running, i;
 
-       ether_setup(ndev);
-
        ndev->netdev_ops = &netx_eth_netdev_ops;
        ndev->watchdog_timeo = msecs_to_jiffies(5000);
 
index 79645f74b3a84400865e6f148dc31ad3cbf3e887..379b7fbded784df52068b92f836c921bf4661559 100644 (file)
@@ -943,7 +943,6 @@ static int w90p910_ether_setup(struct net_device *dev)
 {
        struct w90p910_ether *ether = netdev_priv(dev);
 
-       ether_setup(dev);
        dev->netdev_ops = &w90p910_ether_netdev_ops;
        dev->ethtool_ops = &w90p910_ether_ethtool_ops;
 
index a44a03c45014903a6410ebe2d665f3651ee30db2..66fd868152e579a4ba3483fd4726f0e9d9662436 100644 (file)
@@ -1377,9 +1377,6 @@ static int lpc_eth_drv_probe(struct platform_device *pdev)
                goto err_out_iounmap;
        }
 
-       /* Fill in the fields of the device structure with ethernet values. */
-       ether_setup(ndev);
-
        /* Setup driver functions */
        ndev->netdev_ops = &lpc_netdev_ops;
        ndev->ethtool_ops = &lpc_eth_ethtool_ops;
index 74eb520e264956f3330db850bcc12f1d18eabf76..2c811f66d5acc47da86407222c4eea21d14954d0 100644 (file)
@@ -810,8 +810,6 @@ qcaspi_netdev_setup(struct net_device *dev)
 {
        struct qcaspi *qca = NULL;
 
-       ether_setup(dev);
-
        dev->netdev_ops = &qcaspi_netdev_ops;
        qcaspi_set_ethtool_ops(dev);
        dev->watchdog_timeo = QCASPI_TX_TIMEOUT;
index 9778cba9fc74906b68a26625f06f1cf4a88648c8..e88df9c7f1c071120b09e6f8ee896c261365bc6f 100644 (file)
@@ -1927,9 +1927,6 @@ static int smc911x_probe(struct net_device *dev)
        }
        dev->irq = irq_canonicalize(dev->irq);
 
-       /* Fill in the fields of the device structure with ethernet values. */
-       ether_setup(dev);
-
        dev->netdev_ops = &smc911x_netdev_ops;
        dev->watchdog_timeo = msecs_to_jiffies(watchdog);
        dev->ethtool_ops = &smc911x_ethtool_ops;
index bcaa41af1e628e9f8d2e84fccc7a38ea716e428c..5e94d00b96b301d9dc9cd09a17943520b96e8b52 100644 (file)
@@ -1967,9 +1967,6 @@ static int smc_probe(struct net_device *dev, void __iomem *ioaddr,
        }
        dev->irq = irq_canonicalize(dev->irq);
 
-       /* Fill in the fields of the device structure with ethernet values. */
-       ether_setup(dev);
-
        dev->watchdog_timeo = msecs_to_jiffies(watchdog);
        dev->netdev_ops = &smc_netdev_ops;
        dev->ethtool_ops = &smc_ethtool_ops;
index 5e13fa5524ae911f5516f060a5acdb729fec7be0..affb29da353e29139fb758c5a5ce68b701479815 100644 (file)
@@ -2255,7 +2255,6 @@ static int smsc911x_init(struct net_device *dev)
        if (smsc911x_soft_reset(pdata))
                return -ENODEV;
 
-       ether_setup(dev);
        dev->flags |= IFF_MULTICAST;
        netif_napi_add(dev, &pdata->napi, smsc911x_poll, SMSC_NAPI_WEIGHT);
        dev->netdev_ops = &smsc911x_netdev_ops;
index 9979f67de3aa32663969a6387ab0b4aa0f980279..6f77a46c7e2c446da3b3b4ae139057a621ef00ef 100644 (file)
@@ -2765,8 +2765,6 @@ struct stmmac_priv *stmmac_dvr_probe(struct device *device,
        priv->device = device;
        priv->dev = ndev;
 
-       ether_setup(ndev);
-
        stmmac_set_ethtool_ops(ndev);
        priv->pause = pause;
        priv->plat = plat_dat;
index 104d46f37969f990c920d4ee42c61e7258579f8a..0f56b1c0e08203ea4fdc6dd29b54480bdd87a41b 100644 (file)
@@ -708,7 +708,6 @@ static int w5100_probe(struct platform_device *pdev)
        priv = netdev_priv(ndev);
        priv->ndev = ndev;
 
-       ether_setup(ndev);
        ndev->netdev_ops = &w5100_netdev_ops;
        ndev->ethtool_ops = &w5100_ethtool_ops;
        ndev->watchdog_timeo = HZ;
index 1f33c4c86c207f6a4fecf52bca8645c5e114be9e..f961f14a047343e7b06a3ea6601656e7ce2c139b 100644 (file)
@@ -620,7 +620,6 @@ static int w5300_probe(struct platform_device *pdev)
        priv = netdev_priv(ndev);
        priv->ndev = ndev;
 
-       ether_setup(ndev);
        ndev->netdev_ops = &w5300_netdev_ops;
        ndev->ethtool_ops = &w5300_ethtool_ops;
        ndev->watchdog_timeo = HZ;