Fix indentation

Add Tomasz Wasilczyk to AUTHORS

Don't set EEPROM serial when it's not used.

This makes a difference in USB enumeration -- if serial is missing,
its string is not present in string descriptor list.

Remove legacy EEPROM fields from FT4232H.

These are not present when flashed with FT_Prog tool.

Don't fail reading missing strings.

This happens i.e. when serial number is not set -- in such case the
string is missing from string descriptor list.

ftdi_eeprom_initdefaults const arguments

This function don't need to modify its arguments.

Don't spam stderr when flashing FT232H eeprom

examples/bitbang_cbus.c: Check return value of fgets()

Fix building unit tests without FTDIPP

Needed to run the baudrate unit tests.

Probably another fallout from:

****************************
commit 0209a3633dc877a577af07d883eb5059e22f6a91

cmake: do not check for g++ when FTDIPP is disabled
****************************

Fix GCC fall-through warnings.

When compiling ftdi.c in the context of blackmagic debug, more enabled
warnings and -Werror made me to add appended fixes.

Fixing the signedness of baudrate would probably be more logically by
making all baudrate and clk unsigned, but would need more code changes.
I decided to make clk signed as a fix with minimal
impact. The FTDI chips are far away from 2 GHz clock rate...

Ignore purge_test example build for MinGW

termios.h is not available on MinGW.

CMakeLists.txt: fix paths when FTDIPP is set

Use the same project name (i.e. libftdi1 and not libftdipp1) when FTDIPP
is enabled as suggested by Aurelien Jarno in
http://developer.intra2net.com/mailarchive/html/libftdi/2020/msg00044.html

Without this change, the libftdi1.pc config file defines the include
path as /usr/local/include/libftdipp1 while the ftdi.h file is actually
installed in /usr/local/include/libftdi1

This is an issue for example for libsigrok which will fail on:

In file included from src/hardware/asix-sigma/protocol.c:27:
src/hardware/asix-sigma/protocol.h:28:10: fatal error: ftdi.h: No such file or directory
   28 | #include <ftdi.h>
      |          ^~~~~~~~

Fixes:
 - http://autobuild.buildroot.org/results/1427f44e36752c337791597fab47a1889552a2fe

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Release libftdi 1.5 final

Merge branch 'prepare-release'

Set version number to 1.5rc1

Prepare changelog for 1.5rc1

Increase copyright year to 2020

Update AUTHORS file

Update release checklist

Remove outdated rpm .spec file template

The distro source packages are much more useful than this.

Grepped source tree for ".spec" and "rpm",
hopefully I caught all occurrences.

Update Autotools Mythbuster link to avoid multiple redirects.

While I'm hoping to maintain the chain of redirects for the foreseeable
future, it makes sense to point at the final URL now that Autotools
Mythuster has had its domain for a long while.

Signed-off-by: Diego Elio Pettenò <flameeyes@flameeyes.com>

ftdi_eeprom_build(): Fix inverted handling of VCP driver field for TYPE_R chips

From doc/EEPROM-structure:

    On TYPE_R 00.3 set means D2XX, on other devices VCP

-> invert the field for TYPE_R so "eeprom->channel_a_driver"
   set to 1 always means VCP driver for all chips.

eeprom handling: Unify handling of boolean bit flags

Boolean bit flags should be zero or one in memory,
not the actual eeprom value FTDI assigned to them.

Example of the code change in ftdi_eeprom_build():

ftdi_eeprom_build():
-            if (eeprom->suspend_dbus7 == SUSPEND_DBUS7_BIT)
+            if (eeprom->suspend_dbus7)

The changes should not break any backward compatibility
in ftdi_build_eeprom().

The function ftdi_decode_eeprom() will return slightly different values
for boolean fields, but callers should test using "if (flag_enabled)"
instead of "if (flag_enabled == specific_value)" anyway.

Test done: Decoded eeprom of a FT232 serial cable stayed the same.
$ diff -u dump-old.txt dump-new.txt
-> no change

ftdi_eeprom: Add device release number support

In process of firmware repairing of some FTDI-chip based device I have
found that I can't get binary identical EEPROM dumps from properly
working device and ftdi_eeprom --build_eeprom command because of
ftdi_eeprom does not support changing of device release number (byte
with addr 0x06).

[TJ: Keep old behavior if release_number is not set in the config file]

Ignore patch files

ftdi_eeprom: Minor cosmetic fix on usage screen

ftdi_eeprom: Add more detailed example.conf file

