Skip to content

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

Merged
fpistm merged 2 commits into
stm32duino:mainfrom
FRASTM:lorawan_issue38
Aug 19, 2024
Merged

stm32RTC start alarm with a 64bit accuracy on Subseconds param#109
fpistm merged 2 commits into
stm32duino:mainfrom
FRASTM:lorawan_issue38

Conversation

@FRASTM

@FRASTM FRASTM commented Mar 12, 2024

Copy link
Copy Markdown
Contributor

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
Comment thread src/rtc.c Outdated
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 <francois.ramu@st.com>
@fpistm fpistm added this to the 1.5.0 milestone Apr 12, 2024

@fpistm fpistm left a comment

Copy link
Copy Markdown
Member

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
Comment thread src/rtc.c Outdated
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm merged commit 62a6175 into stm32duino:main Aug 19, 2024
@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