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 platform configuration for CF2.1 brushless #1383

Merged
merged 4 commits into from
Jun 10, 2024
Merged

Conversation

gemenerik
Copy link
Member

  • Platform comes with modified defaults for PID tuning, idle thrust, platform mass, and battery limit.
  • The brushless defconfig (make cf21bl_defconfig) enables manual arming and the DSHOT ESC protocol.
  • Moved idle thrust configuration out of kbuild.
  • Forces manual arming for the brushless in kbuild.

gemenerik added 3 commits June 4, 2024 13:53
- Platform comes with modified defaults for PID tuning, idle thrust, platform mass, and battery limit compared to CF2.1.
- The brushless defconfig enables manual arming and the DSHOT ESC protocol.
- Moved idle thrust configuration out of kbuild.
This automatically forces manual arming if users manually select the brushless as the platform in kbuild instead of by loading the brushless defconfig.
Without this, users could compile a flyable firmware and accidentally forget to enable manual arming.
People that want autoarming on the brushless platform will have to modify the kbuild system.
@gemenerik gemenerik requested a review from tobbeanton June 4, 2024 12:54
Kconfig Outdated Show resolved Hide resolved
src/platform/interface/platform_defaults_cf21bl.h Outdated Show resolved Hide resolved
src/modules/src/Kconfig Show resolved Hide resolved
@tobbeanton tobbeanton merged commit b15eebc into master Jun 10, 2024
24 checks passed
@tobbeanton tobbeanton deleted the rik/blplatform branch June 10, 2024 13:34
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