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

Convert CO_driver error handling to error_cb based implementation #27

Open
heliochronix opened this issue Aug 13, 2020 · 0 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@heliochronix
Copy link
Member

Implement the CO_CANerr_cb() method of setting and clearing CAN error status flags in CANopenNode. This method will replace the cyclic call of CO_CANmodule_process() in the CO NMT processing thread (aka the main thread) which basically checks the status of CAN all the time manually.

The challenge is we need to either ensure the callback triggers when errors are cleared, or have some other way of indicating that errors have cleared so we can unset the CO error flags. I'm not sure if the interrupt for CAN errors triggers on any change of state or only when an error occurs. If it's the latter, then the error callback will not trigger when things clear, and so the CANopen code will always think we are in an error state.

@heliochronix heliochronix added the enhancement New feature or request label Aug 13, 2020
@heliochronix heliochronix self-assigned this Aug 13, 2020
@wrh2 wrh2 added the C3 label Apr 22, 2022
@ryanpdx ryanpdx removed the C3 label Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Backlog
Development

No branches or pull requests

3 participants