Skip to content
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

[stm32] add advanced initialize on iwdg #1139

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

victorandrehc
Copy link
Contributor

This PR reinstates the ability to directly set the prescale and reload values on iwdg without letting modm to calculate this value directly.

@rleh rleh added this to the 2024q1 milestone Mar 8, 2024
@victorandrehc victorandrehc force-pushed the feature/advanced_iwdg branch from 54e013d to 3bf6c81 Compare March 8, 2024 10:38
initialize(Prescaler prescaler, uint16_t reload)
{
configure(prescaler, reload);
}
Copy link
Member

Choose a reason for hiding this comment

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

Let's instead rename configure to initialize and move it here to avoid the unnecessary forwarding.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just pushed a version without the forwarding. Please also don't forget the PR #1136.

@victorandrehc victorandrehc force-pushed the feature/advanced_iwdg branch from 3bf6c81 to 97067a6 Compare March 11, 2024 07:13
Copy link
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.

Thanks!

@salkinium salkinium merged commit 9496060 into modm-io:develop Mar 11, 2024
12 checks passed
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.

3 participants