Skip to content

[fix] nucleo-h7431 updated user led pin and name#1344

Merged
salkinium merged 2 commits into
modm-io:developfrom
jankoerner:fix/nucleo_h743zi
Mar 28, 2026
Merged

[fix] nucleo-h7431 updated user led pin and name#1344
salkinium merged 2 commits into
modm-io:developfrom
jankoerner:fix/nucleo_h743zi

Conversation

@jankoerner
Copy link
Copy Markdown
Contributor

According to section 7.6.1 LEDs of the NUCLEO-H743ZI user manual is no blue user led.

Instead the board has a yellow user led connected to PE1.

@salkinium
Copy link
Copy Markdown
Member

Lol, you're correct, but ST has made this unnecessarily complex. What I have lying around is NUCLEO-H743ZI (MB1137), but what you likely have is NUCLEO-H743ZI2 (MB1364), which seems to be a different board layout?

I would recommend to add a new board definition called :board:nucleo-h743zi2 and use a Jinja2 template switch in the :board:nucleo-h743zi header file to service both. Example for the Black Pill F4 Boards.

@jankoerner
Copy link
Copy Markdown
Contributor Author

Thanks for your feedback!
I have a NUCLEO-H753ZI board which adds the Cryptography feature to the NUCLEO-H743ZI2 but is besides from that similar to the NUCLEO-H743ZI2 board.

Would you recommend to add an additional board definition :board:nucleo-h753zi for this board as well?

@salkinium
Copy link
Copy Markdown
Member

Yes absolutely, also with the jinja2 template trick if it's super similar like this.

What MB number is on your board?

@jankoerner
Copy link
Copy Markdown
Contributor Author

Okay, I think it is MB1364.
But I need to check again on Monday.

@jankoerner
Copy link
Copy Markdown
Contributor Author

I just checked, MB1364E is on the board.

Copy link
Copy Markdown
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Very nice!

Comment thread src/modm/board/nucleo_h743zi/board.hpp.in Outdated
Comment thread src/modm/board/nucleo_h743zi/board.hpp.in Outdated
Copy link
Copy Markdown
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Sorry for the wait, I was on business trips back to back and just got back.

@salkinium salkinium merged commit 85d2724 into modm-io:develop Mar 28, 2026
12 checks passed
@salkinium salkinium added this to the 2026q1 milestone Mar 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants