Skip to content

sys/bus: model Kconfig #17791

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
Merged
merged 1 commit into from
Mar 14, 2022
Merged

sys/bus: model Kconfig #17791

merged 1 commit into from
Mar 14, 2022

Conversation

leandrolanzieri
Copy link
Contributor

Contribution description

This models the system buses in Kconfig.

Testing procedure

  • Check usability via menuconfig
  • Green CI

Issues/PRs references

Split from #17789

@leandrolanzieri leandrolanzieri added Type: new feature The issue requests / The PR implemements a new feature for RIOT Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer labels Mar 11, 2022
@github-actions github-actions bot added Area: Kconfig Area: Kconfig integration Area: sys Area: System labels Mar 11, 2022
@leandrolanzieri leandrolanzieri mentioned this pull request Mar 11, 2022
Merged
@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 Mar 11, 2022
MrKevinWeiss
MrKevinWeiss approved these changes Mar 11, 2022
View reviewed changes
Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

Yup, we can change the entry point if this becomes an issue.

@MrKevinWeiss MrKevinWeiss merged commit 5acf191 into RIOT-OS:master Mar 14, 2022
@leandrolanzieri leandrolanzieri deleted the pr/sys/bus/model_kconfig branch March 14, 2022 08:26
@leandrolanzieri
Copy link
Contributor Author

Thanks for the review!

@OlegHahm OlegHahm added this to the Release 2022.04 milestone Apr 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MrKevinWeiss MrKevinWeiss MrKevinWeiss approved these changes

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

@cgundogan cgundogan Awaiting requested review from cgundogan

Assignees
No one assigned
Labels
Area: Kconfig Area: Kconfig integration Area: sys Area: System CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Impact: minor The PR is small in size and might only require a quick look of a knowledgeable reviewer Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Milestone
Release 2022.04
Development

Successfully merging this pull request may close these issues.
3 participants
@leandrolanzieri @MrKevinWeiss @OlegHahm