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

Feature/fix esp upload #144

Merged
merged 4 commits into from
Jul 6, 2018
Merged

Feature/fix esp upload #144

merged 4 commits into from
Jul 6, 2018

Conversation

sarfata
Copy link
Owner

@sarfata sarfata commented Jul 6, 2018

  • 2018 07 06 - v1.3.3
    • Fix a bug which forced us to use program-esp to update the wifi module.
      It is now possible again to just use the following commands to update KBox:

         platform run -e host -t upload
         platformio run -e esp -t upload
      
    • Change default esp upload speed to 921600 because 2000000 does not seem well
      supported on Windows.

    • Changed the 'end of programming' detection method to more reliably detect when
      we are done programming and reboot KBox.

    • Tested the official ESP uploader on Windows and OS X. Comment out the line
      tools/platformio_cfg_esp.py in platformio.ini to use it.
      It will be a little bit slower but might work better for some people.

sarfata added 4 commits July 6, 2018 12:37
The watchdog would cause the host to reboot before upload was done.
Fixed by refreshing the watchdog timer while the upload is in progress.
 - We will detect a timeout by only looking at bytes coming from the
 computer to properly detect when the computer is done programming.
 - Tested the official ESP uploader and made sure it works too with
 speed of 921600.
 - Changed default speed to 921600 because 2000000 does not work on
 Windows
@coveralls
Copy link

coveralls commented Jul 6, 2018

Coverage Status

Coverage increased (+0.5%) to 72.414% when pulling 6455096 on feature/fix-esp-upload into 415cc8d on master.

@sarfata sarfata merged commit 4eb8af3 into master Jul 6, 2018
@sarfata sarfata deleted the feature/fix-esp-upload branch July 6, 2018 22:47
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.

2 participants