-
Notifications
You must be signed in to change notification settings - Fork 17
OTA: Support WiFi setup and firmware update as one change #666
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
Open
LeeLeahy2
wants to merge
12
commits into
sparkfun:release_candidate
Choose a base branch
from
LeeLeahy2:factory-update
base: release_candidate
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
Remove the chicken and egg issue where WiFi needs to be running to be the highest priority, but has not started yet because it is not enabled! The new routine, networkInterfaceRunning, allows any interface to poll the network and determine if the interface should be running. This routine provides the missing feedback between the network and the interface layers to when the network layer wants to stop the interface and a stop sequence does not exist.
…_DELAY Move startup delay from WIFI_STATION_STATE_STARTING into WIFI_STATION_STATE_RESTART_DELAY
Always display the update button and menu item, remove the requirement check for new firmware qualification. When the settings are written back to to the GNSS device the system is able to establish the connection to the remote access point using the newly specified WIFI SSID and password and then perform the OTA firmware update. Note: Due to the WiFi restart the connection is broken between the browser and the GNSS device. As a result, the browser does not receive the status update message for the progress bar and reset. Prior to this change the user would need to check for new firmware before performing the firmware update: 1. Use a browser to connect to 192.168.4.1 2. Click on the "WiFi Configuration" button 3. Specify the WiFi SSID and Password 4. Click on the "Save Configuration" button 5. Reconnect the browser to 192.168.4.1 6. Click on the "System Configuration" button 7. Click on the "Check for New Firmware" button 8. If new firmware is available, click on the "Update to vXX.YY" button While it is possible to change the WiFi SSID and password and then hit the "Check for New Firmware" button the response gets lost because of the WiFi restart. When the browser is reconnected the "Update to vXX.YY" button is still disabled because the page got reloaded and the browser does not have the new firmware version. As such, the "Check for New Firmware" button needs to be pressed again to get the new firmware version and enable the "Update to vXX.YY" button.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Always display the update button and menu item, remove the requirement check for new firmware qualification. When the settings are written back to to the GNSS device the system is able to establish the connection to the remote access point using the newly specified WIFI SSID and password and then perform the OTA firmware update.
Note: Due to the WiFi restart the connection is broken between the browser and the GNSS device. As a result, the browser does not receive the status update message for the progress bar and reset.
Prior to this change the user would need to check for new firmware before performing the firmware update:
While it is possible to change the WiFi SSID and password and then hit the "Check for New Firmware" button the response gets lost because of the WiFi restart. When the browser is reconnected the "Update to vXX.YY" button is still disabled because the page got reloaded and the browser does not have the new firmware version. As such, the "Check for New Firmware" button needs to be pressed again to get the new firmware version and enable the "Update to vXX.YY" button.