On 10/20/2012 12:28 AM, Kevin wrote:
I also found an error in the protocol description, under write block on the ADTPro website its listed as
Host sends: byte stream:
One byte: "E" ($C5)
One byte: $02
Two bytes: requested block number (LSB, MSB)
One byte: EOR of the first four bytes
that tosses up an I/O ERROR, according to CommsThread.java it should be
Host sends: byte stream:
One byte: "E" ($C5)
One byte: $02
Two bytes: requested block number (LSB, MSB)
One byte: EOR of all bytes in block
Drat. It shouldn't include the checksum byte as part of the calculation. I made that same mistake on the read side when I re-implemented a bunch of stuff on the read side to save space this past release - but didn't notice the write was off at the server end. I'll have to look at it and implement some fudge to make it work either way, and make it match the doc - which is what it _should_ be doing. Thanks for catching that.