Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers: mdio_nxp_enet: Fix busy wait #75625

Merged

Conversation

decsny
Copy link
Member

@decsny decsny commented Jul 8, 2024

Fix the busy wait in the MDIO driver that was causing timing problems in systems with real time requirements performing tasks more frequently than about a millisecond.

Restructure the code to be less redundant and change the busy wait kconfig to microseconds instead of millliseconds. Also actually signal to the mdio driver that it can use the interrupt instead of busy waiting, this seems to have been forgotten.

Fixes #75624

Fix the busy wait in the MDIO driver that was causing timing
problems in systems with real time requirements performing tasks
more frequently than about a millisecond.

Restructure the code to be less redundant and change the busy wait
kconfig to microseconds instead of millliseconds. Also actually signal
to the mdio driver that it can use the interrupt instead of busy
waiting, this seems to have been forgotten.

Signed-off-by: Declan Snyder <[email protected]>
@decsny decsny added bug The issue is a bug, or the PR is fixing a bug area: Ethernet platform: NXP Drivers NXP Semiconductors, drivers labels Jul 8, 2024
@decsny decsny added this to the v3.7.0 milestone Jul 8, 2024
@decsny
Copy link
Member Author

decsny commented Jul 8, 2024

I will update the release notes and migration guide about the bug fix and kconfig change in #75001

@nashif nashif merged commit 6ac0d8b into zephyrproject-rtos:main Jul 10, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Ethernet area: MDIO bug The issue is a bug, or the PR is fixing a bug platform: NXP Drivers NXP Semiconductors, drivers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

drivers: nxp_enet: MDIO xfer blocks for a long time causing system latency issues
7 participants