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

Add watchdog component #45

Merged
merged 3 commits into from
Nov 9, 2024
Merged

Add watchdog component #45

merged 3 commits into from
Nov 9, 2024

Conversation

nateinaction
Copy link
Member

@nateinaction nateinaction commented Oct 25, 2024

Partially completes #27

Original acceptance criteria:

  • Initialize the WDT_WDI pin in a LOW state
  • Drive the WDT_WDI pin HIGH for a 100ms (arbitrary amount of time) pulse
  • Ensure the watchdog is pet at least every 20 seconds
  • Log if the watchdog petting is slipping from real time (i.e. other software components are slowing down and causing this component to be late)

Delivered:

Roughly every 100ms the watchdog pin is toggled on for 100ms. Telemetry is emitted every time the pin is toggled on.

Not delivered:

  • Ensure the watchdog is pet at least every 20 seconds
    • Reason: There is an issue on the v4b board that causes the watchdog to trigger roughly every 1 second, on the v4b board the watchdog triggers roughly every 5 seconds
  • Log if the watchdog petting is slipping from real time (i.e. other software components are slowing down and causing this component to be late)
    • Reason: I don't think we can do this without the RTC implementation

@nateinaction nateinaction marked this pull request as ready for review October 25, 2024 19:32
@nateinaction nateinaction requested review from Mikefly123 and a team November 5, 2024 07:45
Copy link
Collaborator

@Lex-ari Lex-ari left a comment

Choose a reason for hiding this comment

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

Looks good! Amazing work 🚀

Copy link
Member

@Mikefly123 Mikefly123 left a comment

Choose a reason for hiding this comment

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

IMG_4293.mov

Looks good to me as well! I was here with Alex and we verified it works with V4c FC Boards (and by proxy should be good on V4b as well). I think it should be no problem to bounce the timing features to a future issue ticket!

@Mikefly123 Mikefly123 added the new component Creating a new component for this task label Nov 9, 2024
@Mikefly123 Mikefly123 added this to the fprime-proves-v1.0 milestone Nov 9, 2024
@nateinaction nateinaction merged commit f2b4fe1 into devel Nov 9, 2024
1 check passed
@nateinaction nateinaction deleted the ngay-watchdog branch November 9, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new component Creating a new component for this task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants