-
Notifications
You must be signed in to change notification settings - Fork 14.6k
[uavcan] Param for interface mask of ESC actuators #24922
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
Conversation
🔎 FLASH Analysispx4_fmu-v5x [Total VM Diff: 224 byte (0.01 %)]
px4_fmu-v6x [Total VM Diff: 216 byte (0.01 %)]
Updated: 2025-05-28T11:56:35 |
3e83a3a
to
38220e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙏
bf89dc8
to
6864ff5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
6864ff5
to
b3fc587
Compare
Maybe 1 or 2 is a better default? Otherwise someone might set this up, see that it's working, and move on without realizing they've just degraded their other CAN bus. |
Solved Problem
When using UAVCAN ESCs on a hexacopter we were having issues with getting other nodes to talk back to the Skynode.
The ESCs were already on their own separate CAN bus, however, by default UAVCAN broadcast the ESC messages on all interfaces, starving the other nodes by message priority of bus bandwidth.
Solution
This adds a parameter that allows to configure the interface mask for the ESC actuators, which completely fixes the problem.
The default mask (0xff) is backward compatible with the default UAVCAN behavior.
Changelog Entry
For release notes:
Alternatives
No.
Test coverage
Tested in hardware on a hexacopter with 6 ESCs and a smart UAVCAN battery.
cc @cmic0