Skip to content

Refactoring settings to enable USB control of runtime-settings#403

Merged
jordens merged 13 commits intomainfrom
feature/usb-settings
Jun 20, 2024
Merged

Refactoring settings to enable USB control of runtime-settings#403
jordens merged 13 commits intomainfrom
feature/usb-settings

Conversation

@ryan-summers
Copy link
Copy Markdown
Member

@ryan-summers ryan-summers commented Jun 18, 2024

This PR fixes #380 by allowing RF channel settings to be configured via the USB port.

TODO:

  • Currently, serial-settings will always say that you need to reboot to apply settings. This isn't true for runtime settings.
  • Save RF channel transforms into the RF channel EEPROM, not device flash
  • Make clear reset the RF channel configurations

@ryan-summers ryan-summers requested a review from jordens June 18, 2024 13:41
@ryan-summers ryan-summers marked this pull request as ready for review June 18, 2024 13:41
Comment thread src/hardware/setup.rs Outdated
Comment thread src/settings/eeprom/main_board.rs Outdated
Comment thread src/settings/flash.rs Outdated
Comment thread src/settings/mod.rs Outdated
Comment thread src/settings/mod.rs Outdated
Comment thread src/settings/runtime_settings.rs
Comment thread src/settings/mod.rs Outdated
Comment thread src/settings/eeprom/mod.rs
Comment thread CHANGELOG.md Outdated
Comment thread src/settings/eeprom/main_board.rs Outdated
Comment thread src/settings/mod.rs Outdated
@ryan-summers ryan-summers requested a review from jordens June 19, 2024 17:01
@ryan-summers
Copy link
Copy Markdown
Member Author

@jordens This should now be good to go.

Comment thread CHANGELOG.md Outdated
Comment thread src/settings/mod.rs Outdated
@ryan-summers ryan-summers requested a review from jordens June 20, 2024 11:34
@jordens jordens added this pull request to the merge queue Jun 20, 2024
Merged via the queue into main with commit 82e608f Jun 20, 2024
@jordens jordens deleted the feature/usb-settings branch June 20, 2024 11:56
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.

Enable runtime settings modification via USB

2 participants