RE: [PATCH] lan78xx: Don't reset the interface on open

From: Nisar.Sayed
Date: Tue Apr 10 2018 - 10:16:30 EST


Thanks Phil, for identifying the issues.

> - ret = lan78xx_reset(dev);
> - if (ret < 0)
> - goto done;
> -
> phy_start(net->phydev);
>
> netif_dbg(dev, ifup, dev->net, "phy initialised successfully");
> --

You may need to start the interrupts before "phy_start" instead of suppressing call to "lan78xx_reset".

+ if (dev->domain_data.phyirq > 0)
+ phy_start_interrupts(dev->net->phydev);

- Nisar