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.

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

Fix problem with motion/door reset message arriving too early. #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AriZuu
Copy link
Contributor

@AriZuu AriZuu commented Aug 15, 2021

It seems that newer Coolcam PIR sensor sends a motion reset
message before actual motion on/off message. This causes unfortunate
side effect of esp8266 shutting down too early. As the reset
message is not used for anything, just acknowledge it to MCU
and keep going.

It seems that newer Coolcam PIR sensor sends a motion reset
message before actual motion on/off message. This causes unfortunate
side effect of esp8266 shutting down too early. As the reset
message is not used for anything, just acknowledge it to MCU
and keep going.
@AriZuu
Copy link
Contributor Author

AriZuu commented Aug 16, 2021

About these messages:

SB1_EVENT_TYPE_DOOR_RESET = 0x6504
SB1_EVENT_TYPE_MOTION_RESET = 0x6604

In README.md these messages are described as battery level messages. If so, it explains why they are arriving
early. Might be a good idea to rename these constants from _RESET to _BATTERY.

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.

1 participant