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 RAM checksum failed #37

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 a RAM 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
102-ERROR: Download failed: -1

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

001-Opening file '../myfile.binary'
002-Downloading file to port ...
009-... bytes sent
003-Verifying RAM
124-ERROR: RAM 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

Tested on v1.0-37.