sys/shell: add shell wrapper for openthread cli #21862
Open
+85
−10
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
Area: network
morigawa
changed the title
Until now OpenThread CLI and RIOT shell could not be used at the same time. Now it is possible to use all CLI command with the prefix "ot". To get an overview use "ot help".
Automatically use RIOT shell when compiling OpenThread with CLI module. Ordered Makefile.dep alpha numerically.
Add shell to example, update readme and add adafruit-feather-sense to whitelisted boards, after testing.
morigawa
force-pushed
the
pr/integrate-openthread-into-shell
branch
from
eaa1033 to
c04fbd3
Compare
Contributor
Author
|
Fixed some errors that were created while doing the cleanup and tested for different boards now. PR is now ready for review. I changed the CLI transport from UART to Console and it works, but I'm not sure if this is correct. At least for the feather it is now possible to easily use the example, because previously the output was only available using a UART to USB adapter. |
crasbe
added
Type: new feature
mguetschow
reviewed
Nov 10, 2025
Contributor
mguetschow
left a comment
mguetschow
left a comment
There was a problem hiding this comment.
Nice one, thank you! Haven't tested it yet, but I do have some comments already :)
morigawa
force-pushed
the
pr/integrate-openthread-into-shell
branch
from
bf9f69d to
07a0125
Compare
mguetschow
reviewed
Nov 14, 2025
Contributor
|
Tested to work nicely on an |
morigawa
force-pushed
the
pr/integrate-openthread-into-shell
branch
from
8abe2f0 to
61e1f85
Compare
mguetschow
approved these changes
Nov 14, 2025
Contributor
mguetschow
left a comment
mguetschow
left a comment
There was a problem hiding this comment.
Thanks for bearing with me :) Looks good, please squash!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contribution description
Adds a wrapper, so that the OpenThread CLI can be used through the RIOT shell. Without this, it was not possible to use the OpenThread CLI and RIOT shell capabilities simultaneously, as far as I am aware. To solve this, I created a new shell pseudo-module that adds the "ot" command, to provide access to the OpenThread CLI. If the openthread-cli-% module is used, the shell module will be compiled as a dependency. The OpenThread example previously made use of the OpenThread CLI, therefor the example has been adapted for the new usage.
Testing procedure
Tested shell and basic OpenThread utility with new wrapper for nrf52840dk, adafruit-feather-nrf52840-sense (added this to example whitelist) and frdm-kw41z.