-
Notifications
You must be signed in to change notification settings - Fork 1.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
[bp-15102} update documentation #15133
Conversation
since the basepri is always used without any configuraion Signed-off-by: Xiang Xiao <[email protected]>
This is a follow-up to 366c8a5 (PR-15102). * Documentation/guides/zerolatencyinterrupts.rst (Title): Make title case consistent. (Getting Back into the Game, Maskable Nested Interrupts): Clarify discussion about priorities. (Cortex-M3/4 Implementation): The first half of a sentence was deleted in 366c8a5 because the Kconfig that was described there no longer exists: CONFIG_ARMV7M_USEBASEPRI. Write a new beginning for this sentence that matches current implementation. (Disabling the High Priority Interrupt): Change "cannot" to "must not be allowed to" to improve clarity. (Configuring High Priority Interrupts): Change "to NVIC" to "in NVIC" to improve clarity.
Signed-off-by: buxiasen <[email protected]>
[Experimental Bot, please feedback here] No, this PR summary does not adequately meet the NuttX requirements. While it references other PRs, it fails to provide the necessary context and details required by the template. Here's why it's insufficient and how it could be improved:
How to improve this PR summary:
By addressing these points, the PR summary will provide the necessary information for reviewers to understand the changes and assess their impact effectively, ultimately leading to a smoother and more efficient review process. |
Summary
this patch includes
#15102
#15121
#15122
Impact
RELEASE
Testing
CI