Skip to content

boards/nucleo: fix Cortex-M4F family in doc page#21561

Merged
crasbe merged 2 commits intoRIOT-OS:masterfrom
krzysztof-cabaj:nucleo-fix-M4F-family
Jun 24, 2025
Merged

boards/nucleo: fix Cortex-M4F family in doc page#21561
crasbe merged 2 commits intoRIOT-OS:masterfrom
krzysztof-cabaj:nucleo-fix-M4F-family

Conversation

@krzysztof-cabaj
Copy link
Copy Markdown
Contributor

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

Contribution description

This PR fix family for Nucleo boards with Cortex-M4 with FPU, family changed to Cortex-M4F.
The issue was spotted in PR #21554 (comment) .

Testing procedure

Generate doc and check if everything looks good, for example:

make doc
xdg-open ./doc/doxygen/html/group__boards__nucleo-f439zi.html

Issues/PRs references

#21554 (comment)

@krzysztof-cabaj krzysztof-cabaj requested a review from aabadie as a code owner June 23, 2025 18:16
@github-actions github-actions bot added Area: doc Area: Documentation Area: boards Area: Board ports labels Jun 23, 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 CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Jun 23, 2025
@riot-ci
Copy link
Copy Markdown

riot-ci commented Jun 23, 2025

Murdock results

✔️ PASSED

01ce1e8 boards/nucleo-l433rc: fix MCU table caption

Success Failures Total Runtime
1 0 1 01m:20s

Artifacts

@crasbe
Copy link
Copy Markdown
Contributor

crasbe commented Jun 23, 2025

I think you could squash the three nucleo commits to something like boards/nucleo{32,64,144}: fix Cortex-M4F family.

@krzysztof-cabaj krzysztof-cabaj force-pushed the nucleo-fix-M4F-family branch from d4579c5 to 01ce1e8 Compare June 24, 2025 07:06
@krzysztof-cabaj
Copy link
Copy Markdown
Contributor Author

Done.

@crasbe crasbe added this pull request to the merge queue Jun 24, 2025
Merged via the queue into RIOT-OS:master with commit 5a7a567 Jun 24, 2025
25 checks passed
@krzysztof-cabaj
Copy link
Copy Markdown
Contributor Author

@crasbe Thanks for support!

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: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs 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