Skip to content

ports/zephyr: Create options for frozen modules #16323

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 30, 2025

Conversation

VynDragon
Copy link
Contributor

@VynDragon VynDragon commented Nov 29, 2024

Enables the ability to use frozen modules in the zephyr port

Summary

This enables having frozen modules in zephyr port builds.
Enabled by adding CONFIG_MICROPY_FROZEN_MODULES to the board configuration file.
Manually set manifest path with CONFIG_MICROPY_FROZEN_MANIFEST.

Testing

Tested on supermicro nrf52840 (zephyrproject-rtos/zephyr#80168), and the platforms i work on (zephyrproject-rtos/zephyr#78795)

@ThinkTransit
Copy link
Contributor

Tested and working on nrf5340dk

Copy link
Contributor

@projectgus projectgus left a comment

Choose a reason for hiding this comment

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

This change looks reasonable to me @VynDragon, thanks for contributing it! I can't think of a better place to enable frozen modules for Zephyr builds than the config file.

I have a few suggestions for minor cleanup and formatting, but otherwise looks good as far as I'm concerned! Thanks for contributing it.

@projectgus projectgus added this to the release-1.26.0 milestone Apr 8, 2025
Enables the ability to use frozen modules in the zephyr port.

Enabled by adding `CONFIG_MICROPY_FROZEN_MODULES` to the board
configuration file.  Manually set manifest path with
`CONFIG_MICROPY_FROZEN_MANIFEST`.

Signed-off-by: Vdragon <mail@massdriver.space>
@dpgeorge dpgeorge merged commit d939511 into micropython:master Apr 30, 2025
7 checks passed
@VynDragon VynDragon deleted the upstream_frozen branch April 30, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants