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

Submitted by:  Hermann Kraus <herm>
Submitted on:  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:  Hannes Weisbach <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>
Project MemberIn charge of this item.
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>

 

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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    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.3