> If I look at the patch I think it should call kszphy_config_init() not > _reset() > in the resume function: > > > @@ -715,8 +723,14 @@ static int kszphy_suspend(struct phy_device *phydev) > > static int kszphy_resume(struct phy_device *phydev) > { > + int ret; > + > genphy_resume(phydev); > > - ret = kszphy_config_reset(phydev); > + ret = kszphy_config_init(phydev); > + if (ret) > + return ret; > + >
Hi Chris I think there has been a patch for this posted. If i remember correctly, the PHY you have does not call probe, hence phydev->priv is a NULL pointer, so priv->rmii_ref_clk_sel does not work. It would be good to find the patch and make sure it has been accepted, and marked for stable. Andrew