Commit d813d7e5 authored by Linus Walleij's avatar Linus Walleij Committed by Jakub Kicinski

net: ethernet: ixp4xx: Use distinct local variable

Use "ndev" for the struct net_device and "dev" for the
struct device in probe() and remove(). Add the local
"dev" pointer for later use in refactoring.

Take this opportunity to fix inverse christmas tree
coding style.
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent c83db9ef
...@@ -1367,20 +1367,23 @@ static const struct net_device_ops ixp4xx_netdev_ops = { ...@@ -1367,20 +1367,23 @@ static const struct net_device_ops ixp4xx_netdev_ops = {
static int ixp4xx_eth_probe(struct platform_device *pdev) static int ixp4xx_eth_probe(struct platform_device *pdev)
{ {
struct port *port; char phy_id[MII_BUS_ID_SIZE + 3];
struct net_device *dev;
struct eth_plat_info *plat = dev_get_platdata(&pdev->dev);
struct phy_device *phydev = NULL; struct phy_device *phydev = NULL;
struct device *dev = &pdev->dev;
struct eth_plat_info *plat;
struct net_device *ndev;
struct port *port;
u32 regs_phys; u32 regs_phys;
char phy_id[MII_BUS_ID_SIZE + 3];
int err; int err;
if (!(dev = devm_alloc_etherdev(&pdev->dev, sizeof(struct port)))) plat = dev_get_platdata(dev);
if (!(ndev = devm_alloc_etherdev(dev, sizeof(struct port))))
return -ENOMEM; return -ENOMEM;
SET_NETDEV_DEV(dev, &pdev->dev); SET_NETDEV_DEV(ndev, dev);
port = netdev_priv(dev); port = netdev_priv(ndev);
port->netdev = dev; port->netdev = ndev;
port->id = pdev->id; port->id = pdev->id;
switch (port->id) { switch (port->id) {
...@@ -1432,16 +1435,16 @@ static int ixp4xx_eth_probe(struct platform_device *pdev) ...@@ -1432,16 +1435,16 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
return -ENODEV; return -ENODEV;
} }
dev->netdev_ops = &ixp4xx_netdev_ops; ndev->netdev_ops = &ixp4xx_netdev_ops;
dev->ethtool_ops = &ixp4xx_ethtool_ops; ndev->ethtool_ops = &ixp4xx_ethtool_ops;
dev->tx_queue_len = 100; ndev->tx_queue_len = 100;
netif_napi_add(dev, &port->napi, eth_poll, NAPI_WEIGHT); netif_napi_add(ndev, &port->napi, eth_poll, NAPI_WEIGHT);
if (!(port->npe = npe_request(NPE_ID(port->id)))) if (!(port->npe = npe_request(NPE_ID(port->id))))
return -EIO; return -EIO;
port->mem_res = request_mem_region(regs_phys, REGS_SIZE, dev->name); port->mem_res = request_mem_region(regs_phys, REGS_SIZE, ndev->name);
if (!port->mem_res) { if (!port->mem_res) {
err = -EBUSY; err = -EBUSY;
goto err_npe_rel; goto err_npe_rel;
...@@ -1449,9 +1452,9 @@ static int ixp4xx_eth_probe(struct platform_device *pdev) ...@@ -1449,9 +1452,9 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
port->plat = plat; port->plat = plat;
npe_port_tab[NPE_ID(port->id)] = port; npe_port_tab[NPE_ID(port->id)] = port;
memcpy(dev->dev_addr, plat->hwaddr, ETH_ALEN); memcpy(ndev->dev_addr, plat->hwaddr, ETH_ALEN);
platform_set_drvdata(pdev, dev); platform_set_drvdata(pdev, ndev);
__raw_writel(DEFAULT_CORE_CNTRL | CORE_RESET, __raw_writel(DEFAULT_CORE_CNTRL | CORE_RESET,
&port->regs->core_control); &port->regs->core_control);
...@@ -1461,7 +1464,7 @@ static int ixp4xx_eth_probe(struct platform_device *pdev) ...@@ -1461,7 +1464,7 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
snprintf(phy_id, MII_BUS_ID_SIZE + 3, PHY_ID_FMT, snprintf(phy_id, MII_BUS_ID_SIZE + 3, PHY_ID_FMT,
mdio_bus->id, plat->phy); mdio_bus->id, plat->phy);
phydev = phy_connect(dev, phy_id, &ixp4xx_adjust_link, phydev = phy_connect(ndev, phy_id, &ixp4xx_adjust_link,
PHY_INTERFACE_MODE_MII); PHY_INTERFACE_MODE_MII);
if (IS_ERR(phydev)) { if (IS_ERR(phydev)) {
err = PTR_ERR(phydev); err = PTR_ERR(phydev);
...@@ -1470,10 +1473,10 @@ static int ixp4xx_eth_probe(struct platform_device *pdev) ...@@ -1470,10 +1473,10 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
phydev->irq = PHY_POLL; phydev->irq = PHY_POLL;
if ((err = register_netdev(dev))) if ((err = register_netdev(ndev)))
goto err_phy_dis; goto err_phy_dis;
printk(KERN_INFO "%s: MII PHY %i on %s\n", dev->name, plat->phy, printk(KERN_INFO "%s: MII PHY %i on %s\n", ndev->name, plat->phy,
npe_name(port->npe)); npe_name(port->npe));
return 0; return 0;
...@@ -1490,11 +1493,11 @@ static int ixp4xx_eth_probe(struct platform_device *pdev) ...@@ -1490,11 +1493,11 @@ static int ixp4xx_eth_probe(struct platform_device *pdev)
static int ixp4xx_eth_remove(struct platform_device *pdev) static int ixp4xx_eth_remove(struct platform_device *pdev)
{ {
struct net_device *dev = platform_get_drvdata(pdev); struct net_device *ndev = platform_get_drvdata(pdev);
struct phy_device *phydev = dev->phydev; struct phy_device *phydev = ndev->phydev;
struct port *port = netdev_priv(dev); struct port *port = netdev_priv(ndev);
unregister_netdev(dev); unregister_netdev(ndev);
phy_disconnect(phydev); phy_disconnect(phydev);
ixp4xx_mdio_remove(); ixp4xx_mdio_remove();
npe_port_tab[NPE_ID(port->id)] = NULL; npe_port_tab[NPE_ID(port->id)] = NULL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment