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,
               ^

input/wav: windows: Fix a compiler warning.

  src/input/wav.c:41:0: warning: "WAVE_FORMAT_PCM" redefined
   #define WAVE_FORMAT_PCM          0x0001
   ^
  In file included from [...]/include/windows.h:86:0,
                   from [...]/include/libusb-1.0/libusb.h:70,
                   from ./src/libsigrok-internal.h:31,
                   from src/input/wav.c:28:
  [...]/include/mmsystem.h:482:0: note: this is the location of the previous definition
   #define WAVE_FORMAT_PCM 1
   ^

baylibre-acme: Fix a compiler warning.

    CC       src/hardware/baylibre-acme/protocol.lo
  ../src/hardware/baylibre-acme/protocol.c: In function 'bl_acme_set_power_off':
  ../src/hardware/baylibre-acme/protocol.c:417:6: warning: variable 'val' set but not used [-Wunused-but-set-variable]
    int val;
        ^

baylibre-acme: Set update_interval when modifing samplerate.

Both ina2xx and tmp401 linux drivers used by baylibre-acme expose
the standard hwmon update_interval attribute, which affects the internal
update interval of the chip.

When setting samplerate for data acquisition try to modify this
attribute accordingly.

Signed-off-by: Bartosz Golaszewski <redacted>

baylibre-acme: Dynamically check per probe config options.

PROBE_FACTOR and POWER_OFF options are advertised for all ACME probes
(channel groups) regardless of whether they actually have given capability.

Check these options in config_list() at runtime and only advertise them
for probes which support them.

Signed-off-by: Bartosz Golaszewski <redacted>

Fix a USB timeout related issue in sr_session_iteration().

This issue could lead to e.g. crashes when an OLS was used.

This fixes bug #571.

uni-t-dmm: Add a missing NULL (fixes a crash on Android).

This caused an non-terminated driver list, which lead to a crash
on Android (at least on ARM).

dslogic: Fix FPGA bitstream upload.

dslogic: Add #defines for timeouts and delays.

rigol-ds: Drop unneeded check of g_strdup_printf() result.

configure.ac: Drop unneeded AC_CANONICAL_SYSTEM.

autogen.sh: Drop obsolete MinGW/MSYS items.

For Windows builds (which require MinGW-w64) we currently support:

 - cross-builds using MXE (mxe.cc) and possibly other cross-compile setups

 - native builds using MSYS2 (sf.net/projects/msys2/)

Neither of those require explicitly specifying ACLOCAL_DIR.

cxx: Makefile.am: Use libtool's -no-undefined option.

windows: Fix building shared library on MinGW.

sr_driver_list() now takes a context pointer.

This requires sr_hw_cleanup_all() and sanity_check_all_drivers()
to also take a context.

The (runtime) generation of the driver list now happens in sr_init()
and sr_driver_list() always returns that pre-generated list. This fixes
a segfault when (correctly) invoking multiple sr_init() and sr_exit()
calls with different contexts (caught by the unit tests).

This fixes bug #565.

link-mso: Eliminate unneeded NUM_CHANNELS.

fx2lafw: Use libusb_error_name() for reporting transfer status.

Replace some magic numbers with a #define.

Reduce unnecessarily high indentation level in some places.

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.