parallaxinc / PropLoader

Parallax Propeller loader supporting both serial and wifi downloads
MIT License
27 stars 13 forks source link

Add logic and message to alert when EEPROM checksum failed #38

Closed PropGit closed 7 years ago

PropGit commented 7 years ago

[This request created in response to Issue #27]

In much the same way that the added "Propeller not found" error message clarifies download failures, we need an EEPROM checksum failure message.

This should apply to both wired and wireless downloads and should appear before the "ERROR: Download failed: -1" message so that instead of this:

001-Opening file '../myfile.binary'
002-Downloading file to port ...
009-... bytes sent
003-Verifying RAM
004-Programming EEPROM
102-ERROR: Download failed: -1

we see this (assuming the error code is 125):

001-Opening file '../myfile.binary'
002-Downloading file to port ...
009-... bytes sent
003-Verifying RAM
004-Programming EEPROM
125-ERROR: EEPROM checksum failed
102-ERROR: Download failed: -1
dbetz commented 7 years ago

I've added this to the fast loader code. It is not yet present in the single-stage loader.

dbetz commented 7 years ago

done

PropGit commented 7 years ago

Verified in v1.0-37. EEPROM Checksum error code is 126.