Simplify a few config_set() callbacks.

Also, extended logging and random whitespace fixes.

Constify a few arrays and variables.

Remove unneeded explicit array size specification.

Various minor whitespace fixes.

tondaj-sl-814: Add missing SR_PRIV.

serial-dmm: Use g_malloc()/g_free().

vc870: Fix a compiler warning.

  src/dmm/vc870.c: In function 'flags_valid':
  src/dmm/vc870.c:380:54: warning: unused parameter 'info'
  [-Wunused-parameter]
   static gboolean flags_valid(const struct vc870_info *info)

Minor cosmetics, cleanups.

serial-dmm: Drop obsolete extern declaration.

serial-dmm: Declare each meter type in only one place.

Pass driver struct pointer to driver callbacks.

This lays the groundwork for subdrivers to share callbacks without
needing a separate wrapper function for each subdriver.

Construct driver array at runtime, from an array of per-file arrays.

This lays the groundwork for drivers to define their own array of
subdrivers, rather than having to list each subdriver here.

rigol-ds: Add DS1000Z series support.

Tested on an MSO1104Z with firmware 00.04.02.SP4.

The analog channels are captured correctly. For the MSO series, with digital
channels, there are two outstanding issues:

1. Logic data is retrieved per-channel, one byte per sample, with the value
   in the LSB of each byte. The current datafeed logic format doesn't allow
   this format to be passed on directly. I suggest we resolve that rather than
   making the driver buffer and interleave the data.

   As stands, the code will retrieve data for all channels and pass it onto
   the datafeed with unitsize=1. Channel D0 can used correctly if selected
   alone. For other channels, data is passed to the frontend but the API does
   not provide a way to associate it with the correct channel.

2. Channels CH3 and CH4 are multiplexed with D0-7 and D8-15 respectively, so
   enabling these is mutually exclusive. We don't currently have a way to
   express this constraint to the frontend.

Add initial Voltcraft VC-870 support.

There are a few details that have yet to be implemented or
reverse engineered and tested.

libsigrok.h: Add SR_MQ_POWER_FACTOR and SR_MQ_APPARENT_POWER.

baylibre-acme: Fix a double free in bl_acme_set_shunt().

Signed-off-by: Bartosz Golaszewski <redacted>

scpi-pps: Add profile for Rigol DP821A.

scpi-pps: Initialise sdi with status SR_ST_INACTIVE.

Without this change a segfault occurs at exit after scan, because
dev_close() is called and the device is believed to be open.

rigol-ds: Fix broken channel group check in config_list().

rigol-ds: Data source is a device option, not per channel group.

rigol-ds: Use sr_scpi_get_bool().

rigol-ds: Fix double free.

std_dev_clear() frees all channel groups, so this one is not required.

rigol-ds: Fix wrong channel group malloc size.

Change API of channel accessor functions to take struct sr_channel *.

Add sdi pointer to struct sr_channel.

Simplify channel creation.

We always follow sr_channel_new() with a call to add the channel to the sdi.
Tidy up a bit by adding this functionality to sr_channel_new() instead.

fx2lafw: Replace obsoleted strncmp() calls.

The fixed lengths for strncmp() can no longer be used since strings of
various lengths can be passed to match_manuf_prod(). Use strcmp() instead.

Various fixes/updates to make the driver compile.

This patchset was originally done by eightdot <redacted> by
manually forward-porting parts of the changes done by Bert Vermeulen (see
previous commits), but then heavily modified by Uwe Hermann to be based on
top off the (git-)rebased patches from Bert Vermeulen instead.

Note: This initial DSLogic code is *not* yet in a working or usable
state. It should be considered as a basis for further work only, for now.

fx2lafw: Basic acquisition support for DSLogic.

Add udev rule for DSLogic.

fx2lafw: scan/firmware support for DSLogic.

output: Accept analog packets in csv output module.

Signed-off-by: Bartosz Golaszewski <redacted>

ols: Fix a compiler warning (unused variable).

    CC       src/hardware/openbench-logic-sniffer/api.lo
  ../src/hardware/openbench-logic-sniffer/api.c: In function 'scan':
  ../src/hardware/openbench-logic-sniffer/api.c:103:10: warning: unused
  variable 'probefd' [-Wunused-variable]
    GPollFD probefd;
            ^

ols: Fix detection and acquisition on Windows.

Use the more portable sp_input_waiting() instead of g_poll() with FDs.
Thanks to Martin Ling for the hints. This is tested on Linux and Win7
using an OLS; scanning for the device and starting an acquisition works.

Also, add some more debug output.

This fixes bug #562.

baylibre-acme: correctly handle channel group options

Split device options into general and channel group settings, and
adjust config_list() callback appropriately.

Signed-off-by: Bartosz Golaszewski <redacted>

Rename SR_CONF_NUM_TIMEBASE to SR_CONF_NUM_HDIV.

ut372: Minor cosmetics.

ut372: Support count mode.

Add SR_MQ_COUNT for event count measurements.

ut372: Handle flags correctly.

Packets will now be rejected if the device is not displaying RPM.

ut372: Break out character-pair decoding to a separate function.

ut372: Implement initial protocol parser.

For now this only works correctly if the device is in the default
state showing current RPM. The flags are not checked.

ut372: Initial sub-driver skeleton.

serial-dmm: Add MASTECH MS8250B as a supported DMM.

It is an alias and uses the FS9721 driver.

configure.ac: Two more fixes for the baylibre-acme detection.

asix-sigma: Use the more portable g_usleep().

configure.ac: Fix baylibre-acme OS check.

baylibre-acme: don't report ACME as detected if no probes are present

Signed-off-by: Bartosz Golaszewski <redacted>

Doxyfile: Make exclude patterns more specific.

Current exclude patterns lead to unwanted exclusion of all paths
containing common directory names like output, bindings etc. even
if those names occur higher in the directory structure.

Make exclude patterns more specific by prefixing them with src/.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Fix vendor/device name.

Various Doxygen updates.

unit tests: Drop unneeded check_ filename prefix.

baylibre-acme: Drop unneeded comment.

There's indeed no g_fclose() unfortunately. The g_*() wrappers for file
handling are mainly there to deal with portability issues in file names
(encoding, character sets, etc) on different platforms.

baylibre-acme: Fix a compiler warning.

Use PRIu64 to avoid the following compiler warning:

    CC       src/hardware/baylibre-acme/gpio.lo
  protocol.c: In function 'bl_acme_set_shunt':
  protocol.c:341:2: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type 'uint64_t' [-Wformat=]
    g_fprintf(fd, "%llu\n", MOHM_TO_UOHM(shunt));
    ^

baylibre-acme: Minor coding-style, cosmetics.

baylibre-acme: Add support for SR_CONF_POWER_OFF.

Allow to remotely cut the power at ACME probe level.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Add Linux-specific GPIO helpers.

These functions allow to export, read and set GPIOs using Linux
sysfs interface.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Add support for probe factor setting.

Implement support for SR_CONF_PROBE_FACTOR setting in BayLibre ACME
driver. Given the channel-group parameter this allows to set the
shunt resistance for each probe.

Signed-off-by: Bartosz Golaszewski <redacted>

SR_CONF_PROBE_FACTOR: New option.

Add new configuration option allowing to modify the probe factor
for oscilloscopes and power-monitors.

Signed-off-by: Bartosz Golaszewski <redacted>

configure.ac: Compile baylibre-acme driver for Linux only.

The driver for BayLibre ACME depends on Linux-specific sysfs
interfaces to ina226 and tmp435 devices. Exclude it for different
targets.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Driver implementation.

Implement basic functionalities for baylibre-acme. Add support
for common config options, device detection and sample reading.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Initial driver skeleton.

Signed-off-by: Bartosz Golaszewski <redacted>

rigol-ds: fix the smallest supported vdiv for the DS2000 series.

rigol-ds: return the actual hardware num_vdiv and vdiv list.

rigol-ds: fix search for the closest vdiv.

