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 fpistm requested a review from FRASTM September 8, 2023 16:22
@fpistm fpistm added this to In progress in STM32duino libraries via automation Sep 8, 2023
@fpistm fpistm added this to the 1.4.0 milestone Sep 13, 2023
@fpistm fpistm force-pushed the prediv branch 2 times, most recently from ee39147 to 0385636 Compare September 14, 2023 16:54
fpistm and others added 3 commits September 15, 2023 10:01
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
else this prevent isConfigured(void) to return correct value.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
prediv was not properly computed and some misalignment's
could occur depending of the RTC state at init.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
(a)synchronous prescalers values depend on the RTC clock source.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
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 <francois.ramu@st.com>
Co-authored-by: Frederic Pillon <frederic.pillon@st.com>
Formatted with Arduino astyle.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm merged commit 5f53d03 into stm32duino:main Sep 25, 2023
3 checks passed
STM32duino libraries automation moved this from In progress to Done Sep 25, 2023
@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.

None yet

2 participants