-
Notifications
You must be signed in to change notification settings - Fork 23
BCDAlarmTimeRegister does not support minutely on DS3231 #50
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
Comments
A short summary: alarm2 has minutes but they are not supported. alarm2 does not have seconds and justly are not supported. Changing line 172 of "i2c_bcd_alarm.py" to frequency < 1 fixes the issue. Status will need to be updated too. |
Reading the forum, it looks like that line is (nearly) what the fix should be. Am I misunderstanding the issue? |
I think there is more to be investigated. It seems like there is a base assumption that supporting "minutely" requires seconds to be supported by the RTC: So that logic is correct based on that. But I do not know what drives that requirement. If the DS3231's alarm2 is an odd ball, then maybe the answer is the DS3231 library can not use BCDAlarmTimeRegister? |
I am currently using minutely without problems from alarm2 on the DS3231 using the fix I have suggested. The part I can't find/fix is where in the other libraries "None" is reported when there is a status request of alarm2 on the DS3231 when minutely has been selected. The PCF8523 mentioned above is a different chip which should have a different sub-function in the adafruit_register.py library. Limitations of the PCF8523 should not block functionality of the DS3231. |
I just looked at the data sheets on the PCF8523 and the DS3231. The internal registers do not match. e.g. The PCF8523 uses the first three registers for control where the DS3231 uses them for real time seconds, minutes, hours. The PCF8523 has only one alarm so how can it be a reason to limit the second alarm of the DS3231? |
I downloaded fresh all of the library from here and ran it. It still says minutley is not supported. Line 172 of i2c_bcd_alarm.py needs to read: |
I found where the minutely vs None issue resides: It is lines 108 - 130. I am not sure what the change should be to follow the github rules, etc. |
This code addresses the issue of 'None' being displayed when 'minutely' is selected for alarm2. ~ Line 108
|
Fixed by #57 |
Thanks guys for all your help. Especially Mikey! |
More discussion here:
https://forums.adafruit.com/viewtopic.php?t=201104
The registers for Alarm 2 of the DS3231 do not have seconds:

However, "minutely" (once per minute) triggering is supported:

The text was updated successfully, but these errors were encountered: