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

ACAN_ESP32: Add ACAN_ESP32::end() to stop the controller and remove ISR #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kholk
Copy link

@kholk kholk commented Apr 19, 2024

Some applications require stopping the controller (at least temporarily) for example when starting the OTA process as, during that, getting any interrupt from the CAN controller may be not desirable, as that could be increasing the likeliness of an OTA failure (and noone wants to end up with a corrupt flash).

The OTA case is just one of the possible examples, anyway.

This was tested on ESP32-S3R8 and S3R2.

Some applications require stopping the controller (at least temporarily)
for example when starting the OTA process as, during that, getting any
interrupt from the CAN controller may be not desirable, as that could be
increasing the likeliness of an OTA failure (and noone wants to end up
with a corrupt flash).

The OTA case is just one of the possible examples, anyway.

Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants