input/trace32_ad: silence format errors during input format match

When auto-detection tries to find the input module that can handle a
file, unexpected input format should be considered non-fatal. Only emit
error messages when process_header() got invoked from process_buffer()
after initialization. Emit debug messages in early stages where no input
module context exists yet.

dreamsourcelab-dslogic: Properly set initial voltage threshold.

The initial voltage threshold in dev_open() was being set to a
default value in the devc->cur_threshold variable but not actually set
in the device itself.

Patch by Jörg Alpers <redacted>, thanks a lot!

dmm/ms8250d: Fix two log messages.

Add support for the MASTECH MS8250D multimeter

hantek-4032l: Emit FPGA version log message.

soft-trigger: Fix an issue causing triggers to not work.

rigol-ds: Fix an issue causing only one channel to be acquired.

This fixes bug #1018.

rigol-ds: Make two log messages more specific.

Slightly modified version of a suggested change by Aleksander Alsekseev.

hantek-4032l: Add option to abort acquisition.

Signed-off-by: Andrej Valek <redacted>

hantek-4032l: Fix mismatch in magic number.

- fix LIBUSB_TIMEOUT errors
- fix same data receiving
- send reset vendor request before new data getting
- decrease USB polling timeout

This fixes bug #1190.

Signed-off-by: Andrej Valek <redacted>

hantek-4032l: Fix structure packing.

- use pragma to handle different behavior between gcc and minGW bit-field packing
- bit-field integer variables needs to be align to 2-byte boundary
Compiler does not produce an error when accessing into non-__packed pointer.
However, the field might not be properly aligned for this type.
More information could be found on:
 - https://sourceforge.net/p/mingw-w64/bugs/275/
 - http://www.keil.com/support/man/docs/ARMCC/armcc_chr1359124990875.htm

Signed-off-by: Andrej Valek <redacted>

hantek-4032l: Fix typo from device porting to new lib version.

Signed-off-by: Andrej Valek <redacted>

log: Move log level check so that it affects all handlers.

Before this change, the loglevel check would only be performed for the
default log handler in libsigrok, but not for other handlers set
via sr_log_callback_set().

This fixes bug #698.

output/csv: Make the label values option a list.

This allows UIs to display a drop-down with all possible choices,
instead of forcing the user to look up / remember all of them.

input/csv: Make the data format option a list.

This allows UIs to display a drop-down with all possible choices,
instead of forcing the user to look up / remember all of them.

soft-trigger: Use g_try_malloc() for potentially large allocation.

This fixes bug #1000.

session_driver: Quickfix to prevent looped analog data reads.

A local buffer was too small, snprintf() was used to write the
name of the analog-1-1-xxx ZIP archive names in that buffer.

Due to the limited size, only 3 characters were usable for the last
number component, i.e. the code would loop around from analog-1-1-999
to analog-1-1-100 (instead of analog-1-1-1000).

As a quickfix, increase the buffer size by a large margin, a nicer
fix should be used later on.

input/null: Add a null module that discards all data.

This is useful for testing purposes.

output/null: Add a null module that discards all data.

This is useful for testing purposes.

input/output: Slightly improved module descriptions.

input modules: Increase chunk size to 4MB for all modules.

This reduces overhead and can slightly increase performance, depending
on the module.

input modules: Name chunk size #defines CHUNK_SIZE consistently.

input/trace32_ad: Drop unused #define.

input/wav: Put internal buffer on the heap instead of the stack.

Increasing the buffer/chunk size could cause issues when trying to put
large arrays on the stack.

input/binary: Increase chunk size from 4KB to 4MB.

This can slightly increase performance when loading larger files.

input/binary: Only calculate unitsize once.

input modules: Improve option names and descriptions.

These should be slightly more useful and contain more information
(should also work reasonably well for 'sigrok-cli --show' and as
tooltips in UIs).

zketech-ebd-usb: Drop unneeded log message.

The number of bytes sent is already logged by the libsigrok serial code.

configure.ac: zketech-ebd-usb driver needs libserialport.

