Commit 5db5ea99 authored by Florian Fainelli's avatar Florian Fainelli Committed by David S. Miller

net: phy: Add helpers to determine if PHY driver is generic

We are already checking in phy_detach() that the PHY driver is of
generic kind (1G or 10G) and we are going to make use of that in the SFP
layer as well for 1000BaseT SFP modules, so expose helper functions to
return that information.
Signed-off-by: default avatarFlorian Fainelli <f.fainelli@gmail.com>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6f24e159
...@@ -1277,6 +1277,36 @@ struct phy_device *phy_attach(struct net_device *dev, const char *bus_id, ...@@ -1277,6 +1277,36 @@ struct phy_device *phy_attach(struct net_device *dev, const char *bus_id,
} }
EXPORT_SYMBOL(phy_attach); EXPORT_SYMBOL(phy_attach);
static bool phy_driver_is_genphy_kind(struct phy_device *phydev,
struct device_driver *driver)
{
struct device *d = &phydev->mdio.dev;
bool ret = false;
if (!phydev->drv)
return ret;
get_device(d);
ret = d->driver == driver;
put_device(d);
return ret;
}
bool phy_driver_is_genphy(struct phy_device *phydev)
{
return phy_driver_is_genphy_kind(phydev,
&genphy_driver.mdiodrv.driver);
}
EXPORT_SYMBOL_GPL(phy_driver_is_genphy);
bool phy_driver_is_genphy_10g(struct phy_device *phydev)
{
return phy_driver_is_genphy_kind(phydev,
&genphy_10g_driver.mdiodrv.driver);
}
EXPORT_SYMBOL_GPL(phy_driver_is_genphy_10g);
/** /**
* phy_detach - detach a PHY device from its network device * phy_detach - detach a PHY device from its network device
* @phydev: target phy_device struct * @phydev: target phy_device struct
...@@ -1308,8 +1338,8 @@ void phy_detach(struct phy_device *phydev) ...@@ -1308,8 +1338,8 @@ void phy_detach(struct phy_device *phydev)
* from the generic driver so that there's a chance a * from the generic driver so that there's a chance a
* real driver could be loaded * real driver could be loaded
*/ */
if (phydev->mdio.dev.driver == &genphy_10g_driver.mdiodrv.driver || if (phy_driver_is_genphy(phydev) ||
phydev->mdio.dev.driver == &genphy_driver.mdiodrv.driver) phy_driver_is_genphy_10g(phydev))
device_release_driver(&phydev->mdio.dev); device_release_driver(&phydev->mdio.dev);
/* /*
......
...@@ -1183,4 +1183,7 @@ module_exit(phy_module_exit) ...@@ -1183,4 +1183,7 @@ module_exit(phy_module_exit)
#define module_phy_driver(__phy_drivers) \ #define module_phy_driver(__phy_drivers) \
phy_module_driver(__phy_drivers, ARRAY_SIZE(__phy_drivers)) phy_module_driver(__phy_drivers, ARRAY_SIZE(__phy_drivers))
bool phy_driver_is_genphy(struct phy_device *phydev);
bool phy_driver_is_genphy_10g(struct phy_device *phydev);
#endif /* __PHY_H */ #endif /* __PHY_H */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment