On Mon, Apr 27, 2020 at 06:54:06PM +0530, Calvin Johnson wrote: > Following functions are defined: > phylink_fwnode_phy_connect() > phylink_device_phy_connect() > fwnode_phy_find_device() > device_phy_find_device() > fwnode_get_phy_node() > > First two help in connecting phy to phylink instance. > Next two help in finding a phy on a mdiobus. > Last one helps in getting phy_node from a fwnode. > > Changes in v2: > move phy code from base/property.c to net/phy/phy_device.c > replace acpi & of code to get phy-handle with fwnode_find_reference > replace of_ and acpi_ code with generic fwnode to get phy-handle. > > Calvin Johnson (3): > device property: Introduce phy related fwnode functions > net: phy: alphabetically sort header includes > phylink: Introduce phylink_fwnode_phy_connect() Thanks for this, but there's more work that needs to be done here. I also think that we must have an ack from ACPI people before this can be accepted - you are in effect proposing a new way for representing PHYs in ACPI. > > drivers/net/phy/phy_device.c | 83 ++++++++++++++++++++++++++++++------ > drivers/net/phy/phylink.c | 68 +++++++++++++++++++++++++++++ > include/linux/phy.h | 3 ++ > include/linux/phylink.h | 6 +++ > 4 files changed, 146 insertions(+), 14 deletions(-) > > -- > 2.17.1 > > -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up