Skip to content

Conversation

eivindj-nordic
Copy link
Contributor

Add documentation for event scheduler.

@eivindj-nordic eivindj-nordic self-assigned this Oct 9, 2025
@eivindj-nordic eivindj-nordic requested review from a team as code owners October 9, 2025 13:52
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Oct 9, 2025
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

You can find the documentation preview for this PR here.

@eivindj-nordic eivindj-nordic force-pushed the doc_event_scheduler branch 2 times, most recently from e6fbf05 to e4033cc Compare October 14, 2025 08:58
Comment on lines 46 to 47
| Header file: :file:`include/event_scheduler`
| Source files: :file:`lib/event_scheduler.h/`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| Header file: :file:`include/event_scheduler`
| Source files: :file:`lib/event_scheduler.h/`
| Header file: :file:`include/event_scheduler.h`
| Source files: :file:`lib/event_scheduler/`

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks :/

@eivindj-nordic eivindj-nordic requested a review from a team as a code owner October 16, 2025 07:00

In some applications it is beneficial to defer the execution of certain interrupts, for example some SoftDevice interrupts, to the main application function.
This shortens the time spent in the interrupt service routine (ISR) and allows for other (low priority) events to be raised before the previous event is fully processed.
Note that the application must take care to only defer events that can be interleaved by the processing of other interrupts.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this line makes much sense.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried rewriting this in my commit, @eivindj-nordic pls check if it makes sense

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Squashed, thanks!

@b-gent b-gent force-pushed the doc_event_scheduler branch from 079f832 to d5e7722 Compare October 17, 2025 12:26
@eivindj-nordic eivindj-nordic force-pushed the doc_event_scheduler branch 2 times, most recently from 6728043 to ddd523f Compare October 21, 2025 12:02
Add documentation for event scheduler.

Co-authored-by: Bartosz Gentkowski <[email protected]>
Signed-off-by: Eivind Jølsgard <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants