]> rtime.felk.cvut.cz Git - zynq/linux.git/commitdiff
net: emaclite: Fix position of lp->mii_bus assignment
authorRadhey Shyam Pandey <radhey.shyam.pandey@xilinx.com>
Tue, 13 Feb 2018 10:36:42 +0000 (03:36 -0700)
committerMichal Simek <michal.simek@xilinx.com>
Thu, 1 Mar 2018 08:20:16 +0000 (09:20 +0100)
To ensure MDIO bus is not double freed in remove() path
assign lp->mii_bus after MDIO bus registration.

Signed-off-by: Radhey Shyam Pandey <radheys@xilinx.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
drivers/net/ethernet/xilinx/xilinx_emaclite.c

index c5be744dfa344dcc4ded867c12cf321e3002e451..b9ae4f1cd753cbaafbddc13e5104c2cb91d260e1 100644 (file)
@@ -868,14 +868,14 @@ static int xemaclite_mdio_setup(struct net_local *lp, struct device *dev)
        bus->write = xemaclite_mdio_write;
        bus->parent = dev;
 
-       lp->mii_bus = bus;
-
        rc = of_mdiobus_register(bus, np);
        if (rc) {
                dev_err(dev, "Failed to register mdio bus.\n");
                goto err_register;
        }
 
+       lp->mii_bus = bus;
+
        return 0;
 
 err_register: