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 SPEEDYBEEF405WING target #608

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Nov 14, 2024

No description provided.

configs/SPEEDYBEEF405WING/config.h Outdated Show resolved Hide resolved
@haslinghuis
Copy link
Member

@ot0tot anything needed to add this ?

@ot0tot
Copy link
Contributor Author

ot0tot commented Dec 24, 2024

@ot0tot anything needed to add this ?

The major issue is the EXT1 pin is not connected, so there is no way to sync the gyro with the PID loop. I have a FC, but have not been able to test the target yet.
If you have a Wing FC, please feel free to test and provide feedback. If any of the devs have a way to validate the PID loop rate performance, that would be great also.

@SteveCEvans
Copy link
Member

The lack of the interrupt isn't a major issue. The consequence is increased jitter and thus poorer filter performance. Also DMA transfers can’t be initiated ahead of the gyro task being called, so there’s a performance hit. We can live without it, but it would certainly be better with.

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.

4 participants