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

harden prediv management and add parameter to set binary mode #98

Merged
merged 7 commits into from
Sep 25, 2023

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Sep 8, 2023

  • fix: harden prediv management vs clock config

    prediv was not properly computed and some misalignment's could occur depending of the RTC state at init.

  • feat: add a parameter to set binary mode

    This is valid when the RTC_BINARY_MIX mode exists in the RTC (bitfield in the RTC ICSR register)
    Set the RTC mode through a setBinaryMode function to be called before begin.

fpistm and others added 3 commits September 15, 2023 10:01
else this prevent isConfigured(void) to return correct value.

Signed-off-by: Frederic Pillon <[email protected]>
prediv was not properly computed and some misalignment's
could occur depending of the RTC state at init.

Signed-off-by: Frederic Pillon <[email protected]>
(a)synchronous prescalers values depend on the RTC clock source.

Signed-off-by: Frederic Pillon <[email protected]>
This is valid when the RTC_BINARY_MIX mode exists in the RTC
(bitfield in the RTC ICSR register)
Set the RTC mode through a setBinaryMode function to be called
before begin.

Signed-off-by: Francois Ramu <[email protected]>
Co-authored-by: Frederic Pillon <[email protected]>
Formatted with Arduino astyle.

Signed-off-by: Frederic Pillon <[email protected]>
Signed-off-by: Frederic Pillon <[email protected]>
@fpistm fpistm merged commit 5f53d03 into stm32duino:main Sep 25, 2023
3 checks passed
@fpistm fpistm deleted the prediv branch September 25, 2023 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

2 participants