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 error handling for FRAM code #185

Closed
Tracked by #103
PatrickKa opened this issue Dec 14, 2023 · 4 comments
Closed
Tracked by #103

Add error handling for FRAM code #185

PatrickKa opened this issue Dec 14, 2023 · 4 comments

Comments

@PatrickKa
Copy link
Contributor

Description

Use the Experimental Outcome library for proper error handling in the FRAM code.

@PatrickKa PatrickKa changed the title Implement error handling for FRAM code Add error handling for FRAM code Dec 15, 2023
@PatrickKa
Copy link
Contributor Author

Think about whether this actually makes sense. If communication with the FRAM can fail, but we have no way to recover, error handling doesn't really make sense. Well, maybe we can increment an error counter or something.

@PatrickKa
Copy link
Contributor Author

Blocked by #237

@PatrickKa
Copy link
Contributor Author

No longer blocked by #237 because error handling for SPI communication works on a higher level now with a supervisor thread. This means that the FRAM code can assume that the SPI functions always work. It only needs to take care of other possible issues that can arise. I am not sure if they exist, though. One needs to go through the code and think about it.

@jeromehue jeromehue self-assigned this Apr 3, 2024
@jeromehue
Copy link
Contributor

After FRAM code inspection, no other possible issues were found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants