Skip to content

add new sketch to change the rtc mode #108

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 1, 2024
Merged

add new sketch to change the rtc mode #108

merged 1 commit into from
Mar 1, 2024

Conversation

FRASTM
Copy link
Contributor

@FRASTM FRASTM commented Feb 28, 2024 •
edited
Loading

New application to demonstrate that changing the RTC mode from BIN to MIX and BCD does not affect the calendar nor the Alarm

Requires #106
@FRASTM
Copy link
Contributor Author

FRASTM commented Feb 28, 2024

Output log:

12:08:13.010 -> RTC mode is MODE_BCD at 01/03/24 - 14:02:58 12:08:15.000 -> Alarm A Match! : change RTC mode at 14:03:00 12:08:15.001 -> RTC mode is MODE_MIX at 01/03/24 - 14:03:00 12:09:15.005 -> Alarm A Match! : change RTC mode at 14:04:00 12:09:15.005 -> RTC mode is MODE_BIN at 01/03/24 - 14:04:00 12:10:14.992 -> Alarm A Match! : change RTC mode at 14:05:00 12:10:14.992 -> RTC mode is MODE_BCD at 01/03/24 - 14:05:00 ...
@FRASTM
Copy link
Contributor Author

FRASTM commented Feb 28, 2024

rebase
@fpistm fpistm marked this pull request as ready for review February 28, 2024 15:58
@fpistm fpistm added the enhancement New feature or request label Feb 28, 2024
@fpistm fpistm added this to the 1.5.0 milestone Feb 28, 2024
fpistm
fpistm requested changes Feb 28, 2024
View reviewed changes
@FRASTM
new sketch to demonstrate the RTC mode change
9878cfa 
New application to demonstrate that changing the RTC mode from BIN to BCD does not affect the calendar nor Alarm. Signed-off-by: F. Ramu <francois.ramu@st.com>
fpistm
fpistm approved these changes Feb 28, 2024
View reviewed changes
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 merged commit fd5a7bc into stm32duino:main Mar 1, 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

2 participants

@FRASTM @fpistm