Commit 7d885863 authored by Michael Walle's avatar Michael Walle Committed by Paolo Abeni

net: phy: allow a phy to opt-out of interrupt handling

Until now, it is not possible for a PHY driver to disable interrupts
during runtime. If a driver offers the .config_intr() as well as the
.handle_interrupt() ops, it is eligible for interrupt handling.
Introduce a new flag for the dev_flags property of struct phy_device, which
can be set by PHY driver to skip interrupt setup and fall back to polling
mode.

At the moment, this is used for the MaxLinear PHY which has broken
interrupt handling and there is a need to disable interrupts in some
cases.
Signed-off-by: default avatarMichael Walle <michael@walle.cc>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
parent 90c47eb1
...@@ -1487,6 +1487,13 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev, ...@@ -1487,6 +1487,13 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev,
phydev->interrupts = PHY_INTERRUPT_DISABLED; phydev->interrupts = PHY_INTERRUPT_DISABLED;
/* PHYs can request to use poll mode even though they have an
* associated interrupt line. This could be the case if they
* detect a broken interrupt handling.
*/
if (phydev->dev_flags & PHY_F_NO_IRQ)
phydev->irq = PHY_POLL;
/* Port is set to PORT_TP by default and the actual PHY driver will set /* Port is set to PORT_TP by default and the actual PHY driver will set
* it to different value depending on the PHY configuration. If we have * it to different value depending on the PHY configuration. If we have
* the generic PHY driver we can't figure it out, thus set the old * the generic PHY driver we can't figure it out, thus set the old
......
...@@ -739,6 +739,9 @@ struct phy_device { ...@@ -739,6 +739,9 @@ struct phy_device {
#endif #endif
}; };
/* Generic phy_device::dev_flags */
#define PHY_F_NO_IRQ 0x80000000
static inline struct phy_device *to_phy_device(const struct device *dev) static inline struct phy_device *to_phy_device(const struct device *dev)
{ {
return container_of(to_mdio_device(dev), struct phy_device, mdio); return container_of(to_mdio_device(dev), struct phy_device, mdio);
......
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