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

Adjust GYRO_LOCK_COUNT #11186

Closed
wants to merge 1 commit into from
Closed

Conversation

SteveCEvans
Copy link
Member

The skew between gyro interrupt and gyro loop is accumulated over GYRO_LOCK_COUNT cycles and with the previous value, on an F411 with bit-banged DSHOT enabled, the effective gain created by this resulted in a failure to lock. Tested on F411, F405, F722 and H743

@SteveCEvans SteveCEvans added this to the 4.3 milestone Dec 28, 2021
@haslinghuis
Copy link
Member

Good find!

@SteveCEvans
Copy link
Member Author

Do not merge as will be included in #11198

@haslinghuis haslinghuis added this to For discussion in Finalizing Firmware 4.3 Release via automation Jan 4, 2022
@haslinghuis haslinghuis moved this from For discussion to Steve's PR in Mashup in Finalizing Firmware 4.3 Release Jan 4, 2022
@haslinghuis
Copy link
Member

Merged with #11198

@haslinghuis haslinghuis closed this Jan 7, 2022
Finalizing Firmware 4.3 Release automation moved this from Steve's PR in Mashup to Finished (Merged) Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

5 participants