diff options
| author | Russell King (Oracle) <rmk+kernel@armlinux.org.uk> | 2023-05-22 16:58:08 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2023-05-24 08:28:27 +0100 |
| commit | 59088b5a946ee8a6603a9a84781670cedb01c40d (patch) | |
| tree | 1aa6b3afb0cd87b4b09608d92c6a8aa51a704fb9 /net/unix/af_unix.c | |
| parent | 18731fe01d3de0721c7f6ba326e838a034d3114e (diff) | |
net: phy: avoid kernel warning dump when stopping an errored PHY
When taking a network interface down (or removing a SFP module) after
the PHY has encountered an error, phy_stop() complains incorrectly
that it was called from HALTED state.
The reason this is incorrect is that the network driver will have
called phy_start() when the interface was brought up, and the fact
that the PHY has a problem bears no relationship to the administrative
state of the interface. Taking the interface administratively down
(which calls phy_stop()) is always the right thing to do after a
successful phy_start() call, whether or not the PHY has encountered
an error.
Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions