Skip to content

Add an external watchdog module#1446

Open
Quency-D wants to merge 6 commits intomeshcore-dev:devfrom
Quency-D:solar-watchdog
Open

Add an external watchdog module#1446
Quency-D wants to merge 6 commits intomeshcore-dev:devfrom
Quency-D:solar-watchdog

Conversation

@Quency-D
Copy link
Contributor

  1. It has already been tested on heltec-mesh-solar and heltec-v4.
  2. The newly added watchdog module does not conflict with the existing meshsolar.

@0x156
Copy link

0x156 commented Jan 27, 2026

Found this from here: http://community.heltec.cn/t/heltec-meshtower-resets-its-configuration-itself/20570/36?u=brikots
I have flashed it to my Heltec mesh solar/tower, and I can confirm it works properly: it does not reboot every 9 minutes.

Copy link
Contributor

@weebl2000 weebl2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few suggestions — the main one is a units mismatch (ms vs seconds) in the repeater sleep guard.

@Quency-D
Copy link
Contributor Author

Quency-D commented Feb 9, 2026

@weebl2000 Thank you for your suggestions. All have been revised.

@liamcottle
Copy link
Member

Please refactor all instances of ex to external, so the language is more explicit.

E.g:

  • HAS_EX_WATCHDOG -> HAS_EXTERNAL_WATCHDOG
  • EX_WATCHDOG_DONE_PIN -> EXTERNAL_WATCHDOG_DONE_PIN
  • EX_WATCHDOG_WAKE_PIN -> EXTERNAL_WATCHDOG_WAKE_PIN
  • EX_WATCHDOG_TIMEOUT_MS -> EXTERNAL_WATCHDOG_TIMEOUT_MS
  • ExWatchdogManager -> ExternalWatchdogManager
  • SolarExWatchdog -> SolarExternalWatchdog
  • ex_watchdog -> external_watchdog

@weebl2000
Copy link
Contributor

@Quency-D addressed the review comments and opened a PR to your branch Quency-D#4

@weebl2000
Copy link
Contributor

@liamcottle addressed the comments, variables are clearly named now!

@Quency-D
Copy link
Contributor Author

Quency-D commented Mar 4, 2026

@Quency-D addressed the review comments and opened a PR to your branch Quency-D#4

Thank you for your contribution; the connection has been merged.

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.

4 participants