Skip to content

boards/nucleo-l412kb: enable ADC #21566

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

krzysztof-cabaj
Copy link
Contributor

@krzysztof-cabaj krzysztof-cabaj commented Jun 28, 2025

Contribution description

This PR enables ADC for nucleo-l412kb board.

I don't have access to this boards, but because ADC pinout is the same as for nucleo-l432kc (see board manual tables 15 and 16) code should works fine. I compile code without any warnings.

Testing procedure

Flash the board using tests/periph/adc and tests/periph/vbat programs.

The outputs should be similar to the ones presented in #21238 (comment)

Issues/PRs references

None.

@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: boards Area: Board ports Area: cpu Area: CPU/MCU ports labels Jun 28, 2025
@crasbe crasbe added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Jun 28, 2025
@riot-ci
Copy link

riot-ci commented Jun 28, 2025

Murdock results

✔️ PASSED

7816da2 boards/nucleo-l412kb: add VBAT monitoring feature

Success Failures Total Runtime
10504 0 10504 18m:52s

Artifacts

Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Thx a lot ❤️

The code looks good and I trust in your testing

@maribu maribu added this pull request to the merge queue Jun 28, 2025
@crasbe
Copy link
Contributor

crasbe commented Jun 28, 2025

The code looks good and I trust in your testing

Which testing 👀

I don't have access to this boards, [...]

But yeah, let's hope it works as it is similar enough.

@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 28, 2025
@krzysztof-cabaj
Copy link
Contributor Author

Thank you for your trust ... but as @crasbe notices I don't have this boards and only look at documentation for similar board and check if tests/periph/adc and tests/periph/vbat compile for this board with success.

BTW. There is probably some problem with Murdock - from few days all full merge at 2 o'clock failed, in most cases due to some bizzare timeouts. Moreover, failed test for this PR is associated with timeout, too ... and I don't think that this is this PR fault.

@crasbe
Copy link
Contributor

crasbe commented Jun 30, 2025

Yes, that error is unrelated to your code. The timing tests are a bit finnicky lately which probably correlates to load on the CI servers.

I was waiting for a response from @maribu before putting it back on the queue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: cpu Area: CPU/MCU ports 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 Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants