Build: Cleanup up and modernize autotools setup

Note that some --enable options of configure have become
--with or --without options.

Build: Simplify the hardware driver selection

Replace DRIVER() and DRIVER2() by a single SR_DRIVER() macro.
Derive the names of shell variables and preprocessor defines
programatically to cut down on repetition.

Add dist tarball to gitignore

setup.py: Do VPATH search for swig/enums.i

Also, in swig/classes.i include swig/enums.i rather than plain
enums.i, to guard against future name clashes.

Build: Depend on swig/enums.i where appropriate

enums.py: Place swig/enums.i into build directory

Build: Do not distribute generated headers

version.h and enums.hpp are generated at build time and should
not be included in the distribution tarball.  This seems to have
triggered a weird error when doing a VPATH build of the dist
tarball.

Build: Pass nostdinc option to automake

It seems automake automatically adds the directory containing
the generated version.h to the include path.  Use nostdinc to
disable default includes altogether.

Build: Use angle brackets for enums.cpp include

Build: Distribute bindings sources

A couple of files referenced during the build were missing from
the distribution, causing a tarball build failure.

Build: Prefix directory to include of enums.hpp

This fixes a build problem due to the reduced include search paths
introduced by my recent changes.  Also fix a couple of other
includes to use angle brackets.

Build: Append git revision hash to version

Append the git revision hash to the libsigrok package version,
unless HEAD exactly matches a release tag.  Note that this does
not affect the version known to autoconf -- e.g. source tarballs
created by make dist will not receive a revision suffix.

Changes to git HEAD automatically trigger a reconfiguration.
Uncommitted changes do not, which is why I left out the -dirty
suffix.

Consistently use SR_PACKAGE_VERSION instead of VERSION

Build: Delay expansion of $datadir in FIRMWARE_DIR

Make it so that $(datadir) is resolved at make time, as per
autotools recommendations.  Note that $datadir is not fully
resolved at configure time to begin with, i.e. part of it
already was evaluated at make time.

Build: Make version.h a configuration header

Use the proper tool for the job and make libsigrok/version.h
a secondary configuration header, so that autoconf's AC_DEFINE
machinery can be used to generate it.  Note that the header
template is still hand-written, enabling fine control of the
content.

tests: Include <libsigrok/libsigrok.h>

This matches the global style and avoids the "../" path
components.

Build: Set local include directories in Makefile.am

Move the include flags for files in the source tree from
configure.ac to Makefile.am where they belong.  Also use
AM_CPPFLAGS instead of CFLAGS/CXXFLAGS to make sure the
files in the build/source tree are always picked up first.

Also, remove the include/libsigrok sub-directory from the
search path, thereby making the <libsigrok/> prefix mandatory
when building libsigrok itself.  This matches the convention
already imposed on users of the library.

Introduce OutputFlag

output/srzip: Prevent memory leak in case filename is empty

backend: Output lib versions, host, and endianness in sr_init().

This should help to more easily debug issues reported by users.

brymen-bm86x: add some error checking

Various minor cosmetics and consistency fixes.

Drop comment mentioning non-existing function.

Doxygen consistency fixes (@foo instead of \foo).

Various errno/strerror() related fixes.

 - Don't #include <errno.h> in files that don't actually need it.

 - Don't use strerror() on error codes from functions that don't set
   errno. Replace strerror() with sr_strerror() for libsigrok functions.

libsigrok: demo: close pipes on dev_acquisition_stop()

Fixes a bug where new acquisition failes due to leftover pipes from
previous acquisitions:
sr: demo: dev_acquisition_start: pipe() failed
Indeed, PulseView had 2024 pipes opened. With this fix, it stabilizes at
33 with sampling thread active.

Signed-off-by: Hubert CHAUMETTE <redacted>

output/srzip: Use sr_output->filename instead of option

This fixes parts of bug #570.

Add filename field to sr_output and make it accessible

This fixes parts of bug #570.

Only build BayLibre ACME driver if <sys/timerfd.h> is available.

This fixes bug #610 (Android build issue).

ols: Don't #include <libserialport.h> directly.

The driver is not using sp_* calls directly, so no need to #include
the header file.

Bump version to 0.4.0 (the upcoming next major release).

Append "-git" to the version string.

This avoids confusion with the released tarballs.

Convert a few more occurrences of d->priv to d->context.

Fix #442 by renaming sr_dev_driver.priv to .context

fx2lafw: Use the USB_INTERFACE #define.

fx2lafw: Fix segfault wrt libusb_detach_kernel_driver().

Move the libusb_detach_kernel_driver() call after the code that
sets the usb->devhdl pointer, otherwise it'll be NULL and result
in a segfault.

  #0  libusb_kernel_driver_active (dev=0x0, interface_number=0) at libusb/core.c:1711
  #1  dev_open (sdi=0x12d99f0) at src/hardware/fx2lafw/api.c:374
  [...]

Tested on a device with the default Cypress VID/PID and one with
the Saleae Logic VID/PID, both works fine.

fx2lafw: Fix #445 by detaching the kernel driver (if any)

This avoids the need to run "rmmod usbtest" on Linux for devices
with the standard Cypress FX2 USB VID/PID (04b4:8613).

Demo: Fix #314 by always honoring sample limit changes

Fix #505 by providing a separate property list for the analog group

Fix #574 by setting up the transfer first, then starting acquisition

baylibre-acme: Remove unnecessary close().

The timerfd descriptor is closed automatically by
g_io_channel_shutdown(). No need to close it manually.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Use SR_HZ_TO_NS() when configuring the timerfd.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Add a workaround for slow data acquisition.

At high sampling rates and maximum channels we are not able to acquire
samples fast enough, even though frontends still think that samples
arrive on time. This causes visible shifts in frontend plots.

To compensate for the delay introduce the following workaround: check
if we are late (if any clock events have been missed) and resend the
last frame n times (n == number of missed clock events).

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Use timerfd instead of a fake pipe.

Currently baylibre-acme uses a fake pipe as the input channel required by
libsigrok API and calls sleep() in the data acquisition callback to create
intervals between measurements.

Switch to a more elegant approach: use Linux' timerfd and set a periodic
timer equal to the sampling rate. Then read the data every time the timer
expires.

Signed-off-by: Daniel Lezcano <redacted>
Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Close sysfs files after stopping the acquisition.

Add a missing call to dev_acquisition_close() in dev_acquisition_stop().

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Optimize reading of values from sysfs.

Opening a file has a cost (security, allocation, syscalls). The
read_sample() function always does an open/read/close sequence.

In order to optimize that, let's open the file at the moment the
acquisition starts, close it when the acquisition stops and make
read_sample() only lseek() to the beginning of the file and read
the value.

Signed-off-by: Daniel Lezcano <redacted>
Signed-off-by: Bartosz Golaszewski <redacted>

sysclk-lwla: Do not create channels in reverse order

This was originally done as an optimization in combination with a list
reversal which has since been removed from the code.  Thus, un-reverse
the channels so that the UI lists them in the correct order again.

yokogawa-dlm: Minor cosmetics, consistency fixes, typos.

yokogawa-dlm: Return correct value for SR_CONF_HDIV

yokogawa-dlm: Remove dlm_setup_channels()

yokogawa-dlm: Introduce config_channel_set()

yokogawa-dlm: Config get/set/list handler updates

yokogawa-dlm: Fix number of digital groups

yokogawa-dlm: Prevent duplicate channel index

yokogawa-dlm: Add support for the DL9000 series

fx2lafw: Add DSLogic Pro and DSCope firmware loading support

dslogic: Fix incorrect samplerate setting.

bindings/python: Handle import failures without crashing

When the import of gi.repository.GLib failed, we would get a NULL
pointer that we passed along without any checking. In this situation,
the entire program would crash with a segmentation fault, and no
message to indicate the problem.

When the import fails, abort the SWIG init and print a message. The
Python interpreter then prints out a backtrace, which can be useful
in tracking down the problem.

scpi-pps: Chroma 62000P: Get capabilities from device name

The Chroma 62000P series comes in various models with different
current and voltage capabilities. These are encoded in the *IDN
string, so just get them from there, rather than needing a profile
for every model.

scpi-pps: Add profile for Chroma 62024P-80-60 DC Source

Java: install files into DESTDIR

This fixes bug #537.

saleae-logic16: Emit debug info of which bitstream is used.

saleae-logic16: Support new bitstream version 1.3 with renumbered registers

scpi: Update names of a few *_OUTPUT_* items.

modbus: Consistently use the "Modbus" spelling.

(as per modbus.org spelling)

modbus: Minor typos, cosmetics and consistency fixes.

Actual implementation for the maynuo-m97 driver.

maynuo-m97: Initial driver skeleton.

libsigrok.h: add new type of device: ELECTRONIC_LOAD.

libsigrok.h: Add SR_CONF_OVER_TEMPERATURE_PROTECTION_ACTIVE.

modbus: add a serial RTU backend.

Add a modbus communication helper module.

libsigrok-internal.h: add helper macro to read floats from unaligend memory.

Rename SR_CONF_OUTPUT_* to SR_CONF_*.

All those options are currently applied only to power-supplies
but they could apply as well to electronic loads, except for the
fact that electronic loads channels are called inputs and not
outputs.
Also when you think about an SMU (or any kind of 4-quadrants
power-supply), their channels can both source and sink current,
so they can be considered as input as much as output.
Those SR_CONF_* are thus renamed so that they can be used in all
those situations.

scpi-pps: don't fail acquisition start for devices with no CMD_SELECT_CHANNEL

For devices such as the HP 6632B the following invocation was failing due to
scpi_cmd(sdi, SCPI_CMD_SELECT_CHANNEL, ...) returning SR_OK_CONTINUE.

./sigrok-cli -d scpi-pps:conn=/dev/ttyUSB0:serialcomm=9600/8n1 --continuous
sr: session: sr_session_start: could not start an acquisition (not enough data to decide error status yet)
Failed to start session.

Use G_PI instead of defining our own version of PI.

scpi-pps: Minor cleanups.

scpi-pps: Fix a compiler warning.

    CC       src/hardware/scpi-pps/profiles.lo
  src/hardware/scpi-pps/profiles.c:67:2: warning: missing initializer
  for field 'frequency' of 'const struct channel_spec'
  [-Wmissing-field-initializers]
    { "1", { 0, 60, 0.0001 }, { 0, 25, 0.1 } },
    ^
  In file included from src/hardware/scpi-pps/profiles.c:24:0:
  src/hardware/scpi-pps/protocol.h:106:8: note: 'frequency' declared
  here
    float frequency[3];
          ^

scpi-pps: Make some more items static.

spci-pps: Add profile for Agilent N5767A DC source

spci-pps/profiles: Support frequency control in Chroma 61604

scpi-pps: Add infrastructure for controlling output frequency

This patch only adds the needed infrastructure to control output
frequency in the same manner as output voltage or current limit. This
does require a new field in the channel_spec struct, for the sake of
symmetry.

global: Add configuration key for output frequency target

This will be used to set up the output frequency of AC sources.

global: Treat SR_CONF_OUTPUT_FREQUENCY as float instead of uint64_t

This makes 'output_frequency' symmetrical with 'output_current' and
'output_voltage'. On a more fundamental level, there's no reason why
frequency should be treated as a discrete quantity, other than
"es51919 used it this way".

asix-sigma: Avoid use of variable length arrays

This was only done once in sigma_write_register().

