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

DBC Assertion Implementation #69

Merged
merged 1 commit into from
Nov 11, 2024
Merged

DBC Assertion Implementation #69

merged 1 commit into from
Nov 11, 2024

Conversation

ztnel
Copy link
Contributor

@ztnel ztnel commented Nov 11, 2024

Description

Implemented a small assertion library which allows tracing assertion failures without the need to store variable length strings for filenames etc. The assertion saves the PC and LR registers into a global assert tracing struct for inspection. For debug builds the assert halts the cpu with bkpt to allow developers to inspect the tracing struct while the CPU is halted. In release builds we enter a spin lock to allow the hardware watchdog to reset the CPU.

We will need a way to save the trace variables to NVM so it can be reported on next boot.

Issues

closes #68

@ztnel ztnel added firmware embedded software development enhancement Lower priority improvements labels Nov 11, 2024
@ztnel ztnel self-assigned this Nov 11, 2024
Copy link

codecov bot commented Nov 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 36.46%. Comparing base (7cc4aac) to head (7864033).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #69   +/-   ##
=======================================
  Coverage   36.46%   36.46%           
=======================================
  Files          12       12           
  Lines         650      650           
  Branches      185      185           
=======================================
  Hits          237      237           
  Misses        303      303           
  Partials      110      110           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ztnel ztnel merged commit 34713ae into master Nov 11, 2024
5 checks passed
@ztnel ztnel deleted the assertions branch November 11, 2024 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Lower priority improvements firmware embedded software development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement DBC Asserts
1 participant