Skip to content

Variant addition: Nucleo-64 F410RB #2716

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

Merged
merged 1 commit into from
Apr 25, 2025
Merged

Conversation

askuric
Copy link
Contributor

@askuric askuric commented Apr 24, 2025

This PR adds the support for the nucleo f410rb variant.

Validation

  • The code has been tested using the CheckVariant example:

The built-in led blinks and this is the serial output:

#####
NUM_DIGITAL_PINS = 52
NUM_ANALOG_INPUTS = 16
#####
Checking digital pins...
End check digital pins
#####
Checking analog pins definition...
End check analog pins
#####
Checking IP instances (I2C, UART, SPI)...
End check IP instances (I2C, UART, SPI)

########################################
#### Test PASSED
########################################

Code formatting

I'm happy to adapt it to your requirements if needed.

** Comment **
This is a very constrained Nucleo variant with not that many features so it might not be used too much.
For example. the it only has a 4 few PWM pins in the Arduino R3 header which do not even correspond to the arduino's PWM pins.

@fpistm
Copy link
Member

fpistm commented Apr 24, 2025

Hi @askuric
Thanks for the PR.
Please, could you rebase on top of the main to avoid the 11 commits and have only the one for f410RB.

@askuric askuric force-pushed the variant_f410rb branch 2 times, most recently from 634141a to 6d96592 Compare April 25, 2025 08:31
@askuric askuric changed the base branch from main to codeql April 25, 2025 08:32
@askuric askuric changed the base branch from codeql to main April 25, 2025 08:32
@askuric
Copy link
Contributor Author

askuric commented Apr 25, 2025

Hi @fpistm,

Please, could you rebase on top of the main to avoid the 11 commits and have only the one for f410RB.

I'm not really sure how this separation in the history happened :/
In any case, the branch is rebased to the main and there is only one commit left.

Let me know if you need me to do other changes.

Signed-off-by: gospar <antun.skuric@outlook.com>
@fpistm
Copy link
Member

fpistm commented Apr 25, 2025

Hi @askuric
Thanks for the rebase.
I've made some fixes and push on your branch.
Mainly fix astyle issue and Pin number PD2 which does not exist and replaced by PB11.

@fpistm fpistm added the new variant Add support of new bard label Apr 25, 2025
@fpistm fpistm added this to the 2.11.0 milestone Apr 25, 2025
@fpistm fpistm merged commit bb565fd into stm32duino:main Apr 25, 2025
25 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in STM32 core based on ST HAL Apr 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new variant Add support of new bard
Projects
Development

Successfully merging this pull request may close these issues.

2 participants