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
Remove dshot_telemetry_start_margin setting to use auto-tune(#13075) #13082
Remove dshot_telemetry_start_margin setting to use auto-tune(#13075) #13082
Conversation
…taflight#13075) * Remove dshot_telemetry_start_margin setting and auto-tune instead * Add USE_DSHOT_BITBAND
original PR to master contained other F4 |
4.4.x does not contain AT32 target support. |
Has this been tested? |
@damosvil , can you test this for 4.4.x lease? i cannot for several days. |
It seems to be working fine on my side |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approving, but it does not count toward merge-ability.
|
||
|
||
uint32_t decode_bb(uint16_t buffer[], uint32_t count, uint32_t mask); | ||
#if defined(STM32F4) || defined(AT32F4) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn’t this use #ifdef USE_DSHOT_BITBAND
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Good catch. Fixed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with eyes-only review, it seems to be equivalent to the original PR.
synchronizing #13075 to 4.4-maintenance