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

ipq40xx: dont panic on PSGMII calibration fail #15542

Merged
merged 1 commit into from
May 22, 2024

Conversation

robimarko
Copy link
Contributor

Currently, in case that PSGMII calibration fails it will panic the kernel which is not ideal and is preventing any debugging to be done.

So, since PGMII calibration failing only means that wired networking wont work lets convet the panic() call to dev_error.

@github-actions github-actions bot added the target/ipq40xx pull request/issue for ipq40xx target label May 22, 2024
Currently, in case that PSGMII calibration fails it will panic the kernel
which is not ideal and is preventing any debugging to be done.

So, since PGMII calibration failing only means that wired networking wont
work lets convet the panic() call to dev_error.

Link: openwrt#15542
Signed-off-by: Robert Marko <robimarko@gmail.com>
@openwrt-bot openwrt-bot merged commit 4a78c46 into openwrt:main May 22, 2024
@robimarko
Copy link
Contributor Author

Thanks! Rebased on top of main and merged!

@robimarko robimarko deleted the ipq40xx-psmgii-panic branch May 22, 2024 15:27
Vladdrako pushed a commit to Vladdrako/openwrt that referenced this pull request Jun 8, 2024
Currently, in case that PSGMII calibration fails it will panic the kernel
which is not ideal and is preventing any debugging to be done.

So, since PGMII calibration failing only means that wired networking wont
work lets convet the panic() call to dev_error.

Link: openwrt#15542
Signed-off-by: Robert Marko <robimarko@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target/ipq40xx pull request/issue for ipq40xx target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants