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

HAL_ChibiOS: switch to minimum scheduling delta of 10us #18416

Merged
merged 1 commit into from
Aug 23, 2021

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Aug 22, 2021

this avoids an issue with the ChibiOS 20.3 virtual timer implementation that can result in a scheduling slip equal to the
system timer period.

Andy has been suggesting this change for a while. I resisted it as I thought it would impact on soft-serial parsing, but it turns out it
doesn't.

Fixes issue #18383

Thanks to help from Bob and Giovanni from the ChibiOS community

this avoids an issue with the ChibiOS 20.3 virtual timer
implementation that can result in a scheduling slip equal to the
system timer period.

Andy has been suggesting this change for a while. I resisted it as I
thought it would impact on soft-serial parsing, but it turns out it
doesn't.

Fixes issue ArduPilot#18383
Copy link
Contributor

@peterbarker peterbarker left a comment

Choose a reason for hiding this comment

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

GyroFFT was the only thing I spotted as perhaps an issue.

@rmackay9 rmackay9 added this to pending in Copter 4.1 Aug 28, 2021
@rmackay9 rmackay9 moved this from pending to 4.1.0-beta8 in Copter 4.1 Aug 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Copter 4.1
4.1.0-beta8
Plane 4.1
Awaiting triage
Rover 4.1
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

5 participants