Skip to content

Commit

Permalink
ethernet: phy_mc_ksz8081: Don't reset in cfg link
Browse files Browse the repository at this point in the history
No need to reset in cfg link, this was blocking system workqueue during
phy callbacks that call cfg link, since this happens from monitor work
handler which is in the system workqueue.

Signed-off-by: Declan Snyder <declan.snyder@nxp.com>
(cherry picked from commit e904743)
  • Loading branch information
decsny authored and nashif committed Oct 15, 2024
1 parent 9739759 commit 46a1300
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions drivers/ethernet/phy/phy_microchip_ksz8081.c
Original file line number Diff line number Diff line change
Expand Up @@ -329,12 +329,6 @@ static int phy_mc_ksz8081_cfg_link(const struct device *dev,
/* We are going to reconfigure the phy, don't need to monitor until done */
k_work_cancel_delayable(&data->phy_monitor_work);

/* Reset PHY */
ret = phy_mc_ksz8081_reset(dev);
if (ret) {
goto done;
}

/* DT configurations */
ret = phy_mc_ksz8081_static_cfg(dev);
if (ret) {
Expand Down

0 comments on commit 46a1300

Please sign in to comment.