bugAVR Downloader/UploaDEr - Bugs: bug #38951, AVR109 use byte offset instead of...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #38951: AVR109 use byte offset instead of word offset

Submitter:  Fred <magister>
Submitted:  Thu 09 May 2013 03:36:01 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Privacy:  Public
Assigned to:  joerg_wunsch Originator Name:  Magister
Open/Closed:  Closed Release:  None
Programmer hardware:  Device type: 

Thu 09 May 2013 06:05:41 PM UTC, comment #2: 

What's impressive is that this bug is here since the beginning of the project, but what the code was doing was to send a "A 00 00" then send in a while() all the data by after. Auto-incrementing doing its job.

It's only since SVN1007 that the bug surfaced.

Fred <magister>
Thu 09 May 2013 05:38:11 PM UTC, comment #1: 

I made the patch #8045 to fix it

Fred <magister>
Thu 09 May 2013 03:36:01 PM UTC, original submission:  

The 'A' command must send the address in word offset, it is currently sent in byte offset.

For instance with a 128 bytes buffer, avrdude send:

A 00 00
A 00 80
A 01 00
A 01 80
etc

each command followed by 128 bytes of data.

This is wrong, the correct offset is:

A 00 00
A 00 40
A 00 80
A 00 C0
etc

for flash data you have to >>1 the address before sending it with 'A' command.

Fred <magister>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by joerg_wunsch (Updated the item)
  • -email is unavailable- added by magister (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-09-03 joerg_wunsch StatusNone Fixed
        Assigned toNone joerg_wunsch
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code