How to stop an defined alarm about ds3231m HOT 10 CLOSED

I see what you mean; you've done the correct thing by setting the alarm state to FALSE in the SetAlarm() method.

Looking at my code for the method isAlarm(), I don't check whether or not that bit is set. And since the 2 bits for the alarms keep their old values, this results in the problem you see.

I'm going to make some changes to the isAlarm() method in the library. I don't have a DS3231M handy to test my logic, can you check to see if it is now working?

from ds3231m.

I've made the changes to the isAlarm() method. Can you add that to your code and see if that correctly resolves the alarm problem?

from ds3231m.

Hi,
I try the new version, there is a problem, now the alarm is fire on every test : if (isAlarm() ...
like the isAlarm() is always true even with the clearAlarm()

from ds3231m.

That's not good :(
I will look at the code again; it sounds like I've mixed up some && and || conditions...

from ds3231m.

OK, now I think I've done it correctly, I also simplified the logic to

uint8_t controlByte = readByte(DS3231M_CONTROL) & 0x02; // Mask out all but last 2 bits for ALM1&2
uint8_t statusByte = readByte(DS3231M_STATUS) & 0x02; // Mask out all but last 2 bits for ALM1&2

The last two bits of both variables contain corresponding bits for "active" and "alarmed", so "and"ing them together will give either 0 for no alarms, or 0,1 and 2 (11b, 10b, 01b) for the alarm that is set.

from ds3231m.

Sorry, I didn't want to close this until you confirm that it is working.

from ds3231m.

hi,

I will do more test but for the first test it seem to work 👍

from ds3231m.

Excellent news. I've rechecked the datasheet and my programming logic and can't find a mistake. But I'll wait for your further testing before you or I close the ticket.

from ds3231m.

Hello

All tests i made are OK, i think you can close the case.
regards

from ds3231m.

That's good news. I think I'll draft a new version that incorporates this fix.

from ds3231m.