-
Notifications
You must be signed in to change notification settings - Fork 2k
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
pm: don't (un)block IDLE mode #17974
Conversation
This is already unblocked and will trigger an assertion. The code is still broken as other modes might be unblocked too, but at least it is just as broken as it was before RIOT-OS#17895
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proxy ACK from @kaspar030 (via Matrix)
let's see if this fixes the LoRaWAN issue |
/* sleeping is currently not supported, block all sleep modes */ | ||
for (uint8_t i = 0; i < PM_NUM_MODES; i++) { | ||
/* sleeping is currently not supported, block all sleep modes except IDLE */ | ||
for (uint8_t i = 0; i < PM_NUM_MODES - 1; i++) { | ||
pm_block(i); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tbh I don't see what's the point of this code - sleep modes are blocked by default on RIOT anyway.
Contribution description
This is already unblocked and will trigger an assertion.
The code is still broken as other modes might be unblocked too, but at least it is just as broken as it was before #17895
Testing procedure
examples/lorawan
should no longer crash on most boards.(should still crash on samd21 boards as it would since ed9f740)
Issues/PRs references
follow-up to #17895