-
Notifications
You must be signed in to change notification settings - Fork 2k
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
boards/efm32-based: Model features in Kconfig #14310
boards/efm32-based: Model features in Kconfig #14310
Conversation
@leandrolanzieri I guess that there is not much to test, except that it passes CI? |
Yes, that and that the symbol organization makes sense. In this case it was pretty straightforward because the CPUs are nicely organized in families (thanks for that :) |
Should the modules like |
No, this is only to model features. |
That makes sense. Thanks for explaining. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also took a secondary look and the diff seems to be fine. Murdock also passes the tests. ACK
Thanks for reviewing! |
Contribution description
This models in Kconfig all the features of the efm32-based boards:
ikea-tradfri
slstk3401a
slstk3402a
sltb001a
slwstk6000b-slwrb4150a
slwstk6000b-slwrb4162a
stk3600
stk3700
Testing procedure
tests/kconfig_features
should pass for all boardsIssues/PRs references
Part of #14148