Skip to content

Add direct SDO CAN map clear action#19

Merged
jsphuebner merged 1 commit into
jsphuebner:can-backendfrom
Modellfan:jsphuebner-can-backend
Apr 29, 2026
Merged

Add direct SDO CAN map clear action#19
jsphuebner merged 1 commit into
jsphuebner:can-backendfrom
Modellfan:jsphuebner-can-backend

Conversation

@Modellfan

Copy link
Copy Markdown

Summary

Add a dedicated "Remove all mappings" action for libopeninverter CAN mapping that uses a direct SDO write instead of the terminal command transport. Button was anyways dead and copy and paste error from uart backend.

Changes

  • replace the CAN mapping clear button handler with a confirmation flow in the web UI
  • add a frontend CAN-map clear request path that reloads the table after success
  • add /canmap?clear=1 support in the firmware CAN map handler
  • send SDO write 0x5002 / subindex 4 / data 0 and require the matching SDO write reply before reporting success

@jsphuebner

Copy link
Copy Markdown
Owner

Looks good, thanks!
if you want: #20

@jsphuebner
jsphuebner merged commit 14f9974 into jsphuebner:can-backend Apr 29, 2026
1 of 2 checks passed
@Modellfan

Copy link
Copy Markdown
Author

jsphuebner/stm32-sine#58

Also includes the updated libinv to make this function available in stm sine main you also need to merge it

@Modellfan

Copy link
Copy Markdown
Author

This commit crashes the CAN bus update. I froze my sdu with it yesterday. Likely the ongoing CAN Map polling is interrupting the upload. Fix to come. Maybe stage the merge for now.

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