Minor cosmetics and consistency fixes.

Don't check g_free() arguments for NULL.

The g_free() call is guaranteed to not segfault when NULL is passed.

cem-dt-885x: Add a missing break statement.

zeroplus: Add a missing break statement.

This fixes a bug when trying to trigger on a channel being 0/low.

Fix a typo in a for loop (wrong variable).

sysclk-lwla: Use ARRAY_SIZE instead of G_N_ELEMENTS.

Both do exactly the same, but we consistently use ARRAY_SIZE
in the rest of the code-base.

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

unit tests: Update for sr_session_new() API change.

bindings/cxx/classes.cpp: Fix sr_session_load() invocation.

bindings/cxx/classes.cpp: Fix a typo.

Call libusb_get_next_timeout() to get minimum timeout for g_poll().

We should have been doing this all along, but we get away with it
on Linux where libusb can do everything with fds, and we get away
with it for many drivers that have a short timeout on their events.

On Windows though, handling this correctly is essential.

Fixes bug #343.

Store a context pointer in struct sr_session.

Make sr_session_new() and sr_session_load() require a context.

Revert "session_file.c: Use config_*() wrappers."

This temporarily reverts commit 421bc3eba02f060319c752a26461148fc93563ec.

We cannot yet use the sr_config_*() wrappers, otherwise loading *.sr
files is broken. A fix is being worked on.

Initial fix attempt for a thread-related issue on Windows.

This is a partial fix for bug #343, which lead to a large amount of
handles being created, and eventually to a frontend "hang".

It's not yet a "full" fix as some issues are still observable,
but it successfully improves the situation on Windows to the extent
that frontend hangs due to large amounts of handles no longer seem
to happen.

Thanks to Boris Gjenero <redacted> for the debugging
efforts, testing, and updating of this patch!

Additionally, this seems to also fix a "SysClk LWLA hanging" bug
and apparently not receiving any samples during an acquisition
(tested on an LWLA1034).

This closes bug #328.

Various #include file cosmetic fixes.

Generally include system headers before local headers, unless there's
a technical reason to use another order.

Various NULL-check consistency fixes.

Some more g_try_*alloc() fixes.

As per documented rules in HACKING, we don't check "small" allocations.

Improve readability and clarity of some numbers.

Fix a bunch of typos.

Add udev rule for Rigol 1000Z series

demo: Fix SR_CONF_DEVICE_OPTIONS variant type.

Channel names consistency fixes and simplifications.

Remove unneeded #endif comments.

Consistency and whitespace fixes for switch statements.

Make memset() invocations consistent across all files.

Random whitespace and other minor fixes.

session_file.c: Use config_*() wrappers.

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);
                    ^