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.

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

add new sketch to change the rtc mode #108

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 this to the 1.5.0 milestone Feb 28, 2024
fpistm
fpistm requested changes Feb 28, 2024
View reviewed changes
examples/changeMode/changeMode.ino Outdated Show resolved Hide resolved
@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 <[email protected]>
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
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fpistm fpistm fpistm approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
STM32duino libraries
Status: Done
Milestone
1.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@FRASTM @fpistm