Skip to content
Snippets Groups Projects

Watchdog timer implemented

Merged Jeff Gough requested to merge jeffmakes/firmware:wdt into master

Enable watchdog timer on boot. Add FreeRTOS task to clear the timer. Watchdog expires after 10.7 seconds, and is cleared every 8 seconds.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • rahix
  • rahix
  • added 3-Epicardium label

  • Author Contributor

    Oops, forgot to remove that debug message! Will do. Ack on the other suggestions. Schneider suggested not making the watchdog too aggressive. How about 2 seconds?

  • 2 seconds sounds good, go ahead!

  • rahix changed milestone to %v1.4

    changed milestone to %v1.4

  • @jeffmakes, I have nominated this MR for version 1.4 which is due in a few hours. What is your state on this?

  • Author Contributor

    Starting in 10 min, should get it done for 1.4. I'm just reading about FreeRTOS Timers...

  • If I can answer any questions ping me!

  • If you haven't seen it already: https://www.freertos.org/xTimerCreateStatic.html contains an example.

  • rahix assigned to @rahix

    assigned to @rahix

  • Jeff Gough added 6 commits

    added 6 commits

    • 4da6dd8a - Set watchdog to expire after 2.7 seconds
    • 665ee44a - Refactor to use software timer instead of task to clear the watchdog.
    • 03ca1ae0 - Output log message in case of system reset due to watchdog timeout
    • f4b4ef71 - Tweak watchdog reset log message
    • 4b77f64c - Clear watchdog every 2 seconds
    • b43f20b5 - Ran code-format.sh

    Compare with previous version

  • Jeff Gough resolved all threads

    resolved all threads

  • Jeff Gough added 1 commit

    added 1 commit

    • ab9a3bce - Remove log message on watchdog clear

    Compare with previous version

  • Author Contributor

    Ok, now running nicely with the software timer. Also added a log message that prints on boot if the system was reset due to watchdog timeout. Other than that, there is no noise in the log.

  • LGTM, will test, then merge

  • merged

  • rahix mentioned in commit 2e84e69a

    mentioned in commit 2e84e69a

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading