Skip to content
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -705,6 +705,7 @@ scripts/gen_image_info.py @tejlmand
/subsys/debug/asan_hacks.c @vanwinkeljan @aescolar @daor-oti
/subsys/demand_paging/ @dcpleung @nashif
/subsys/emul/ @sjg20
/subsys/event_manager/ @nordic-krch
/subsys/fb/ @jfischer-no
/subsys/fs/ @nashif
/subsys/fs/fcb/ @nvlsianpu
Expand Down
1 change: 1 addition & 0 deletions boards/riscv/tlsr9518adk80d/doc/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ The following example projects are supported:
- samples/hello_world
- samples/synchronization
- samples/philosophers
- samples/event_manager
Copy link
Contributor

Choose a reason for hiding this comment

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

sample/application_development/event_manager

Copy link
Member

Choose a reason for hiding this comment

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

I am not sure why are we adding this for this board specifically? (and why the board is listing all the samples it supports in the first place)

- samples/basic/threads
- samples/basic/blinky
- samples/basic/blinky_pwm
Expand Down
1 change: 1 addition & 0 deletions doc/reference/kernel/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -126,3 +126,4 @@ These pages cover other kernel services.
other/version.rst
other/fatal.rst
other/thread_local_storage.rst
other/event_manager.rst
Copy link
Member

Choose a reason for hiding this comment

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

Why are you adding this under kernel/other? This should be alongside other subsystems maybe? The other/ under kernel is historical and needs some cleanup, so lets not add more things here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your review. Could you point me where should this documentation in zephyr be placed?

Copy link
Member

Choose a reason for hiding this comment

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

Please place it in doc/reference/event_manager

205 changes: 205 additions & 0 deletions doc/reference/kernel/other/em_overview.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading