Skip to content

Added getAlarmedState() function#320

Open
CoolEou wants to merge 2 commits into
adafruit:masterfrom
CoolEou:master
Open

Added getAlarmedState() function#320
CoolEou wants to merge 2 commits into
adafruit:masterfrom
CoolEou:master

Conversation

@CoolEou

@CoolEou CoolEou commented Mar 19, 2026

Copy link
Copy Markdown

Adds a function that checks the interrupt flag of either of the two alarms on the DS3231. The name of the function could maybe be better, but other than that i haven't touched anything other than adding this function, so everything else should still work as normal, my change only adds a little bit of functionality to the library.

I needed this function to control an indicator LED that indicates to the user whether the alarm is active or not (indicating whether the alarm will go off at the alarm time or not).

@edgar-bonet

Copy link
Copy Markdown
Contributor

Note that the CI failure is due to clang-format complaining that there should be a space before the opening brace:

--- ./src/RTC_DS3231.cpp	(original)
+++ ./src/RTC_DS3231.cpp	(reformatted)
@@ -356,7 +356,7 @@
     @return False if interrupt flag is not set, otherwise true
 */
 /**************************************************************************/
-bool RTC_DS3231::getAlarmedState(uint8_t alarm_num){
+bool RTC_DS3231::getAlarmedState(uint8_t alarm_num) {
   uint8_t alarmedState = read_register(DS3231_CONTROL);
   alarmedState &= (1 << (alarm_num - 1));
   return alarmedState;

@justinnewitter

Copy link
Copy Markdown
Contributor

This is functionally equivalent to my PR that has been pending since 2022 🙈
#269

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants