> +static int adin_config_init(struct phy_device *phydev) > +{ > + int rc; > + > + rc = genphy_config_init(phydev); > + if (rc < 0) > + return rc; > + > + return 0; > +} Why not just return genphy_config_init(phydev); Andrew
> +static int adin_config_init(struct phy_device *phydev) > +{ > + int rc; > + > + rc = genphy_config_init(phydev); > + if (rc < 0) > + return rc; > + > + return 0; > +} Why not just return genphy_config_init(phydev); Andrew