OSDN Git Service

net: phy: remove useless check in state machine case PHY_RESUMING
authorHeiner Kallweit <hkallweit1@gmail.com>
Wed, 7 Nov 2018 19:44:56 +0000 (20:44 +0100)
committerDavid S. Miller <davem@davemloft.net>
Thu, 8 Nov 2018 23:02:05 +0000 (15:02 -0800)
commitc96469f830568d8859cd65d723da3b79ec0219e0
tree2f617e428823a279b8d1539064222ae9a47f4efe
parent3b01ea72f34cf8e18d16d993bb20f3cd241e4e52
net: phy: remove useless check in state machine case PHY_RESUMING

If aneg isn't finished yet then the PHY reports the link as down.
There's no benefit in setting the state to PHY_AN because the next
state machine run would set the status to PHY_NOLINK anyway (except
in the meantime aneg has been finished and link is up). Therefore
we can set the state to PHY_RUNNING or PHY_NOLINK directly.

In addition change the do_carrier parameter in phy_link_down() to true.
If carrier was marked as up before (what should never be the case because
PHY was in state PHY_HALTED before) then we should mark it as down now.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/phy.c