AVR 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
Submitter: | Alexey Reshetnikov <rs_alexey> | ||
Submitted: | Mon 04 Apr 2016 12:15:05 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | None | Privacy: | Public |
Assigned to: | None | Open/Closed: | Open |
Mon 04 Apr 2016 08:59:27 PM UTC, comment #1: |
Joerg Wunsch <joerg_wunsch>![]() |
Mon 04 Apr 2016 12:15:05 PM UTC, original submission:
FT2XX is more stable and drivers for windows is signed.
|
Alexey Reshetnikov <rs_alexey> |
Attached Files
Depends on the following items: None found
Items that depend on this one: None found
Carbon-Copy List
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.
Follows 1 latest change.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2016-04-04 | rs_alexey | Attached File | - | ![]() |
Added avrdude-6.3-diff.tar.gz, #36833 |
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:
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.
header.
non-ISO8859-1, non-UTF-8 characters. ('\x93', '\x94')