patchAVR Downloader/UploaDEr - Patches: patch #8176, butterfly.c (AVR109 protocol...

 
 

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

patch #8176: butterfly.c (AVR109 protocol implementation) clean-up and bug-fixing

Submitter:  Enoch <wexi>
Submitted:  Sun 08 Sep 2013 09:08:22 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  In Progress Privacy:  Public
Assigned to:  joerg_wunsch Open/Closed:  Open

Fri 13 Sep 2013 07:40:22 PM UTC, comment #1: 

Bug fixes #1 and #3 are (in my opinion) already present.

I adopted your fix for bug #2.

I'm not yet convinced about the cleanups (as partially already
mentioned).

Joerg Wunsch <joerg_wunsch>
Group administrator
Sun 08 Sep 2013 09:08:22 PM UTC, original submission:  

While writing my own AVR109 compliant (SRAM) buffered bootloader I needed the attached butterfly.c changes.

Three bug fixes:

1. Word flash addresses (already fixed in master).

2. Absent butterfly_page_erase()
Seg fault when trying to program the eeprom only.

3. Terminal mode uses butterfly_read_byte_flash() to dump flash content. Original got byte order wrong.

Clean-up:

1. Skips address setting if the bootloader auto increments the address.

2. Allows multi-page load/write (if just the calling avr.c would take advantage of!).

For your perusal. Thanks, Enoch.

Enoch <wexi>

 

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

Attached Files
file #29087:  patch added by wexi (9KiB - application/octet-stream)

 

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 (Posted a comment)
  • -email is unavailable- added by wexi (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-13 joerg_wunsch StatusNone In Progress
        Assigned toNone joerg_wunsch
    2013-09-08 wexi Attached File- Added patch, #29087

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code