zketech-ebd-usb: Shorten function name prefix for better readability.

zketech-ebd-usb: Make a few functions static.

zketech-ebd-usb: Add some underscores to #defines.

zketech-ebd-usb: First version of the driver.

zketech-ebd-usb: Initial driver skeleton.

usbtmc: Silence some overly verbose log messages.

These trigger quite often with unrelated devices and confuse people.

  scpi_usbtmc: Failed to get configuration descriptor: LIBUSB_ERROR_NOT_FOUND, ignoring device.

rdtech-dps: User ternary operator.

rdtech-dps: New driver for RDTech DPS/DPH series PSUs.

rdtech-dps: Initial driver skeleton.

modbus: Return explicit SR_ERR values as required by modbus.c

The Modbus RTU implementation was inappropriately returning lengths
from the serial functions when the calling functions expect only an
sr_error_code value.

modbus: Increase modbus reply timeout to 500ms.

Needed by rdtech-dps driver, 100ms is too short.

resource.c: Fix firmware loading bug (#1140)

backend: Emit firmware search paths in a log message.

Introduce sr_resourcepaths_get()

This provides an interface to fix #1128.

korad-kaxxxxp: Korad OEM: RND KA3005P

output/csv: fix out-of-bounds array access in process_analog()

Make sure to not exceed the ctx->analog_samples[] array bounds. Don't
use the (huge) channel's index in the device's(!) channel list, instead
use the zero-based and dense index into the array of analog samples in
the accumulation buffer, before writing to the external file.

This fixes the segfault reported in bug #1124.

output/csv: use longer names for iteration variables

The process_analog() logic is rather complex, dealing with the total
list of channels in the device (which can be of different types), and a
number of submitted samples for a specified list of channels. Replace
the rather short variable names for i, j, c (and num_channels) with
something longer that hopefully increases readability of the complex
loop bodies.

Note that this change merely renames identifiers, and does not change
behaviour.

output/csv: reduce indentation in process_analog()

Instead of nesting indentation levels upon equality of a value, skip
iterations upon inequality. This reduces indentation, and might improve
readability.

[ Indentation changes, see 'diff -w -b' for the essence. ]

hp-3478a: Remove unnecessary curly brackets.

hp-3478a: Set correct number of digits

hp-3478a: spec_digits must be parsed before range parsing.

arachnid-labs-re-load-pro: Replace C++-style comments with C-style comments.

metex14: Fix wrong measurement modes

metex14: Add power factor measurement mode

libsigrok.h: Fix SR_CONF_UNDER_VOLTAGE_CONDITION_THRESHOLD order.

This has to be the last entry in its "category", otherwise it'll mean an
ABI breakage.

arachnid-labs-re-load-pro: Simplify a code chunk.

arachnid-labs-re-load-pro: Add encoding.digits to analog packet

arachnid-labs-re-load-pro: Index for channel "I"

arachnid-labs-re-load-pro: Get a response when in acquision mode.

arachnid-labs-re-load-pro: Make SR_CONF_REGULATION listable

arachnid-labs-re-load-pro: Add SR_CONF_UNDER_VOLTAGE_THRESHOLD.

arachnid-labs-re-load-pro: Setting correct current limit.

Use round() instead of just truncate the value.

arachnid-labs-re-load-pro: Change serial read in acquisition mode.

Use serial_readline in acquisition mode, otherwise data from the
Re:load Pro could get lost.
Use reloadpro_receive_data() for all commands when in acquisition
mode. When not using a single point of receiving data, data could get
lost.

pipistrello-ols: style nit, replace DIY endianess conversion

link-mso19: improve endianess conversion, avoid mem access alignment issue

Prefer sigrok's endianess conversion helper over the inet htons()
routine which is harder to read (is "network order" little or big?).

Writing the conversion results in units of bytes also avoids misaligned
memory access. The header length is odd, each payload item got written
as an uint16_t item to an odd address.

uni-t-dmm: style nits, copy loop vs memmove

saleae-logic-pro: style nits, copy loop vs memcpy

pce-322a: style nits, copy loop vs memmove

mic-985xx: style nits, var decl vs assignment, copy loop vs memmove

kern-scale: style nits, var decl vs assignment, copy loop vs memmove

center-3xx: style nits, var decl vs assignment, copy loop vs memmove

siglent-sds: Drop superfluous log message.

This information is already emitted by other layers.

siglent-sds: Added trigger position to device config code

siglent-sds: Fixed issue with hard coded vdiv.

siglent-sds: Drop unused variable from siglent_sds_read_header function

siglent-sds: Added averaging function.

siglent-sds: Fixed timebase problem where NS could not be selected

This fixes bug #1120.

tests: Fix incomplete fail message for locale tests.

hp-3478a: Fix compiler warning.

Check usb_get_port_path() return value

This function can fail. If so, do not ignore the failure.

Ensure device is closed before usb_get_port_path()

The usb_get_port_path() function opens the passed device on FreeBSD,
which fails if the device has already been open.

This fixes bug #1109.

bindings/ruby: include config.h before any other header file

bindings/python: include config.h before any other header file

bindings/cxx: include config.h as early as possible

Move the inclusion of <config.h> before any other source code.

pce-322a: include config.h before any other header file

beaglelogic: Use UINT64_MAX instead of (uint64_t)-1.

beaglelogic: Drop unneeded uint64_t cast.

lecroy-xstream: Fix potential issue by adding UINT64_C.

Use UINT64_C instead of "ULL" number suffix.

Avoid hardcoding a "ULL" number suffix, use the more portable and more
correct UINT64_C.

Random whitespace/cosmetic fixes.

siglent-sds: Fix two memory leaks.

siglent-sds: Drop incorrect glib includes.

hp-3478a: Shorten some functions.

hp-3478a: Drop superfluous log message prefixes.

hp-3478a: Initial HP 3478A diver

hp-3478a: Initial driver skeleton.

output/analog: Fix a compiler warning on Mac OS X.

Apparently PRIu64 and G_GUINT64_FORMAT differ on some systems for
unknown reasons. Use G_GUINT64_FORMAT to get rid of the warning
for now.

Fix two compiler warnings on MinGW/MSYS2.

The config.h file must always be included as first file.

  src/output/csv.c: In function 'gen_header':
  src/output/csv.c:64:20: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Wformat=]
   #define LOG_PREFIX "output/csv"
                      ^
  ./src/libsigrok-internal.h:753:42: note: in expansion of macro 'LOG_PREFIX'
   #define sr_info(...) sr_log(SR_LOG_INFO, LOG_PREFIX ": " __VA_ARGS__)
                                            ^
  src/output/csv.c:244:3: note: in expansion of macro 'sr_info'
     sr_info("Set sample period to %" PRIu64 " %s",
     ^
  src/output/csv.c: In function 'dump_saved_values':
  src/output/csv.c:462:34: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Wformat=]
       g_string_append_printf(*out, "%" PRIu64 "%s",
                                    ^

  In file included from src/hardware/ftdi-la/protocol.c:21:0:
  src/hardware/ftdi-la/protocol.c: In function 'send_samples':
  src/hardware/ftdi-la/protocol.h:28:20: warning: format '%u' expects argument of type 'unsigned int', but argument 3 has type 'uint64_t {aka long long unsigned int}' [-Wformat=]
   #define LOG_PREFIX "ftdi-la"
                      ^
  ./src/libsigrok-internal.h:751:42: note: in expansion of macro 'LOG_PREFIX'
   #define sr_spew(...) sr_log(SR_LOG_SPEW, LOG_PREFIX ": " __VA_ARGS__)
                                            ^
  src/hardware/ftdi-la/protocol.c:29:2: note: in expansion of macro 'sr_spew'
    sr_spew("Sending %" PRIu64 " samples.", samples_to_send);
    ^

siglent-sds: Fix multiple compiler warnings.

siglent-sds: Remove unused variable.

dmm: Remove unnecessary casts