Commit 03536cc3 authored by Jisheng Zhang's avatar Jisheng Zhang Committed by David S. Miller

net: mdio-sun4i: Convert to devm_* api

Use devm_ioremap_resource instead of of_iomap() and devm_kzalloc()
instead of kmalloc() to make cleanup paths simpler. This patch also
fixes the resource leak caused by missing corresponding iounamp()
of the of_iomap().
Signed-off-by: default avatarJisheng Zhang <jszhang@marvell.com>
Acked-by: default avatarMaxime Ripard <maxime.ripard@free-electrons.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 4c9d546f
...@@ -101,6 +101,7 @@ static int sun4i_mdio_probe(struct platform_device *pdev) ...@@ -101,6 +101,7 @@ static int sun4i_mdio_probe(struct platform_device *pdev)
struct device_node *np = pdev->dev.of_node; struct device_node *np = pdev->dev.of_node;
struct mii_bus *bus; struct mii_bus *bus;
struct sun4i_mdio_data *data; struct sun4i_mdio_data *data;
struct resource *res;
int ret, i; int ret, i;
bus = mdiobus_alloc_size(sizeof(*data)); bus = mdiobus_alloc_size(sizeof(*data));
...@@ -114,7 +115,8 @@ static int sun4i_mdio_probe(struct platform_device *pdev) ...@@ -114,7 +115,8 @@ static int sun4i_mdio_probe(struct platform_device *pdev)
snprintf(bus->id, MII_BUS_ID_SIZE, "%s-mii", dev_name(&pdev->dev)); snprintf(bus->id, MII_BUS_ID_SIZE, "%s-mii", dev_name(&pdev->dev));
bus->parent = &pdev->dev; bus->parent = &pdev->dev;
bus->irq = kmalloc(sizeof(int) * PHY_MAX_ADDR, GFP_KERNEL); bus->irq = devm_kzalloc(&pdev->dev, sizeof(int) * PHY_MAX_ADDR,
GFP_KERNEL);
if (!bus->irq) { if (!bus->irq) {
ret = -ENOMEM; ret = -ENOMEM;
goto err_out_free_mdiobus; goto err_out_free_mdiobus;
...@@ -124,10 +126,11 @@ static int sun4i_mdio_probe(struct platform_device *pdev) ...@@ -124,10 +126,11 @@ static int sun4i_mdio_probe(struct platform_device *pdev)
bus->irq[i] = PHY_POLL; bus->irq[i] = PHY_POLL;
data = bus->priv; data = bus->priv;
data->membase = of_iomap(np, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
if (!data->membase) { data->membase = devm_ioremap_resource(&pdev->dev, res);
ret = -ENOMEM; if (IS_ERR(data->membase)) {
goto err_out_free_mdio_irq; ret = PTR_ERR(data->membase);
goto err_out_free_mdiobus;
} }
data->regulator = devm_regulator_get(&pdev->dev, "phy"); data->regulator = devm_regulator_get(&pdev->dev, "phy");
...@@ -139,7 +142,7 @@ static int sun4i_mdio_probe(struct platform_device *pdev) ...@@ -139,7 +142,7 @@ static int sun4i_mdio_probe(struct platform_device *pdev)
} else { } else {
ret = regulator_enable(data->regulator); ret = regulator_enable(data->regulator);
if (ret) if (ret)
goto err_out_free_mdio_irq; goto err_out_free_mdiobus;
} }
ret = of_mdiobus_register(bus, np); ret = of_mdiobus_register(bus, np);
...@@ -152,8 +155,6 @@ static int sun4i_mdio_probe(struct platform_device *pdev) ...@@ -152,8 +155,6 @@ static int sun4i_mdio_probe(struct platform_device *pdev)
err_out_disable_regulator: err_out_disable_regulator:
regulator_disable(data->regulator); regulator_disable(data->regulator);
err_out_free_mdio_irq:
kfree(bus->irq);
err_out_free_mdiobus: err_out_free_mdiobus:
mdiobus_free(bus); mdiobus_free(bus);
return ret; return ret;
...@@ -164,7 +165,6 @@ static int sun4i_mdio_remove(struct platform_device *pdev) ...@@ -164,7 +165,6 @@ static int sun4i_mdio_remove(struct platform_device *pdev)
struct mii_bus *bus = platform_get_drvdata(pdev); struct mii_bus *bus = platform_get_drvdata(pdev);
mdiobus_unregister(bus); mdiobus_unregister(bus);
kfree(bus->irq);
mdiobus_free(bus); mdiobus_free(bus);
return 0; return 0;
......
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