]> git.itanic.dy.fi Git - linux-stable/commit
net: bcmgenet: Restore phy_stop() depending upon suspend/close
authorFlorian Fainelli <f.fainelli@gmail.com>
Mon, 15 May 2023 02:56:07 +0000 (19:56 -0700)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 16 May 2023 07:34:38 +0000 (09:34 +0200)
commit225c657945c4a6307741cb3cc89467eadcc26e9b
treee5b55850c1acff733353be04907b4d96d746b087
parentdf0acdc59b094cdaef19b1c8d83c9721082bab7b
net: bcmgenet: Restore phy_stop() depending upon suspend/close

Removing the phy_stop() from bcmgenet_netif_stop() ended up causing
warnings from the PHY library that phy_start() is called from the
RUNNING state since we are no longer stopping the PHY state machine
during bcmgenet_suspend().

Restore the call to phy_stop() but make it conditional on being called
from the close or suspend path.

Fixes: c96e731c93ff ("net: bcmgenet: connect and disconnect from the PHY state machine")
Fixes: 93e0401e0fc0 ("net: bcmgenet: Remove phy_stop() from bcmgenet_netif_stop()")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Reviewed-by: Pavan Chebbi <pavan.chebbi@broadcom.com>
Link: https://lore.kernel.org/r/20230515025608.2587012-1-f.fainelli@gmail.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/broadcom/genet/bcmgenet.c