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

Allow compile against latest emonlib, transmit fixes and add watchdog #2

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

alandpearson
Copy link
Contributor

Sorry about the formatting making the diffs hard to read.. CLANG clobbered the code.

  1. Transmit fix to correctly terminate received serial string with "\0" in all cases
  2. Add watchdog timer of 2 seconds to restart in case of hanging
  3. Allow compilation against GitHub hosted emonlib (looks like EmonLibCM_setPulseMinPeriod was changed in emonlib.h breaking compilation)
  4. Bump firmware version
  5. Cleanup/improve logging for transmit

Not sure you actual ship/use this firmware any longer, but it would be good to stay in sync.

(old handling caused packets to combine if send command close together)
This is not perfect, if two send commands arrive close together, it
appears only the first one is sent. Better that than an erronous packet
in the previous case.
'\0' in all cases)
Improve clarity of logging messages when sending
EmonLibCM_setPulseMinPeriod
Add watchdog
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