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 flash code #184

Closed
Tracked by #103
PatrickKa opened this issue Dec 14, 2023 · 3 comments · Fixed by #274
Closed
Tracked by #103

Add error handling for flash code #184

PatrickKa opened this issue Dec 14, 2023 · 3 comments · Fixed by #274

Comments

@PatrickKa
Copy link
Contributor

Description

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

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

PatrickKa commented Dec 15, 2023

Think about whether this actually makes sense. If communication with the flash 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 flash 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.

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

Successfully merging a pull request may close this issue.

1 participant