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

[Feature Request] Add interrupt priority checking to CM33 ports #526

Closed
johnrhen opened this issue Jul 20, 2022 · 3 comments
Closed

[Feature Request] Add interrupt priority checking to CM33 ports #526

johnrhen opened this issue Jul 20, 2022 · 3 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@johnrhen
Copy link

Is your feature request related to a problem? Please describe.
Interrupt priority checking is missing on CM33 ports.

Describe the solution you'd like
Update the CM33 ports to check interrupt priorities, in line with the other Cortex-M ports. The other Cortex-M ports check interrupt priorities using configMAX_SYSCALL_INTERRUPT_PRIORITY and use the function vPortValidateInterruptPriority() inside of theportASSERT_IF_INTERRUPT_PRIORITY_INVALID() macro.

Additional context
https://forums.freertos.org/t/cm33-missing-vportvalidateinterruptpriority/14836

If you have the same (or similar) feature request, please upvote this issue with thumbs up 👍
and use the comments section to provide answers to the questions above.

@johnrhen johnrhen added enhancement New feature or request help wanted Extra attention is needed labels Jul 20, 2022
@nilesh-dryad
Copy link

Is there any update on this enhancement?

@urutva
Copy link
Contributor

urutva commented Apr 20, 2023

@johnrhen Thanks for raising this request and @nilesh-dryad Thanks for the follow-up. I'll start looking into this and provide an update soon.

@urutva
Copy link
Contributor

urutva commented May 15, 2023

The PR #673 merged to main. Closing the issue.

@urutva urutva closed this as completed May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants