firmware update / state indication improvement#513
Merged
ligenxxxx merged 9 commits intohd-zero:mainfrom May 26, 2025
pitts-mo:main
Merged
firmware update / state indication improvement#513ligenxxxx merged 9 commits intohd-zero:mainfrom pitts-mo:main
ligenxxxx merged 9 commits intohd-zero:mainfrom
pitts-mo:main
Conversation
-simplified -notable change: now erase entire mtd before write (prior code only erased initial 65536 of rx mtd)
-disable mtd write success beeps -simplified -notable change: now erase entire mtd before write (prior code only erased initial 65536 of rx mtd)
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
Contributor
Author
|
tested: |
Contributor
Author
|
notable change: now erase entire mtd before write (prior code only erased initial 65536 of rx mtd) |
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
pitts-mo
changed the title
Contributor
Author
|
This resolves failure of gui status bar during mtd flash and disables successful mtd flash beeps during normal gui update introduced by PR #495. I think this is now good enough to consider for merge. Potential items for future enhancement: |
pitts-mo
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
firmware update / state indicator improvements:
-prevent soft bricking VA / RX (very rare)
-add start, success and failure beeps to existing end beep indicator of recovery flash
-add beep indication for failure during gui flash
-silenced beep indications for successful gui flash
-erase entire mtd before write
-fix gui status bar indication
-simplified scripts to improve consistency
-display distinct RX versions (indicate RX firmware issue/mismatch)
You SHOULD never see a mismatch in RX versions. However, It did happen to me at least once.
I had considered displaying both RX versions only when a mismatch is detected. However, it is more accurate and appears to promote user awareness to persistently report the distinct RX versions.