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

Bf merges 20200603 #3098

Merged
merged 19 commits into from
Jun 3, 2020
Merged

Bf merges 20200603 #3098

merged 19 commits into from
Jun 3, 2020

Conversation

hydra
Copy link
Contributor

@hydra hydra commented Jun 3, 2020

No description provided.

mikeller and others added 19 commits May 31, 2020 13:58
Updated comments in 'smartport.c' to reflect the specification.
…ogging starts

The normal "flush" for SD card only queues a cache sector for writing and the actual sync to the media happens asynchronously. During this period the cache entry is not available until the write completes sometime later. So as the blackbox header fields were written they end up consuming the majority of the cache. A "flush" was made before actual logging starts, but the async writes were not completing fast enough to ensure available cache sectors for the actual logging. This resulted in the cache getting overwritten and corrupting the header.

Changed to wait until the sector cache completes writing to the media before starting the actual logging. This ensures that the logging has ample cache sectors.

Changes only affect SD card logging.
Blackbox flush SD card sector cache after writing header and before logging starts
…tm32f745

Moved custom defaults to dedicated block for STM32F745.
Added Makefile target to dump compiler versions used to build tests.
Moved calculation of available boxes into 'activateConfig()' to avoid a reboot.
…sing

Fixed parsing of over length custom default headers.
@hydra hydra merged commit 2663768 into cleanflight:master Jun 3, 2020
@hydra hydra deleted the bf-merges-20200603 branch June 3, 2020 20:09
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.

None yet

3 participants