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

AP_HAL_QURT: Add support for the ModalAI IO board to support PWM ESCs #28834

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

katzfey
Copy link
Contributor

@katzfey katzfey commented Dec 9, 2024

Adding in support for the ModalAI IO board. This can be used to interface to PWM ESCs since the VOXL boards have no PWM capability on their own. This code will attempt to detect whether a ModalAI ESC is attached or the ModalAI IO board. Depending on what it detects it will send the appropriate PWM commands.

This has been flight tested with ModalAI ESC to verify that the original functionality has not been impacted.
This has been bench tested with a ModalAI IO board and 4 PWM ESCs. Can arm and spin up motors. RC throttle will control motor spin speed. Still needs an actual flight test but I think the PR is good to go at this time.

@katzfey katzfey force-pushed the pr-qurt-add-io-board-support branch from 35d6782 to be73530 Compare December 10, 2024 19:47
@katzfey
Copy link
Contributor Author

katzfey commented Dec 15, 2024

WIP label can be removed. It's complete, tested, and ready to go.

Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nicely done!
have you checked the feedback from the ESCs still works in ESC mode?

@tridge tridge removed the WIP label Dec 15, 2024
@katzfey
Copy link
Contributor Author

katzfey commented Dec 15, 2024

Yep, in fact there was a bug with ESC feedback before that I just noticed and fixed. It uses the low bit to request feedback. It wasn't being cleared from all 4 ESCs before choosing the one to request and setting that bit. So often multiple requests would be sent at once which would result in no feedback because they share the UART and the feedback packets would collide.

@tridge tridge merged commit 99bfc32 into ArduPilot:master Dec 16, 2024
99 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants