Skip to content

[4.5.2] Fix missing ACCGYRO custom alignment defines (#14092) #14178

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

Merged
merged 3 commits into from
Jan 18, 2025

Conversation

haslinghuis
Copy link
Member

Copy link

Do you want to test this code? You can flash it directly from Betaflight Configurator:

  • Simply put #14178 (this pull request number) in the Select commit field of the Configurator firmware flasher tab (you need to Enable expert mode, Show release candidates and Development).

WARNING: It may be unstable. Use only for testing!

blckmn
blckmn previously approved these changes Jan 17, 2025
Copy link
Member

@blckmn blckmn left a comment

Choose a reason for hiding this comment

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

Not sure this is entirely needed - but approving.

@haslinghuis
Copy link
Member Author

Only four configs using custom alignment option

configs/NEUTRONRCF411SX1280/config.h:114:#define GYRO_1_ALIGN ALIGN_CUSTOM
configs/SKYSTARSF405AIO/config.h:110:#define GYRO_1_ALIGN ALIGN_CUSTOM
configs/SPRACINGH7EXTREME/config.h:247:#define GYRO_2_ALIGN ALIGN_CUSTOM
configs/GEPRCF411SX1280/config.h:120:#define GYRO_1_ALIGN ALIGN_CUSTOM

ledvinap
ledvinap previously approved these changes Jan 18, 2025
Co-authored-by: Petr Ledvina <ledvinap@gmail.com>
@haslinghuis haslinghuis dismissed stale reviews from ledvinap and blckmn via 78c40b1 January 18, 2025 11:23
@haslinghuis haslinghuis merged commit 7e55f9e into betaflight:4.5-maintenance Jan 18, 2025
23 checks passed
@haslinghuis haslinghuis deleted the 452-custom-align branch January 18, 2025 20:03
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