We try to find the smallest diff by comparing each diff with
the previously known smallest diff, so initially, this smallest diff
should be INFINITY so that we are sure to find a smaller one.

This fixes the following exception:
sr: rigol-ds: Negative vdiv index: -1.
Caught exception: not applicable

rigol-ds: Add missing "break" statements.

rigol-ds: Add missing 20/50/100V vdiv entries.

These are available on e.g. Rigol DS1102E (or "upgraded" DS1052E).

Without this, if one of the channels happens to have been set to
one of the missing vdiv settings frontends (e.g. PulseView) will
have some trouble using the scope:

  sr: hwdriver: sr_config_get(): key 30012 (vdiv) sdi 0x11bcb70 cg CH1
  sr: rigol-ds: Negative vdiv index: -1.
  std::exception

rigol-ds: Add more debug output.

rigol-ds: handle full word trigger slope in config_get().

Some scope can return POSITIVE/NEGATIVE instead of POS/NEG,
so accept this as well.

This closes bug #558.

rigol-ds: SR_CONF_TRIGGER_SLOPE is actually listable.

input/vcd: fix parse_header() return value check.

Mixing tests for both a boolean and an SR_ERR at the same time is not
really a good idea.
parse_header() actually returns a boolean so only check if it returns FALSE.

This fixes the following gcc-5 warning:

src/input/vcd.c: In function 'receive':
src/input/vcd.c:506:34: warning: logical not is only applied to the left hand side of comparison [-Wlogical-not-parentheses]
   if (!parse_header(in, in->buf) != SR_OK)
                                  ^

Remove the inline qualification from sr_rational_set().

Inlining can only happen in the same compilation unit where the
function was defined, so there is no sense declaring an inline
function in a header if this function is not defined in this
same header.

This fixes the following gcc-5 warning:

In file included from include/libsigrok/libsigrok.h:1066:0,
                 from src/version.c:21:
include/libsigrok/proto.h:36:20: warning: inline function 'sr_rational_set' declared but never defined
 SR_API inline void sr_rational_set(struct sr_rational *r, uint64_t p, uint64_t q);
                    ^

Correctly copy sr_datafeed_meta in sr_packet_copy().

Commit 5801d558 replaced g_slist_copy_deep() by some incorrect code
that actually leaks the newly allocated memory, instead of doing
a deep copy.

This new version should be more correct, more concise, and it fixes
the following warning:

src/session.c: In function 'sr_packet_copy':
src/session.c:1025:38: warning: passing argument 2 of 'g_slist_foreach' from incompatible pointer type [-Wincompatible-pointer-types]
   g_slist_foreach(meta_copy->config, (GCopyFunc)copy_src, NULL);
                                         ^
In file included from /usr/include/glib-2.0/glib/gmain.h:26:0,
                 from /usr/include/glib-2.0/glib/giochannel.h:33,
                 from /usr/include/glib-2.0/glib.h:54,
                 from src/session.c:24:
/usr/include/glib-2.0/glib/gslist.h:125:10: note: expected 'GFunc {aka void (*)(void *, void *)}' but argument is of type 'void * (*)(const void *, void *)'
void     g_slist_foreach                 (GSList           *list,
         ^

sr_strerror_name(): Add missing SR_ERR_IO entry.

SR_ERR_IO: new error code

Add new error code which can be used to notify the user about
general input/output errors.

Signed-off-by: Bartosz Golaszewski <redacted>

configure.ac: Add AC_CANONICAL_SYSTEM macro.

In order to determine the target OS when cross-compiling libsigrok
we need autotools to set the 'target_os' variable. This macro
determines the system type and sets output variables to the names
of the canonical system types.

Signed-off-by: Bartosz Golaszewski <redacted>

backend: Add basic transform module sanity checks.

transform: Add a few basic unit tests.

transform: Add an "invert" transform module.

This inverts the data values:

 - An analog value of x becomes 1/x.

 - A digital value of 0 becomes 1 (and vice versa).

transform: Add a "scale" transform module.

transform: Add a "nop" transform module.

This doesn't do anything, just passes input packets on unmodified.

transform: Hook up transforms.

transform: Add a basic set of API calls.

transform: Add struct sr_transform and struct sr_transform_module.

Fix problem building with BSD make

This fixes bug #556:

Bug 556 - libsigrok fails to build with BSD Make

Makefile.am: Add NEED_SERIAL condition.

Makefile.am: Unconditionally build src/lcr/es51919.c.

We do the same with DMM parsers currently, and this change also
fixes an issue in sigrok-util's 'new-driver' script.

This fixes bug #545.

Update list of files ignored by git

configure: Avoid bashism breaking the C++ bindings

sysclk-lwla: Widen constant to 64 bit before shifting

(lwla_convert_trigger): Fix trigger mask computation bug introduced
by recent change:  Widen constant to 64 bit before shifting so that
channel nunmbers beyond 32 are processed correctly.

Lower dependency to glib 2.32.

By avoiding g_slist_copy_deep() for now, we can easily allow libsigrok
to build against glib 2.32 (less hassle for users of stable/older
distros or OSes).

output/gnuplot: Use .dat as suggested file extension.

Gnuplot doesn't have any "official" file name extension(s). It uses
(at least) two different types of files basically:

 - "control files": These can have many different somewhat commonly
   used extensions such as .gpi, .gnu, .gnuplot, .gp, .plt, .gih,
   others. These files don't contain data, only Gnuplot commands such
   as 'set yrange [75:105]', 'set ylabel "foo" offset 1', and so on.

 - "data files": This is what libsigrok reads and writes. These files
   contain actual data to be graphed by Gnuplot (with the help of a
   specially-crafted control file, see above). The data is usually in
   a tab-separated format. The common file extension is usually .dat,
   though many others are possible as well.

in/out: Minor consistency renames.

 - 'struct sr_input *' variables are consistently named 'in'.
 - 'struct sr_input_module *' variables are consistently named 'imod'.

 - 'struct sr_output *' variables are consistently named 'o'.
 - 'struct sr_output_module *' variables are consistently named 'omod'.

output/wav: Improved description

output: Added preferred file extension field

This fixes parts of bug #541.

input: Added preferred file extension field

This fixes parts of bug #541.

cxx: Fix a linking issue.

Fix "undefined reference to `sigrok::EnumValue<sigrok::LogLevel,
sr_loglevel>::_values'", which happens at least when using clang(++),
e.g. on Linux, Mac OS X, or FreeBSD.

This fixes bug #534.

Thanks to Uffe Jakobsen and Martin Ling for reporting and investigating!

Fix FreeBSD issue with libusb_get_port_numbers()

Currently (as of date 20150122) an ioctl problem within the
FreeBSD kernel is preventing libusb_get_port_numbers() from working.
Hence calls to libusb_get_port_numbers() will always return 0.
This makes it impossible to establish a physical path the the usb device.

This problem has existed "forever" -
meaning that libusb_get_port_numbers() has never worked.

A fix is committed to FreeBSD "current" head  -
and will later be merged (MFC'ed) to maintenance branches.
See: https://svnweb.freebsd.org/base?view=revision&revision=277417

Additionally FreeBSD requires that devices prior to calling
libusb_get_port_numbers() have been opened with libusb_open().

The patch is "forwards-compatible".
Currently it acts specificly to libusb_get_port_numbers()
currently returning 0 on FreeBSD.
In these situations it constructs an artificial path to the device.
When FreeBSD kernels appears with proper working ioctl
supporting libusb_get_port_numbers() the code will construct
proper physical paths for newer kernels - while still generating
artificial physical paths for older defective kernels.

demo: implement averaging support

Add support for averaging and avg_samples option to the demo device.

Signed-off-by: Bartosz Golaszewski <redacted>

new config options: averaging

Add new config options to libsigrok - 'averaging', which allows to
enable averaging of samples and 'avg_samples' for setting the number
of samples to be averaged over in each cycle.

Signed-off-by: Bartosz Golaszewski <redacted>