python: classes.i: Declare strings with std::string

This makes it consistent with the rest of the string declarations in
this file, all of which use std::string.

spci-pps: Add profile for Chroma 61604 AC Source

Only the capabilities which map directly to SCPI commands supported by
sigrok are implemented at this time. This is sufficient to control
the most often used functionality of this AC source

scpi: Accept *IDN responses with more than four tokens

Some devices with more than one microcontroller report the firmware
version for each of them, giving us more than four tokens. When that
happens, sigrok aborts, even though it received a valid response.

This happens, for example with the Chroma 61604:
'Chroma ATE,61604,001060,1.25,1.34,1.20'

Compile with -std=c11 along with _POSIX_C_SOURCE=200112L.

With gcc 5.1 released and defaulting to std=gnu11, the code will be compiled
according to different standards depending on the compiler version so we
should better specify explicitly what standard we are targetting.
C11 is now quite mature, it is supported in the just release Debian stable
(gcc 4.9) and also in old-stable (gcc 4.7), so there should be no reason to
use anything more ancient.
We also should have no reason to need any non-standard GNU extension.
So using only C11 + POSIX sounds like the best option right now.

Define our own constant for PI.

M_PI is not defined in the C standard nor in POSIX, it is a
non-standard GNU extension.

strcasecmp() is defined in strings.h.

wav: Stricter check for valid chunk ID.

isascii() is a superset of isalpha() and isblank() so the current
code doesn't really make sense.
Moreover, isascii(x) is just a funky and non standard way to
write x < 128.

C++: Change arguments of Input::send() from std::string to data+length.

Lower libserialport requirement to 0.1.1.

Fix various warnings when building without libusb.

  ../src/backend.c: In function 'sr_init':
  ../src/backend.c:435:1: warning: label 'done' defined but not used [-Wunused-label]
   done:
   ^

  ../src/device.c: In function 'sr_dev_inst_connid_get':
  ../src/device.c:525:7: warning: unused variable 'connection_id' [-Wunused-variable]
    char connection_id[64];
         ^
  ../src/device.c:524:20: warning: unused variable 'b' [-Wunused-variable]
    int r, cnt, i, a, b;
                      ^
  ../src/device.c:524:17: warning: unused variable 'a' [-Wunused-variable]
    int r, cnt, i, a, b;
                   ^
  ../src/device.c:524:14: warning: unused variable 'i' [-Wunused-variable]
    int r, cnt, i, a, b;
                ^
  ../src/device.c:524:9: warning: unused variable 'cnt' [-Wunused-variable]
    int r, cnt, i, a, b;
           ^
  ../src/device.c:524:6: warning: unused variable 'r' [-Wunused-variable]
    int r, cnt, i, a, b;
        ^
  ../src/device.c:523:22: warning: unused variable 'drvc' [-Wunused-variable]
    struct drv_context *drvc;
                      ^

ols: Whitespace fixes.

.gitignore: Add tests/main.

baylibre-acme: Add a missing return value check.

Check the return value of sr_gpio_setval_export() in bl_acme_set_power_off()
and return an appropriate error if the call fails.

Signed-off-by: Bartosz Golaszewski <redacted>

Revert "baylibre-acme: Fix a compiler warning."

This reverts commit 4cd97e5ad7bb63cb90d209506464fafd9f9eef8a.

We should actually check the return value of sr_gpio_setval_export().

Signed-off-by: Bartosz Golaszewski <redacted>

windows: Fix various compiler warnings.

Add LIBUSB_CALL where needed to avoid warnings such as the following:

  In file included from src/hardware/hantek-dso/api.c:34:0:
  src/hardware/hantek-dso/dso.h:212:13:
  note: expected 'libusb_transfer_cb_fn' but argument is of type 'void (*)(struct libusb_transfer *)'
   SR_PRIV int dso_get_channeldata(const struct sr_dev_inst *sdi,
               ^