On 04/01/16 09:36, Andrew Lunn wrote: > We will soon support devices other than phys on the mdio bus. Look at > a child's compatibility string to determine if it is a phy, before > registering a phy device. > > Signed-off-by: Andrew Lunn <and...@lunn.ch> > ---
[snip] > + if (of_get_phy_id(child, &phy_id) != -EINVAL) > + return true; > + > + if (of_device_is_compatible(child, "ethernet-phy-ieee802.3-c45")) > + return true; Is there a particular reason not to check for "ethernet-phy-ieee802.3-c22 here? > + > + if (!of_find_property(child, "compatible", NULL)) > + return true; That seems reasonable, we should probably make the compatible string for PHY Device Tree nodes not an optional property anymore though, but that's for a separate change and topic. -- Florian -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html