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 DAKEFPVF405X8 #612

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

Conversation

ThanYangFPV
Copy link

Schematics and configuration files have been emailed to [email protected].

Please review. Thank you.

Copy link
Member

@haslinghuis haslinghuis left a comment

Choose a reason for hiding this comment

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

Have not checked timers and DMA

configs/DAKEFPVF405X8/config.h Outdated Show resolved Hide resolved
configs/DAKEFPVF405X8/config.h Show resolved Hide resolved
Comment on lines +71 to +72
#define LED0_PIN PC14
#define LED1_PIN PC15
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
#define LED0_PIN PC14
#define LED1_PIN PC15
#define LED0_PIN PC14
#define LED1_PIN PC15

These pins cannot be used to drive LEDs

@ot0tot
Copy link
Contributor

ot0tot commented Nov 26, 2024

I think we have decided the F405 cannot reliably support 8 motor outputs. Recommend H743 for X8, or reduce the number of motor outputs to use F405.

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.

3 participants