> @@ -1604,7 +1599,7 @@ static int smsc911x_open(struct net_device *dev) > netdev_warn(dev, "ISR failed signaling test (IRQ %d)\n", > dev->irq); > retval = -ENODEV; > - goto out; > + goto mii_free_out;
A nit pick, but it is not really freeing anything. Maybe you can think of a better name? Otherwise Reviewed-by: Andrew Lunn <[email protected]> Andrew
