Skip to content

Conversation

bjarki-andreasen
Copy link

Introduce shell support based on zephyr shell subsystem with a BM specific backend based on NRFX UARTE. This provides at least feature parity with nRF5 (https://docs.nordicsemi.com/bundle/sdk_nrf5_v17.1.0/page/lib_cli.html) since the zephyr implementation is waay nicer IMO. It also naturally provides a nice upgrade path to NCS given its the same shell subsystem :)

Size of shell sample (shell + console):

Memory region         Used Size  Region Size  %age Used
           FLASH:       21084 B      1524 KB      1.35%
             RAM:        4712 B       238 KB      1.93%
        IDT_LIST:          0 GB        32 KB      0.00%

compared with uarte peripheral sample (console only):

Memory region         Used Size  Region Size  %age Used
           FLASH:       15564 B      1524 KB      1.00%
             RAM:        4032 B       238 KB      1.65%
        IDT_LIST:          0 GB        32 KB      0.00%

We can estimate that shell support adds 6kB FLASH, 700B RAM.

Update sdk-nrf to include shell support

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
@bjarki-andreasen bjarki-andreasen requested review from a team as code owners July 17, 2025 11:14
Copy link
Contributor

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

doc/nrf-bm/samples.rst should be updated to link to this documentation also

Implement shell support using zephyr's shell subsys with slight
modifications to allow it to be used single threaded, and add nrfx
uarte based shell backend for the shell to use.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
@bjarki-andreasen
Copy link
Author

made the shell backend buffer defaults a bit smaller as well, they where a bit oversized

Add section for shell samples, link shell samples in there, and
add link to section.

Signed-off-by: Bjarki Arge Andreasen <[email protected]>
Fixing some nits in the sample doc.

Signed-off-by: Bartosz Gentkowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants