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.

Sign up for GitHub

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

Jump to bottom

[cmake] A board should support multiple configurations (variants) #1256

Closed
nashif opened this issue Aug 24, 2017 · 1 comment
Closed

[cmake] A board should support multiple configurations (variants) #1256

nashif opened this issue Aug 24, 2017 · 1 comment
Assignees
@nashif
Labels
Enhancement Changes/Updates/Additions to existing features
Milestone
v1.10.0

Comments

@nashif
Copy link
Member

nashif commented Aug 24, 2017

right now we support 1 defconfig per board, there are cases where different configuration might be required, for example one will be fully loaded with all peripherals loaded, another would be only enabling basic IOs. We should also be able to have defconfigs for testing and test harnesses etc.

The idea is to have one yaml file per board and support multiple variants.
@nashif nashif added the Enhancement Changes/Updates/Additions to existing features label Aug 24, 2017
@nashif nashif modified the milestones: v1.9, v1.10 Aug 26, 2017
@nashif nashif changed the title A board should support multiple configurations (variants) [cmake] A board should support multiple configurations (variants) Aug 26, 2017
@nashif nashif added the In progress For PRs: is work in progress and should not be merged yet. For issues: Is being worked on label Sep 21, 2017
@nashif nashif self-assigned this Oct 2, 2017
@nashif
Copy link
Member Author

nashif commented Oct 2, 2017

done in cmake

@nashif nashif closed this as completed Oct 2, 2017
@ghost ghost removed the In progress For PRs: is work in progress and should not be merged yet. For issues: Is being worked on label Oct 2, 2017
@nashif nashif modified the milestones: v1.10, v1.10.0 Oct 3, 2017
nagineni pushed a commit to nagineni/zephyr that referenced this issue Nov 20, 2017
@grgustaf
[analyze] Handle whitespace in parsing for require()
8458c8a 
Also, replace four similar checks for require with a common function.

Fixes ble portion of zephyrproject-rtos#1256.

Signed-off-by: Geoff Gustafson <[email protected]>
nagineni pushed a commit to nagineni/zephyr that referenced this issue Nov 20, 2017
@grgustaf
[ws] Make websockets module name match require
73a36db 
This is a workaround because analyze is mixing up the 'module' display
name and the 'require' name, better solution is to allow those to be
different.

Fixes websocket portion of zephyrproject-rtos#1256.

Signed-off-by: Geoff Gustafson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nashif nashif

Labels
Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Milestone
v1.10.0
Development

No branches or pull requests
1 participant
@nashif