In the function ipq8064_mdio_probe(), of_mdiobus_register() might
returned zero, so the direct return can simplify code. Thus remove
redundant parameter and check.

Signed-off-by: Zhang Shengju <zhangshen...@cmss.chinamobile.com>
Signed-off-by: Tang Bin <tang...@cmss.chinamobile.com>
---
 drivers/net/mdio/mdio-ipq8064.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/net/mdio/mdio-ipq8064.c b/drivers/net/mdio/mdio-ipq8064.c
index 1bd1885..33cccce 100644
--- a/drivers/net/mdio/mdio-ipq8064.c
+++ b/drivers/net/mdio/mdio-ipq8064.c
@@ -102,7 +102,6 @@ ipq8064_mdio_probe(struct platform_device *pdev)
        struct device_node *np = pdev->dev.of_node;
        struct ipq8064_mdio *priv;
        struct mii_bus *bus;
-       int ret;
 
        bus = devm_mdiobus_alloc_size(&pdev->dev, sizeof(*priv));
        if (!bus)
@@ -125,12 +124,9 @@ ipq8064_mdio_probe(struct platform_device *pdev)
                return PTR_ERR(priv->base);
        }
 
-       ret = of_mdiobus_register(bus, np);
-       if (ret)
-               return ret;
-
        platform_set_drvdata(pdev, bus);
-       return 0;
+
+       return of_mdiobus_register(bus, np);
 }
 
 static int
-- 
2.20.1.windows.1



Reply via email to