patchAVR Downloader/UploaDEr - Patches: patch #8967, Change ft2xx library to...

 
 

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

patch #8967: Change ft2xx library to ftdi_syncbb instead of libusb

Submitted by:  Alexey Reshetnikov <rs_alexey>
Submitted on:  Mon Apr 4 12:15:05 2016  
 
Category: NonePriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open

Mon Apr 4 20:59:27 2016, comment #1:

Well, from a licensing point of view, it's somewhat questionable
whether we'd really like to introduce a dependency on a
closed-source library in code licensed under GPL. Such
dependencies are always tolerable if the library qualifies
as a "system library" (otherwise, you could not use GPL
software on a closed-source OS), but ftd2xx would certainly
not qualify as such.

Second, there are a number of platforms AVRDUDE supports
where libftd2xx is not available at all (*BSD, Solaris).

I'd like to see the real benefit of such an implementation
before having that in the code. However, it makes more sense
to discuss that on the mailing list rather than in a patch
tracker. In particular, your submission suggests the
existing implementation were less stable, so we certainly
ought to fix that. (OK, the signed driver for Windows is
accepted as a benefit.)

Finally, some stylistic issues:

  • If at all, the use of ftd2xx must be completely optional,

so people who cannot or don't want to install it don't lose
any functionality. Replacing the existing ft245r.c
implementation is thus not acceptabel.

  • Spaces and braces are welcome. ;-)
  • If you wrote the file, please add your name in the copyright

header.

  • In the comment in ftd2xx_initialize, there are non-ASCII,

non-ISO8859-1, non-UTF-8 characters. ('\x93', '\x94')

Joerg Wunsch <joerg_wunsch>
Project Administrator
Mon Apr 4 12:15:05 2016, original submission:

FT2XX is more stable and drivers for windows is signed.
This patch changes libusb to ft2xx to ftdi_syncbb (FT245R and FT232R).

To compile you need:
1. patch sources
2. execute aclocal
3. autoconf
4. update config.guess to the newest
5. configure && make

Alexey Reshetnikov <rs_alexey>

 

Attached Files
file #36833:  avrdude-6.3-diff.tar.gz added by rs_alexey (7kB - application/gzip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by joerg_wunsch (Posted a comment)
  • -unavailable- added by rs_alexey (Submitted the item)
  •  

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

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follows 1 latest change.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Apr 4 12:15:05 2016rs_alexeyAttached File-=>Added avrdude-6.3-diff.tar.gz, #36833

    Back to the top


    Powered by Savane 3.1-cleanup1