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

metex14: Add missing modes and set correct digits value.

serial-dmm, metex14: add support for multiple channels per DMM

Optionally create multiple analog channels in serial-dmm's scan()
routine. Allow the meters' parse routines to fill in more than one
analog value from the inspection of a single packet.

Use "large" (4 times 14 bytes) packets for the Metex M-3860M and the
PeakTech 4390A meters, and have those large packets parsed by wrapping
the routines for regular 14-byte Metex packets, and sending four values
to the session bus after reception of one large packet.

Thanks to Frank Stettner <redacted> for testing and
fixing the initial implementation of this extension.

serial-dmm: style nit (init vs assign, memmove(3))

Move the initial assignment to the 'offset' variable to the very spot
where it gets evaluated and subsequently manipulated.

Replace a DIY copy loop with the corresponding memmove(3) call.

siglent-sds: Bring driver up-to-date with current code conventions.

Use ARRAY_AND_SIZE where possible, use std.c helpers where possible,
make scanopts/drvopts/devopts consistent with other drivers.

siglent-sds: Drop obsolete SR_ST_ACTIVE checks.

These are now done in the wrapper functions.

siglent-sds: Random cosmetics, drop unused stuff.

siglent-sds: Use PRIu64 for uint64_t variables.

This fixes various compiler warnings on some systems.

siglent-sds: Fix SR_CONF_TRIGGER_SLOPE config_set() code.

Also use "r" and "f" as trigger slope values for now, since that's
what most other drivers do currently.

This also fixes two scan-build issues:

  api.c:559:3: warning: Value stored to 'tmp_str' is never read
                  tmp_str = g_variant_get_string(data, NULL);
                  ^         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  api.c:561:9: warning: Use of memory after it is freed
                  ret = siglent_sds_config_set(sdi, "%s:TRSL %s",
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

udev: Add entries for Siglent USBTMC devices.

siglent-sds: Add 100s timebase (seen on e.g. SDS1202X-E).

siglent-sds: Add SDS1052DL+ model.

siglent-sds: Various cosmetics and coding-style fixes.

Also, drop some unneeded comments and log messages.

hantek-4032l: Minor cosmetics.

std.c: Add missing <sys/time.h> #include and _XOPEN_SOURCE.

This is required for gettimeofday() to be available (at least on
FreeBSD), causes compiler warnings otherwise.

siglent-sds: Replace non-portable strcasestr() with g_strstr_len().

The strcasestr() function is non-portable (e.g. not available on
MinGW, possibly elsewhere). Replace it with g_strstr_len() for the time
being. While the latter is not case-insensitive it appears that
that property doesn't matter here ("." should not be relevant anyway,
and e.g. an SDS1202X-E does indeed report "Mpts" as such).

Should it become necessary to have this code be case-insensitive,
we'll have to find a more portable solution than strcasestr().

siglent-sds: Drop currently unused <rpc/rpc.h> #include.

This causes some compile errors on platforms where RPC is not available, e.g.

  siglent-sds/api.c:25:21: fatal error: rpc/rpc.h: No such file or directory

Surrounding the #include (and later code) with "#if HAVE_RPC" should avoid
these compile errors, but since it's not used at all for now, drop it.

README.devices: add Siglent SDS to the firmware and conn= sections

usbtmc: only reset Siglent (and Atten) devices in open

Make the USB reset for USB TMC devices conditional. Check a whitelist,
which in this implementation exclusively contains the Siglent VID.

The whitelist's comment may need adjustment. The VID probably not only
applies to SDS devices, but could be used for SDG and others as well.
And lsusb output suggests the VID is used by Atten, too.

usbtmc: add the Siglent VID and reset USB devices in open

Add the Siglent's VID to a list of blacklisted USB TMC devices.
Unconditionally reset USB devices in the open routine.

This was taken from marchel's development for Siglent SDS. Though the
reset should probably be conditional, and only get applied to devices
which are known to need it.

The comment may need adjustment, maybe individual PID entries are
required. The VID probably not only applies to SDS devices, but could
be used for SDG and other gear as well. And lsusb output suggests the
VID is used by Atten, too.

siglent-sds: initial driver implementation for Siglent SDS

Implement a first version of the driver for the Siglent SDS1000 and
SDS2000 oscilloscopes.

[ gsi: This commit corresponds to git 0228126017e6 of marchelh's tree,
  I adjusted the source to closer match the project's coding style. The
  conversion was verified by this command:

  $ git diff --word-diff=color -w -b <branch>:<dir> <branch>:<dir>

  Changes include: Whitespace adjustment, dead code removal, separation
  of variable declaration and assignment, alpha-sorted includes. Line
  length was not addressed and not every location got adjusted, to keep
  the diff minimal and to reduce effort during review of this version. ]

siglent-sds: Initial driver skeleton.

README.devices: discuss conn= and USB/TCP/VXI in addition to RS232

The conn= specification is not strictly related to RS232 communication.
Provide examples for other USB specs (as they are used for some DMM and
LCR drivers), as well as TCP and VXI (which are often found with MSO
drivers).

korad-kaxxxxp: Simplify korad_kaxxxxp_receive_data() event loop function.

korad-kaxxxxp: Use locale independent sr_snprintf_ascii() function

korad-kaxxxxp: Synchronize read and write operations.

korad-kaxxxxp: Add two channels "V" and "I", remove channel "CH1"

tests: Tests for the locale independent sr_vsnprintf_ascii() function.

strutil: Locale independent snprintf() and vsnprintf() functions

hameg-hmo: fix potential NULL dereference

Check for successful allocation before accessing struct members. Return
with an error from device initialization when allocation fails.

This was reported by clang's scan-build.

hameg-hmo: fix potential memory leak

Make sure to release the allocated "pod enabled" memory, too, when SCPI
communication during channel setup fails. Defer the return with an error
(instead of duplicating the free() invocation).

This was reported by clang's scan-build.