Commit 07caad0b authored by Lukas Wunner's avatar Lukas Wunner Committed by Jakub Kicinski

net: phy: Deduplicate interrupt disablement on PHY attach

phy_attach_direct() first calls phy_init_hw() (which restores interrupt
settings through ->config_intr()), then calls phy_disable_interrupts().

So if phydev->interrupts was previously set to 1, interrupts are briefly
enabled, then disabled, which seems nonsensical.

If it was previously set to 0, interrupts are disabled twice, which is
equally nonsensical.

Deduplicate interrupt disablement.
Signed-off-by: default avatarLukas Wunner <lukas@wunner.de>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/805ccdc606bd8898d59931bd4c7c68537ed6e550.1651040826.git.lukas@wunner.deSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 6fd1d51c
...@@ -1449,6 +1449,8 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev, ...@@ -1449,6 +1449,8 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev,
phydev->state = PHY_READY; phydev->state = PHY_READY;
phydev->interrupts = PHY_INTERRUPT_DISABLED;
/* 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
...@@ -1471,10 +1473,6 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev, ...@@ -1471,10 +1473,6 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev,
if (err) if (err)
goto error; goto error;
err = phy_disable_interrupts(phydev);
if (err)
return err;
phy_resume(phydev); phy_resume(phydev);
phy_led_triggers_register(phydev); phy_led_triggers_register(phydev);
......
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