-
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/nucleo*: move HSE/LSE configuration for Kconfig to common #15634
boards/nucleo*: move HSE/LSE configuration for Kconfig to common #15634
Conversation
4a4a519
to
1f92845
Compare
1f92845
to
de37259
Compare
no longer waiting for another PR. |
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.
Looks good, I'll run some tests
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, changes look good all timing related tests are still passing https://ci.inria.fr/ci-riot-tribe/job/build-pipeline.jk/214/artifact/ (the failing rtc test was fixed in another unrelated PR)
The build failed on Murdock because of Kconfig: there was a wrong selection of the common kconfig symbol (nucleo64 instead of nucleo144). |
Contribution description
Most of the nucleo boards share the same HSE/LSE hardware configuration:
Instead of describing this for each nucleo boards, move the selection of
BOARD_HAS_HSE
/BOARD_HAS_LSE
config to boards/common/nucleo32/64/144 Kconfig files.Testing procedure
Nucleo boards are still functional when Kconfig is used and the configuration is as expected.
Issues/PRs references
#14975
based on #15632