Build: Make compiler warnings configurable

Use the SR_ARG_ENABLE_WARNINGS macro to configure and check for
the availability of compiler warning flags. Maintain separate
sets of warning flags for C and C++.

The configure option --enable-warnings=[min|max|fatal|no] can
be used to set the compiler warning level for all languages.
The default level is "max".

Build: Move custom Autoconf macros to separate file

Place custom Autoconf macros which other sigrok modules may
re-use into a separate file m4/sigrok.m4. Also, introduce new
macros for defining the package and library versions, and for
gathering compiler warning flags.

Clean up .gitignore

Build: Place Autoconf macros into m4/ directory

This separates the M4 macros from the other cruft in autostuff/
to prepare for the introduction of custom macro files.

Build: Do not omit first argument to m4_warn()

It seems that contrary to what the documentation says, leaving
the category argument to m4_warn() empty is not allowed. Use
the "unsupported" category for lack of a better choice.

FreeBSD: Also check for swig3.0.

On FreeBSD (for example) the SWIG binary can also be called "swig3.0".

This fixes (together with the recent "swig2.0" addition) bug #557.

Build: Get rid of recursive make invocations

Do not invoke $(MAKE) just to execute some clean rule.
This gets rid of the "jobserver unavailable" warnings.

Build: Tell setup.py where to find SWIG

FreeBSD: Also check for swig2.0 in addition to swig.

On FreeBSD (for example) the SWIG binary is called "swig2.0".

This fixes bug #557.

FreeBSD: Fix compile/link error due to missing libusb_get_version().

The FreeBSD libusb-1.0 API implementation doesn't have libusb_get_version().
Use our CONF_LIBUSB_1_0_VERSION macro instead.

Build: Avoid $< in explicit rules

Although useful, makes other than GNU make do not like this.
Name the prerequisite explicitly instead, and circumvent any
VPATH substitution other makes may do.

Build: Skip C++ CFLAGS and LIBS check when disabled

It is not a good idea to invoke PKG_CHECK_MODULES with an empty
list of modules to check, so skip the invocation in that case.

Build: Fix typo that broke the driver enable options

Build: Make SR_DRIVER() handle dependencies

Turn the hardware driver dependency handling on its head, by
making SR_DRIVER() take a list of dependencies to check for.

Build: Make dependency on git change more robust

It can sometimes happen that .git/HEAD or .git/refs/head/*, which
are added as config.status dependencies during configure, do not
exist anymore at build time.  For instance, when the current branch
is deleted after switching to a different one.

Wrap the dependencies inside $(wildcard ...) to avoid this problem.
Note that this is a GNU make feature.  However, it should be fine
as it is only used for git builds.  Even if a non-GNU make is used,
the construct will hopefully just expand to nothing.

Build: Show glib/libzip version requirements in configure summary.

Build: Fix a typo causing libftdi1 detection issues.

Build: Properly namespace the version defines

Use the CONF_ prefix for the configured host define as well as
the various build-time version macros.

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.