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

cpu/{cc26xx, cc13xx}: model kconfig #17236

Merged
merged 5 commits into from Nov 21, 2021
Merged

cpu/{cc26xx, cc13xx}: model kconfig #17236

merged 5 commits into from Nov 21, 2021

Conversation

leandrolanzieri
Copy link
Contributor

Contribution description

This models the Kconfig modules for all boards based on cc26xx and cc13xx CPUs. The list of boards is:

  • cc1312-launchpad
  • cc1352-launchpad
  • cc1352p-launchpad
  • cc2650-launchpad
  • cc2650stk
  • cc1350-launchpad

Testing procedure

  • Green CI
  • Check the modelling and compare make info-modules list with and without TEST_KCONFIG=1, they should match.

Issues/PRs references

Part of #16875

@github-actions github-actions bot added Area: boards Area: Board ports Area: CI Area: Continuous Integration of RIOT components Area: cpu Area: CPU/MCU ports Area: Kconfig Area: Kconfig integration Platform: ARM Platform: This PR/issue effects ARM-based platforms labels Nov 19, 2021
@leandrolanzieri leandrolanzieri added this to the Release 2022.01 milestone Nov 19, 2021
@leandrolanzieri leandrolanzieri mentioned this pull request Nov 19, 2021
Closed
jeandudey
jeandudey approved these changes Nov 19, 2021
View reviewed changes
Copy link
Contributor

@jeandudey jeandudey left a comment

Choose a reason for hiding this comment

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

LGTM

[CTF] Kconfig migration - Model of module dependencies automation moved this from In progress to Reviewer approved Nov 19, 2021
@leandrolanzieri leandrolanzieri added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Nov 19, 2021
@leandrolanzieri
Copy link
Contributor Author

Thanks! Let's run a quick CI check, if everything is green I'll remove the last commit.

@leandrolanzieri leandrolanzieri force-pushed the pr/cpu/cc26xx_cc13xx/model_kconfig branch from c107b7f to 279b0aa Compare November 19, 2021 16:04
@leandrolanzieri
Copy link
Contributor Author

CI passed, I removed the limiting commit

@jeandudey jeandudey merged commit 7ab6729 into RIOT-OS:master Nov 21, 2021
[CTF] Kconfig migration - Model of module dependencies automation moved this from Reviewer approved to Done Nov 21, 2021
@leandrolanzieri leandrolanzieri deleted the pr/cpu/cc26xx_cc13xx/model_kconfig branch November 22, 2021 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeandudey jeandudey jeandudey approved these changes

@cgundogan cgundogan Awaiting requested review from cgundogan

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@kaspar030 kaspar030 Awaiting requested review from kaspar030 kaspar030 is a code owner

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

Assignees
No one assigned
Labels
Area: boards Area: Board ports Area: CI Area: Continuous Integration of RIOT components Area: cpu Area: CPU/MCU ports Area: Kconfig Area: Kconfig integration CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms
Projects
[CTF] Kconfig migration - Model of mo...
  
Done
Milestone
Release 2022.01
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@leandrolanzieri @jeandudey