bugAVR Downloader/UploaDEr - Bugs: bug #38831, [PATCH] Fix avrftdi onchip-buffer...

 
 

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

bug #38831: [PATCH] Fix avrftdi onchip-buffer overrun

Submitter:  Hermann Kraus <herm>
Submitted:  Fri 26 Apr 2013 01:03:20 AM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Ready For Test Privacy:  Public
Assigned to:  hweisbach Originator Name:  Herm
Open/Closed:  Open Release:  None
Programmer hardware:  Device type: 

Tue 30 Apr 2013 01:07:47 PM UTC, comment #2: 

Look like it's working with FT2232D again (tested with Linux), but the support for the FT2232D is now disabled in code. In "avrftdi.c:747-751" between Svn-Version 1141 and 1142 the #if was removed and the FT2232D chip-type is 1 (TYPE_BM).

This chip is working for me with a pin_limit of 7 and package size of 384. Please re-enable it.

Tom <hrbn2>
Sat 27 Apr 2013 10:00:05 PM UTC, comment #1: 

SVN r1142 contains a patch similar to the one submitted by Hermann Kraus. Additionally it takes the chips buffer size into account, so that newer H-type chips do not experience a performance hit.
I tested this with {OS X 10.6.8, Ubuntu 12.04.2 and Win7 x64} x {FT2232C, FT4232H}.

Hannes Weisbach <hweisbach>
Group Member
Fri 26 Apr 2013 01:03:20 AM UTC, original submission:  

The FT2232 only has a 384 byte fifo, but avrftdi tries to send arbitrarily large packets. Reading flash memory with page_size > 380 (example: mega16 has 512) fails.
The attached patch fixes this problem. It also makes sure the error code is checked.

Hermann Kraus <herm>

 

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

Attached Files
file #27942:  fifo_overrun.patch added by herm (4KiB - 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 hrbn2 (Posted a comment)
  • -email is unavailable- added by hweisbach (Posted a comment)
  • -email is unavailable- added by herm (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-04-27 hweisbach StatusNone Ready For Test
        Assigned toNone hweisbach
    2013-04-26 herm Attached File- Added fifo_overrun.patch, #27942

    Back to the top

    Powered by Savane 3.13-cf05.
    Corresponding source code