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 Spektreworks boom board #26646

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

WickedShell
Copy link
Contributor

First commit just fixes an accidental breakage that was inflicted upon the CAN mirroring code, no target in autotest has this enabled so it went unnoticed.

Second commit adds the boom board hwdef we are using, generates PWM's and talks to APD ESC's (also useful, because I never committed an example of what you needed in the hwdef to enable the APD support), uses relays to control battery charging and heating (controlled from the main vehicle over CAN), a battery monitor.

The aircraft has 2 booms in it, a left and a right. I've managed this in the past with 2 different build targets, but it ends up being a support nightmare where the wrong params/firmware are loaded. The clever work around this uses is the boom boards are identical except there is a pull up/down resistor populated based upon which board it is, a script is used to read this value and populate the appropriate defaults for which side of the aircraft it is installed upon.

@WickedShell WickedShell force-pushed the wickedshell/boom-cleanup branch 2 times, most recently from d79f952 to 8f187b7 Compare March 29, 2024 23:46
@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Mar 31, 2024
@Hwurzburg
Copy link
Collaborator

Hwurzburg commented Mar 31, 2024

if you want this in the wiki at all there needs to be a readme.md with links,photos, and setup info or provide the wiki changes/additions in a wiki PR

@WickedShell WickedShell force-pushed the wickedshell/boom-cleanup branch from 8f187b7 to 0b4f08d Compare April 3, 2024 18:10
@peterbarker peterbarker merged commit 9489e6b into ArduPilot:master Apr 4, 2024
47 checks passed
@peterbarker
Copy link
Contributor

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hwdef WikiNeeded needs wiki update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants