-
Notifications
You must be signed in to change notification settings - Fork 2.1k
cpu/cc2538: Add Kconfig support #16719
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
Conversation
I just want to check the CI output first to see what the state is. It looks like there is some radio work done but maybe that would be better to add in another PR (especially since I think it would also be testing the boards with radios such as the samr) |
23a9877
to
3bf7e10
Compare
Some discussion was had about how |
6302bea
to
cff8a5c
Compare
finally got it working... ignore the bot |
Lets see what murdock says. |
There we go, so I just removed the submac test as we are not adding netdev for this PR. |
3b90dfc
to
701087c
Compare
701087c
to
c083f36
Compare
Rebased and fixed the DEVELHELP export (and added some documentation on the variable). ping @fjmolinas To test changes
You can repeat without TEST_KCONFIG to verify it is the same with make. |
Look good to me, IMO you can squash, any remaining concerns @leandrolanzieri? |
c083f36
to
03df42a
Compare
03df42a
to
064a1dc
Compare
None, it looks good to me! |
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.
ACK!
There seems to be an unrelated issue on the CI |
Seems like some kind of lack of space issue.. re-triggered |
Thanks for the review and handholding :) |
Contribution description
Add kconfig dependency modeling for cc2538 boards based of @leandrolanzieri work.
This adds the support for the cc2538 remote based boards and radio.
It also adds some ieee802154 based kconfig tests.
Testing procedure
Check murdock is green and kconfig tests are run for the
remote-revb
Issues/PRs references