Skip to content
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

Allow per module prj.conf #21617

Closed
hadrienk opened this issue Dec 28, 2019 · 5 comments
Closed

Allow per module prj.conf #21617

hadrienk opened this issue Dec 28, 2019 · 5 comments
Assignees
Labels
area: Build System area: Kconfig Enhancement Changes/Updates/Additions to existing features

Comments

@hadrienk
Copy link

hadrienk commented Dec 28, 2019

Is your enhancement proposal related to a problem? Please describe.

I am using mcuboot. I need to manually update the .config under build/mcuboot/zephyr/.config when the build directory is cleaned.

Describe the solution you'd like

Use a special configuration file name convention: prj_mcuboot.conf

This should be compatible with the board configuration prj_mcuboot_YOURBOARD.conf

@hadrienk hadrienk added the Enhancement Changes/Updates/Additions to existing features label Dec 28, 2019
@carlescufi
Copy link
Member

@hadrienk thanks for the feedback. spm is not part of Zephyr, but rather only present in Nordic's Zephyr-based SDK. Could you edit your proposal so that it doesn't mention spm and instead focuses on mcuboot?

@hadrienk
Copy link
Author

hadrienk commented Jan 3, 2020

@carlescufi sure thing.

@carlescufi
Copy link
Member

@ulfalizer could you comment on this?

@SebastianBoe
Copy link
Collaborator

In NCS one can specify the mcuboot prj.conf like this:

https://github.com/NordicPlayground/fw-nrfconnect-nrf/blob/master/applications/nrf_desktop/CMakeLists.txt#L34

In Zephyr upstream it doesn't make sense to put a child image's build dir inside of the parent image's build dir so this problem does not exist.

Please re-open if more clarification is needed.

@hadrienk
Copy link
Author

Thanks. Got it to work. If anyone want to do the same, remember to set the image_CONF_FILE before the include($ENV{ZEPHYR_BASE}/cmake/app/boilerplate.cmake)

set(mcuboot_CONF_FILE "${CMAKE_CURRENT_SOURCE_DIR}/mcuboot.conf")
include($ENV{ZEPHYR_BASE}/cmake/app/boilerplate.cmake)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System area: Kconfig Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

5 participants