Skip to content

Test/try reverting watchdog changes #365

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.

Sign up for GitHub

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

Jump to bottom
Merged
victorsowa12 merged 5 commits into from
Dec 5, 2025
Merged

Test/try reverting watchdog changes #365

victorsowa12 merged 5 commits into from
Dec 5, 2025

Conversation

@victorsowa12
Copy link
Collaborator

@victorsowa12 victorsowa12 commented Dec 4, 2025 •
edited
Loading

What changed?

This PR is in response to Aanderaa current meter apps unexpectedly resetting. Through testing we determined that the watchdog changes in #316 were causing the Aanderaa app to reset. The current meter app appears to be the only app impacted by this at this time. While we are still digging into the details this will allow us to continue shipping current meters without them resetting.

How does it make Bristlemouth better?

Prevents the Aanderaa app from resetting.

Where should reviewers focus?

Double check that I didn't miss any APP_AANDERAA_CURRENT_METER locations.

Checklist

  • Add or update unit tests for changed code
  • Ensure all submodules up to date. If this PR relies on changes in submodules, merge those PRs first, then point this PR at/after the merge commit
  • Ensure code is formatted correctly with clang-format. If there are large formatting changes, they should happen in a separate whitespace-only commit on this PR after all approvals.

@victorsowa12 victorsowa12 merged commit 5d20f16 into develop Dec 5, 2025
1 check passed
@victorsowa12 victorsowa12 deleted the test/try_reverting_watchdog_changes branch December 5, 2025 18:56
matt001k added a commit that referenced this pull request Dec 8, 2025
@matt001k
Revert "Test/try reverting watchdog changes (#365)"
f19f50b 
This reverts commit 5d20f16.
matt001k added a commit that referenced this pull request Dec 9, 2025
@matt001k
Fix Low Power Mode May Get Stuck Errata 2.2.26 (#366)
e7f1ebe 
The STM32U5 might get stuck in low power mode if the WFI instruction is
not 128 bit aligned.
Also, revert "Test/try reverting watchdog changes (#365)"

This reverts commit 5d20f16.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@towynlin towynlin towynlin approved these changes

@matt001k matt001k matt001k approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@victorsowa12 @towynlin @matt001k