Skip to content

stm32RTC start alarm with a 64bit accuracy on Subseconds param #109

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 2 commits into from
Aug 19, 2024

Conversation

FRASTM
Copy link
Contributor

@FRASTM FRASTM commented Mar 12, 2024

When the number of SubSeconds exceeds 32bit value during calculations the Alarm still needs to be set with bigger range. This is done with a new RTC_StartAlarm64 function.

@FRASTM FRASTM force-pushed the lorawan_issue38 branch 3 times, most recently from 38ab929 to e854601 Compare March 12, 2024 10:50
@FRASTM FRASTM force-pushed the lorawan_issue38 branch 2 times, most recently from 380b8b9 to f1db527 Compare March 12, 2024 10:58
@fpistm fpistm marked this pull request as ready for review April 10, 2024 08:41
@fpistm fpistm added the enhancement New feature or request label Apr 10, 2024
When the number of SubSeconds exceeds 32bit value during calculations
the Alarm still needs to be set with bigger range. This is done
with a new RTC_StartAlarm64 function.

Signed-off-by: F. Ramu <[email protected]>
@fpistm fpistm added this to the 1.5.0 milestone Apr 12, 2024
Copy link
Member

@fpistm fpistm left a comment

Choose a reason for hiding this comment

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

LGTM

@fpistm fpistm removed this from the 1.5.0 milestone Aug 19, 2024
Signed-off-by: Frederic Pillon <[email protected]>
@fpistm fpistm merged commit 62a6175 into stm32duino:main Aug 19, 2024
3 checks passed
@fpistm fpistm deleted the lorawan_issue38 branch August 19, 2024 08:52
@fpistm fpistm added this to the 1.5.0 milestone Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

2 participants