]> git.baikalelectronics.ru Git - kernel.git/commit
net: phy: micrel: use kszphy_suspend()/kszphy_resume for irq aware devices
authorClaudiu Beznea <claudiu.beznea@microchip.com>
Tue, 18 Jan 2022 11:08:12 +0000 (13:08 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Jan 2022 14:23:34 +0000 (14:23 +0000)
commit39560238d8ed3646ffde16d33e3890506db79e1f
tree99b493b5739289683de465ed06a41de47f30436b
parente978a9a0decf1be6c02d99df7ab8b26d78d4659c
net: phy: micrel: use kszphy_suspend()/kszphy_resume for irq aware devices

On a setup with KSZ9131 and MACB drivers it happens on suspend path, from
time to time, that the PHY interrupt arrives after PHY and MACB were
suspended (PHY via genphy_suspend(), MACB via macb_suspend()). In this
case the phy_read() at the beginning of kszphy_handle_interrupt() will
fail (as MACB driver is suspended at this time) leading to phy_error()
being called and a stack trace being displayed on console. To solve this
.suspend/.resume functions for all KSZ devices implementing
.handle_interrupt were replaced with kszphy_suspend()/kszphy_resume()
which disable/enable interrupt before/after calling
genphy_suspend()/genphy_resume().

The fix has been adapted for all KSZ devices which implements
.handle_interrupt but it has been tested only on KSZ9131.

Fixes: 1338b1e326ce ("net: phy: micrel: implement generic .handle_interrupt() callback")
Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/micrel.c