Included desription of ArduinoOTA support and other corrections listed #1023
Conversation
This file contains 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
Contributor
krzychb
commented
Nov 15, 2015
krzychb
commented
- Include ArduinoOTA as the way to go - http://www.esp8266.com/viewtopic.php?p=33231#p33231 and Added desription of Arduino IDE OTA process to documentation #984 (comment)
- Need to update boards.txt to have OTA for NodeMCU 1.0 in stable or staging platform package
- Address possible confusion regarding DNS_SD_Arduino_OTA.ino - OTA Update example sketch missing #1001 and Added desription of Arduino IDE OTA process to documentation #984 (comment)
1. Include ArduinoOTA as the way to go - http://www.esp8266.com/viewtopic.php?p=33231#p33231 2. Need to update boards.txt to have OTA for NodeMCU 1.0 in stable or staging platform package 3. Address possible confusion regarding DNS_SD_Arduino_OTA.ino - #1001 and #984 (comment) Next day cleaning review Update ota_updates.md Update ota_updates.md Update ota_updates.md Update ota_updates.md
|
|
||
|
|
||
| ## Basic Requirements | ||
|
|
||
| - Flash chip size is 2x the size of the sketch. | ||
|
|
||
| Two separate, same size memory locations are used alternately. While existing sketch is executed from one location, the new sketch is saved to the other. After reset the new sketch is executed directly from where saved. Location of old sketch will be used for next OTA upload. |
There was a problem hiding this comment.
the sketch is copied over the old one at startup.
take a look here:
https://github.com/esp8266/Arduino/blob/master/cores/esp8266/Updater.cpp#L151-L156
https://github.com/esp8266/Arduino/tree/master/bootloaders/eboot
|
Thank you for review Links2004 |
Links2004
added a commit
to Links2004/Arduino
that referenced
this pull request
Nov 15, 2015
|
I will merge it in |
Merged
|
i close this pull request, will merge #1024 with you change as soon the Travis has finished the build checks. |
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.