eeprom: Fix handling of high_current_drive parameter

Additionally fix it up for type R chips.

eeprom: Add channel_a_driver support for type xxR chips

Make ftdi_eeprom more verbose by adding verbose option

[TJ: Replaced tabs with spaces]

Ignore tags file

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

cmake: do not check for g++ when FTDIPP is disabled

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
[Retrieved (and slightly updated) from:
https://git.buildroot.net/buildroot/tree/package/libftdi1/0003-cmake-do-not-check-for-g-when-FTDIPP-is-disabled.patch]
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Fix typos

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

libftdi: pass eeprom strings as const

These strings are not modified and it doesn't make sense that they ever
would be. Setting these as const ensures that they will not be modified
and allows const strings to be passed in without special manipulation.

Specifically a call like the following failed to compile:
ftdi_eeprom_set_strings(&ftdi, "foo", "bar", "abc123");

The error will look something like this:
error: ISO C++ forbids converting a string constant to ‘char*’
[-Werror=write-strings]

CMake: reorder Python detection

PythonInterp should be detected before PythonLibs otherwise
CMake exits with the following message:

Could NOT find PythonLibs (missing: PYTHON_LIBRARIES PYTHON_INCLUDE_DIRS)

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Fix build dependency if swig wrapper and documentation is both switched on

Seems to affect new cmake / swig versions only.

Reported by Richard Shaw <hobbes1069@gmail.com> during Fedora Rawhide build:

Ok, so far looks like I was right, you have to set the dependency on
ftdi1_doc.i before calling swig_add_library or it doesn't get it right:

[ 11%] Swig compile ftdi1.i for python
/usr/bin/doxygen /builddir/build/BUILD/libftdi-f49ee0d/build-py2/Doxyfile
cd /builddir/build/BUILD/libftdi-f49ee0d/build-py2/python && /usr/bin/cmake
-E make_directory /builddir/build/BUILD/libftdi-f49ee0d/build-py2/python
/builddir/build/BUILD/libftdi-f49ee0d/build-py2/python/CMakeFiles/_ftdi1.dir
/usr/bin/doxygen
/builddir/build/BUILD/libftdi-f49ee0d/build-py2/Doxyfile.xml
cd /builddir/build/BUILD/libftdi-f49ee0d/build-py2/python && /usr/bin/cmake
-E touch
/builddir/build/BUILD/libftdi-f49ee0d/build-py2/python/CMakeFiles/_ftdi1.dir/ftdi1PYTHON.stamp
cd /builddir/build/BUILD/libftdi-f49ee0d/build-py2/python && /usr/bin/cmake
-E env SWIG_LIB=/usr/share/swig/3.0.12 /usr/bin/swig -python -DDOXYGEN=YES
-outdir /builddir/build/BUILD/libftdi-f49ee0d/build-py2/python
-I/builddir/build/BUILD/libftdi-f49ee0d/src -I/usr/include/libusb-1.0
-I/usr/include/python2.7 -o
/builddir/build/BUILD/libftdi-f49ee0d/build-py2/python/CMakeFiles/_ftdi1.dir/ftdi1PYTHON_wrap.c
/builddir/build/BUILD/libftdi-f49ee0d/python/ftdi1.i
/builddir/build/BUILD/libftdi-f49ee0d/python/ftdi1.i:7: Error: Unable to
find 'ftdi1_doc.i'
make[2]: *** [python/CMakeFiles/ftdi1_swig_compilation.dir/build.make:67:
python/CMakeFiles/_ftdi1.dir/ftdi1PYTHON.stamp] Error 1

Simplify CMake policy guard

Signed-off-by: FeRD (Frank Dana) <ferdnyc@gmail.com>

CMake/Swig updates for python wrapper

libftdi was FTBFS[1] in Fedora Rawhide after a GCC 9.0.X mass rebuild but
the problem turned out to be behavior changes[2] in CMake with regards to
SWIG[3].

The UseSWIG module in cmake 3.13+ will now use standard target names and not
generate the SWIG_MODULE_<target>_REAL_NAME variable.

Replace the deprecated swig_add_module with swig_add_library.

[Refinements from Intra2net:
- Use swig_add_library for cmake 3.8+ only
- Add cmake version guard for the new CMP0078 policy
- Use "NOT CMAKE_VERSION VERSION_LESS" construct to
  support cmake versions below 3.7.0]

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1675275
[2] https://cmake.org/cmake/help/latest/policy/CMP0078.html?highlight=cmp0078
[3] https://cmake.org/cmake/help/latest/module/UseSWIG.html

Add SPDX identifiers to the core library and ftdi_eeprom tool

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

swig: improve indentation

Fixes -Wmisleading-indentation warnings.

ftdi_eeprom: Add support for group0 drive levels on x232H chips

swig: python: fix unresolved symbol issue

GCC later than 5.x produce _fdti1.so file with an undefined
symbol str2charp_size due to C99 inline semantics change. So
remove this keyword.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

python/CMakeLists.txt: fix build with cmake < 3.7

VERSION_GREATER_EQUAL has been added only in cmake 3.7:
https://cmake.org/cmake/help/v3.7/release/3.7.html

So replace this statement by NOT CMAKE_VERSION VERSION_LESS

Fixes:
 - http://autobuild.buildroot.org/results/1091872e2b77d789e361d1ddefd235c738933c55

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Remove more example programs during rpm build

Add program to test flush (purge) functionality.

Improve Doxygen output for #defines in header files.

Implement tc[io]flush methods & deprecate broken purge_buffers methods.

The definitions for the SIO_RESET_PURGE_RX or SIO_RESET_PURGE_TX values are
with respect to the FTDI chip, not the CPU.  That is, when the FTDI chip
receives a USB control transfer request with the command SIO_RESET_PURGE_RX,
the FTDI chip empties the FIFO containing data received from the CPU awaiting
transfer out the serial port to the connected serial device (e.g., a modem).
Likewise, upon reception of the SIO_RESET_PURGE_TX command, the FTDI chip
empties the FIFO of data received from the attached serial device destined to
be transmitted to the CPU.

Unfortunately the coding of the previous releases of libfti assumed these
commands had the opposite effect. This resulted in the function
ftdi_usb_purge_tx_buffer clearing data received from the attached serial
device.  Similarly, the function ftdi_usb_purge_rx_buffer cleared the FTDI
FIFO containing data to be transmitted to the attached serial device.  More
seriously, this latter function clear the libftid's internal buffer of data
received from the serial device, destined to the application program.

This patch adds the ftdi_tciflush, ftdi_tcoflush, and ftdi_tcioflush functions
which emulate the Linux serial port tcflush(3) function.

Use a separate Py_ssize_t var instead of casting an int pointer as a Py_ssize_t pointer.

Casting is fine if Py_ssize_t == int, but not when Py_ssize_t == long.

Signed-off-by: Jordan Rupprecht <rupprecht@google.com>

CMake: use find_package (SWIG) for cmake >= 3.0.0

There's a workaround for a bug (fixed in cmake 3.0.0) that does not
work in CMake 3.12.  Only use the workaround with cmake < 3.0.0.

Signed-off-by: Eneas U de Queiroz <cote2004-github@yahoo.com>

Improve new async read/write example

examples/async.c : Short example using async read/write

Bring back ftdi_eeprom by default

CMake: move options to a dedicated file

Also disable all options having extra dependencies.

If an option is selected makes its dependencies as REQUIRED.

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Add Robby McKilliam to AUTHORS

Add kernel driver auto attach/detach

Introduce new detach mode:
AUTO_DETACH_REATACH_SIO_MODULE

Tested on linux (Ubuntu 16.04 and Raspbian Stretch) with an FT230.

Access ftdi->max_packet_size after checking ftdi context != NULL

Reported via email by Mr. Alex Tvema.

Start ChangeLog for next version

Add Pawel Jewstafjew to AUTHORS

fix support for XON/XOFF flow control

Fix ftdi_stream.c compilation on WIN32

Fix small memleak when ftdi_init() fails

If libusb open fails (e.g. permission problem),
then ftdi_init() leaks the memory for the eeprom data.

Increase version to 1.4 final

follow-up on c++ open() fix. fixes logic in open() with long argument list.

Sync README with ChangeLog

Increase version to 1.4rc2

C++ API: Increase .so library version

We have an ABI break in this release (get_strings_and_reopen() changes),
so programs using the C++ wrapper need to be recompiled.

Add Eric Schott to AUTHORS, update ChangeLog

C++ API: Correct the purge Direction and ModemCtl enumerations definitions

Do not use value of zero for bitmasks. The new enumeration values
are defined so programs compiled with the old ftdi.hpp include
would work as previously (doing nothing when the enumeration
with the zero value was bit-or'ed into the argument).

Correct const-ness of write to agree with ftdi.h

Add Matthias Janke to AUTHORS, update ChangeLog

C++ API: Add support to open devices with missing product / serial strings

The C++ API and C API differ in how they open a device. The C API only
considers the user supplied strings iVendor, iProduct and iSerial to open a device.

The C++ API behaves differently. After opening the device with specific supplied strings,
it tries to read all strings via get_strings_and_reopen().
This fails if one string like iSerial is not defined
in the descriptor. As such the whole open() operation fails
and the C++ API would only open devices with complete descriptors.

This commit fixes this behaviour, by extending the get_strings_and_reopen()
and get_strings() argument lists, to configure the strings to read.

Changelog: Rename ftdi_usb_open_bus_port() to ftdi_usb_open_bus_addr()

Change ftdi_open_bus_port to ftdi_open_bus_addr, since libusb_get_port_number is not always unique.

Minor comment fix.

Add missing sys/time.h include

When compiling libftdi with mingw toolchain it complains about
implicit declaration of gettimeofday().

Signed-off-by: Yegor Yefremov <yegorslists@googlemail.com>

Add ftdi_usb_open_bus_port to changelog, update AUTHORS

Add a ftdi_usb_open_bus_port() function

[Thomas Jarosch] from the libftdi mailinglist:

This patch adds a function that will open using the specified bus and
port. I don't know how useful it would be to others, but I decided to
submit it cause why not. I pretty much just copied
"ftdi_usb_open_desc_index" and modified the parameters and loop.

Update Copyright to 2017, switch download URLs to HTTPS

ChangeLog for 1.4rc1

Update AUTHORS and release checklist

Set version number to 1.4rc1

ftdi_eeprom_set_strings(): No need for an open device

Add Adam Malinowski to AUTHORS

Add python (SWIG) support for ftdi_eeprom_get_strings()

Add doxygen documentation for ftdi_eeprom_get_strings()

Mostly borrowed from ftdi_usb_get_strings()

Add new function ftdi_eeprom_get_strings()

This function adds possibility of reading EEPROM strings:
manufacturer, product and serial number.

cbus.py: remove unnecessary semicolons

Update build instructions with yum references

eeprom: Fix USE_SERIAL handling for 230X type chips

Found that the USE_SERIAL was not being set correctly in
the eeprom_build routine for the 230X. Tracked the problem down to a line
that hard coded the value when the 230X was first added. With this line
removed code currently in the code before the switch will set the bit
correctly for the 230X.

Allow LINK_PYTHON_LIBRARY=OFF on Darwin

Add TYPE_230X to baudrate test code

Use BM/R series baud rate computation for FT230X.

The current code uses the AM series baud rate computation for FT230X,
and this results in incorrect divisor calculation for high baud rates.
Use the BM/R series logic instead.

This matches the corresponding code paths in the Linux kernel ftdi_sio
driver.  See ftdi_sio.c:1323 here:

http://lxr.free-electrons.com/source/drivers/usb/serial/ftdi_sio.c#L1323

Specifically attempts to set 2000000 on an FT230X fail currently due to
the if-statement at ftdi.c:1106.  The computed divisor (12) gets clamped
to 16, and thus the for-loop it's in doesn't converge on a usable divisor.

The if-statement at ftdi.c:1152 never fires, as the if-statement at
ftdi.c:1106 in the for-loop precludes a divisor of 12 (which is required
for *encoded_divisor to become 0x4001).

Add Jon Beniston to AUTHORS

Const correctness for char* strings

This changes the API for the ftdi_get_error_string(),
but the fix is really simple.

From the mailinglist:

The attached patch fixes some warnings/errors when compiled with g++ and
another compiler:

- Strings should be const char *
- Malloc needs to be cast from void * to char *
- Avoid mixing declarations with code

Build on _WIN32: Don't include sys/time.h

The attached patch fixes compilation with Visual Studio:

sys/time.h isn't available on Windows. timeval is defined within
windows includes.

Compiles OK for Cygwin/Mingw too.

Fix cbus.py example

FT230X related enums are called CBUSX and not CBUSH.

Make ftdi_read_eeprom_location() endianess independent

Read 16-bit integer as two byte array and combine these two bytes
to unsigned short.

ftdi_eeprom: Fix flashing FT245R

When we were trying to program FT245R, they were always flashed as FT232R.
The attached patch fixes the missing call to type2bit for TYPE_R.

Prepare final 1.3 release

ftdi_eeprom: Improve usability of flash_raw mode