Skip to content

enable aesio in Zephyr port & native_sim build container#10932

Open
FoamyGuy wants to merge 4 commits intoadafruit:mainfrom
FoamyGuy:zephyr_aesio_
Open

enable aesio in Zephyr port & native_sim build container#10932
FoamyGuy wants to merge 4 commits intoadafruit:mainfrom
FoamyGuy:zephyr_aesio_

Conversation

@FoamyGuy
Copy link
Copy Markdown
Collaborator

PR includes the following changes:

  • Ran make all zephyr boards to get their autogen files updated for msgpack and other resent changes
  • Added DISABLED_MODULES list to board circuitpython.toml file in zephyr port to turn core modules off on devices with limited flash. Needed this because nrf7002dk overflowed.
  • Added native_sim_build_Containerfile and two scripts that set up a suitable container to build the native_sim inside of. Documented container usage in the zephyr port readme file.
  • Enabled aesio module and added test for it.

@FoamyGuy FoamyGuy requested a review from tannewt April 10, 2026 12:10
Copy link
Copy Markdown
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

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

Looks good! Want to make this an official devcontainer?

@FoamyGuy
Copy link
Copy Markdown
Collaborator Author

I can give it a try and add it as an alternative option. I wasn't familiar with devcontainer but have read up a little. I think personally I'd still opt to use these scripts over devcontainer in order to not have to bring node.js into the mix.

I think it would be nice to have support for it though. It seems to be well supported in VSCode, and it's a standardized structure that people may already be familiar with so they can kind of get a jumpstart on cp-zephyr dev if they already know/use those tools.

I'm more familiar with podman, but I see that devcontainer defaults to docker. For the devcontainer one I will try to implement it with docker so it can stick to devcontainer defaults wherever possible. It looks like it will need to handle the file ownership a little differently.

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.

2 participants