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

eth: mcux: Do not set carrier ON if interface is not known #28381

Merged

Conversation

Zhaoningx
Copy link
Collaborator

@Zhaoningx Zhaoningx commented Sep 15, 2020

It is possible that the network interface is not yet initialized
when status of the PHY changes. In this case we must not call
net_eth_carrier_on() as that will cause a crash.
This was noticed with mimxrt1050_evk board.
Fix github issue #23210

Signed-off-by: Jukka Rissanen jukka.rissanen@linux.intel.com

It is possible that the network interface is not yet initialized
when status of the PHY changes. In this case we must not call
net_eth_carrier_on() as that will cause a crash.
This was noticed with mimxrt1050_evk board.

Fixes: zephyrproject-rtos#21257

Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
@Zhaoningx
Copy link
Collaborator Author

Zhaoningx commented Sep 16, 2020

Hi all, this PR backported the PR of github mainline:#21294

@Zhaoningx Zhaoningx requested a review from nashif September 16, 2020 05:28
@jocelyn-li jocelyn-li requested a review from LeiW000 September 16, 2020 06:39
@nashif nashif added this to the v1.14.3 milestone Sep 16, 2020
@nashif nashif merged commit b41757d into zephyrproject-rtos:v1.14-branch Sep 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants