Skip to content

DAKEFPV support 12S batteries #764

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

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

Conversation

ThanYangFPV
Copy link
Contributor

In order to support 12S batteries, we changed the voltage divider resistor of VBAT from 10K/100K to 10K/150K, and we want to change the target DEFAULT_VOLTAGE_METER_SCALE from the default 110 to 160.

we changed the voltage divider resistor of VBAT from 10K/100K to 10K/150K, and we want to change the target DEFAULT_VOLTAGE_METER_SCALE from the default 110 to 160
@ot0tot
Copy link
Contributor

ot0tot commented Apr 26, 2025

Have you shipped any of these already with the old voltage divider ratio?

Current sensor scale should not be set unless the ESC is built-in (AIO) or the FC is sold as part of a stack.

DSHOT300 does not need to be set in the config.

@ThanYangFPV
Copy link
Contributor Author

ThanYangFPV commented Apr 26, 2025

Have you shipped any of these already with the old voltage divider ratio?

Current sensor scale should not be set unless the ESC is built-in (AIO) or the FC is sold as part of a stack.

DSHOT300 does not need to be set in the config.

The DAKEF405 and DAKEF722 had been dispatched already. The customer agrees to continue using the 8S battery. But, the customer requests that all subsequent shipments support up to 12S. The F435, F722X8, and H743 have not been dispatched yet. The customer also requests that all of them support 12S.

We do AIO and sell it separately as an accessory.

DSHOT300 I had deleted it out of config, no problem.

@ThanYangFPV
Copy link
Contributor Author

@ot0tot

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.

2 participants