bindings/python: Add __repr__() to 'Channel' class

The following lines show the class string representation before and
after this patch:

<sigrok.core.classes.Channel; proxy of <Swig Object of type 'std::shared_ptr< sigrok::Channel > *' at 0x7fdf4b29ff00> >
Channel (type=ChannelType.LOGIC, name='D0', index=0, enabled=True)

Signed-off-by: Marc Schink <redacted>

bindings/python: Add __repr__() to 'Driver' class

The following lines show the class string representation before and
after this patch:

<sigrok.core.classes.Driver; proxy of <Swig Object of type 'std::shared_ptr< sigrok::Driver > *' at 0x7fdf4b29cff0> >
Driver (name='demo', long_name='Demo driver and pattern generator')

Signed-off-by: Marc Schink <redacted>

bindings/python: More user-friendly class string representation

Add a function to generate a more descriptive and user-friendly class
string representation using class attributes.

Signed-off-by: Marc Schink <redacted>

Drop Python 2 support

Python 2 reached its official end-of-life in 2020 and is no longer
supported. Use the oldest but still maintained Python 3 version as
minimum requirement.

Signed-off-by: Marc Schink <redacted>

Bump minimum libzip requirement to 0.11

The commit 3406d671 ("output/srzip: Replace deprecated libzip calls")
makes use of new libzip API calls but does not bump the libzip version
requirements, fix this.

According to [1], the minimum required libzip version is now 0.11.

[1] https://github.com/nih-at/libzip/blob/main/API-CHANGES.md

Signed-off-by: Marc Schink <redacted>

maynuo-m97: Synchronize read and write operations

This patch is for thread safety when the library is used by SmuView
and should fix https://github.com/knarfS/smuview/issues/27

fluke-dmm: Set the SR_MQFLAG_(AC|DC) flags for sums

The VAC_PLUS_DC and AAC_PLUS_DC units measure the sum of the AC and DC
components. Set both the SR_MQFLAG_AC and SR_MQFLAG_DC flags in these
cases.

Signed-off-by: Andreas Sandberg <redacted>

fluke-dmm: Fix Fluke 28x unit mappings

The Fluke 28x unit mapping has a couple of bugs: 'dBm' incorrectly
maps to SQ_MQ_VOLTAGE and 'AAC_PLUS_DC' incorrectly maps to
SR_MQ_VOLTAGE. Fix these issues.

Signed-off-by: Andreas Sandberg <redacted>

fluke-dmm: Declare tables as constants

There is no need to modify the various lookup tables at
runtime. Declare these as const to avoid surprises.

Signed-off-by: Andreas Sandberg <redacted>

scpi-pps: Add support for Elektro-Automatik PS 20xx

Signed-off-by: Nicolas Labriet <redacted>

gwinstek-psp: Only build if serial_comm is enabled

raspberrypi-pico: Don't derive channel type from name

tcp: Avoid blocking in sr_fd_is_readable if using poll

This is consistent with the select path and the function description.

input/protocoldata: Check if lookup_protocol_name is valid

As of today, the code does not crash because all protocols have names.
But the name of the first element is not checked to be non-NULL
before calling strcmp(). This can result in a future crash if the
code logic changes.

This is mostly a change to robustify the lookup_protocol_name() function.

Also, this commit simplifies the indexing logic of the protocols array.

maynuo-m97: Enable support for the M9710

The reported model number is 28 on this one.

bindings/python: Add support for SR_T_MQ in Python bindings

tests: Replace remaining deprecated check calls

These were not caught by coccinelle.

tests: Replace deprecated fail(_unless) calls

A new API (ck_assert) to replace fail_* was introduced in check
0.9.6 (released in 2008), and fail_* has been deprecated since at least
0.9.10 (released in 2013).

This patch was automated with coccinelle:

  spatch --sp-file patch.coc --in-place tests/*.c

@@
expression msg;
@@
 <...
- fail(msg);
+ ck_abort_msg(msg);
  ...>

@@
expression msg, arg1;
@@
 <...
- fail(msg, arg1);
+ ck_abort_msg(msg, arg1);
  ...>

@@
expression msg, arg1, arg2;
@@
 <...
- fail(msg, arg1, arg2);
+ ck_abort_msg(msg, arg1, arg2);
  ...>

@@
expression expr;
@@

  <...
- fail_unless(expr);
+ ck_assert(expr);
  ...>

@@
expression expr, msg;
@@

  <...
- fail_unless(expr, msg);
+ ck_assert_msg(expr, msg);
  ...>

@@
expression expr, msg, arg1;
@@

  <...
- fail_unless(expr, msg, arg1);
+ ck_assert_msg(expr, msg, arg1);
  ...>

@@
expression expr, msg, arg1, arg2;
@@

  <...
- fail_unless(expr, msg, arg1, arg2);
+ ck_assert_msg(expr, msg, arg1, arg2);
  ...>

@@
expression expr, msg, arg1, arg2, arg3;
@@

  <...
- fail_unless(expr, msg, arg1, arg2, arg3);
+ ck_assert_msg(expr, msg, arg1, arg2, arg3);
  ...>

@@
expression expr, msg, arg1, arg2, arg3, arg4;
@@

  <...
- fail_unless(expr, msg, arg1, arg2, arg3, arg4);
+ ck_assert_msg(expr, msg, arg1, arg2, arg3, arg4);
  ...>

@@
expression expr, msg, arg1, arg2, arg3, arg4, arg5;
@@

  <...
- fail_unless(expr, msg, arg1, arg2, arg3, arg4, arg5);
+ ck_assert_msg(expr, msg, arg1, arg2, arg3, arg4, arg5);
  ...>

output/srzip: Replace deprecated libzip calls

zip_add/replace/rename all gained an additional `flags` argument, which
can be set to zero to get the old behavior.

bindings/cxx: support SWIG 4.2 and later

otherwise building sigrok fails with
bindings/swig/enums.i:1: Error: Unknown directive '%attribute'.

Also indicate that bindings/swig/enums.i is a generated file.

Signed-off-by: Klaus Kämpf <redacted>

README.devices: Mention raspberrypi-pico firmware requirement

raspberrypi-pico: Add link to firmware repo

Since the Raspberry Pi Pico is a programmable device with a myriad of
use-cases, it needs specific firmware to implement the same protocol as
the driver in src/hardware/raspberrypi-pico.

Add a link to make it easier to find the right firmware.

hardware/beaglelogic: Don't include incorrect header <sys/errno.h>

The libc's errno API[1] comes from <errno.h>, not <sys/errno.h>.
On musl-libc, a warning is produced when the latter is used:

In file included from src/hardware/beaglelogic/beaglelogic.h:26,
                 from src/hardware/beaglelogic/protocol.c:33:
/usr/include/sys/errno.h:1:2: warning: #warning redirecting incorrect #include <sys/errno.h> to <errno.h> [-Wcpp]
    1 | #warning redirecting incorrect #include <sys/errno.h> to <errno.h>
      |  ^~~~~~~

Is it not necessary to change <sys/errno.h> to <errno.h>, because
individual C files in the beaglelogic driver already include <errno.h>.

[1]: https://en.cppreference.com/w/c/error/errno

README.devices: Fix typo

configure: Fix bashism in _SR_DRIVER

dmm/vc96: Style fixes

dmm/vc96: Fixed wrong detection and calculation.

- check against "  " (two spaces) for info->is_unitless (line 137)
- introduced the handling of the returncode of sr_atof_ascii (line 73)
- explanation for ignoring case in line 132 added

input/trace32_ad: Minor header documentation addition

configure: Also test for new "check" option

When configure was called without any --enable-... parameters, the HAVE_DRIVERS
define would be undefined, leading to the driver list being empty:

https://sigrok.org/gitweb/?p=libsigrok.git;a=blob;f=configure.ac;h=f846fcdc12171bb487596bf92e398d197a6b74dc;hb=HEAD#l304

https://sigrok.org/gitweb/?p=libsigrok.git;a=blob;f=src/drivers.c;h=6dc2c55cbd18785813186a5e814d0a7d020d1583;hb=HEAD#l50

This was masked when configure is called with --enable-all-drivers as this
parameter is counted as an enabled driver (even though it technically isn't).

configure: Make dependency error message more explicit

configure: differ between check and yes/no

So far one could only specify yes/no (optionally for all drivers through
--enable-all-drivers={yes,no}). However, even when stating a =yes, the
dependency check would override it to a =no if the dependencies are
missing.

This is a bit annoying for scripts since a script may specify
--enable-foo-driver=yes but in fact the build may not contain
`foo-driver` due to missing build dependencies. This is especially
problematic for distro's which look like they currently do not build a
defined driver set - which results in random drivers missing [1].

This fixes that by following the autoconf examples [2] that allow a third
state `check`. With that `no` means no, `yes` means yes (raise an error
if deps are unavailable) and `check` gracefully disables drivers with
unavailable dependencies.

With that --enable-all-drivers can be used to ensure that all drivers are
built in. Certain drivers can still be explicitly disabled by overriding
the individual driver setting.

This way distros can specify --enable-all-drivers and new drivers with
unmet dependencies will cause an error rather than silently getting
dropped of the build.

[1] https://sigrok.org/bugzilla/show_bug.cgi?id=1699
[2] https://www.gnu.org/software/autoconf/manual/autoconf-2.67/html_node/External-Software.html#External-Software

To: sigrok-devel@lists.sourceforge.net
Cc: Daniel Thompson <redacted>

input/vcd: Add parsing support for SV 'logic' type

logic is a SystemVerilog data type which is an extension of Verilog.
The relevant specification is contained in IEEE 1800-2017.

input/vcd: Add option to manually specify & enforce a samplerate

bkprecision-1856d: Add implementation

bkprecision-1856d: Initial driver skeleton

libsigrok.h: Add config keys SR_CONF_GATE_TIME and SR_CONF_FREQUENCY_COUNTER

link-mso19: Refactor to remove TRIG_UPDATE_* macros

We now calculate the trigger register on the fly like threshold value.

link-mso19: Make mso_calc_trigger_threshold() static

It's only used in protocol.c so there's no need to export it.

link-mso19: Refactor mso_limit_trigger_level to mso_set_trigger_level

Function acts as a setter now, instead of updating devc based on
existing settings.

Min and max limits are now easier to understand.

link-mso19: Change update functions to setter functions

mso_update_trigger_pos(devc) -> mso_set_trigger_pos(devc, pos)
mso_update_offset_value(devc) -> mso_set_offset_value(devc, offset)

link-mso19: Add comments and macros for magic numbers

Also make the code mso_calc_trigger_threshold easier to understand.

link-mso19: Style changes to keep lines less than 80 chars

link-mso19: Refactor code style to declare variables first

Variables are now declared without any value, and then set later in
the code.

link-mso19: Rename variables to be more descriptive

link-mso19: Remove update function mso_update_logic_threshold_value

link-mso19: Use upper case CG_IS_DIGITAL/CG_IS_ANALOG macro names

link-mso19: Use correct value for 5MHz sample rate

link-mso19: Add offset config option

link-mso19: Add logic_threshold config option

link-mso19: Fixup horiz_triggerpos and add trigger location

link-mso19: Add trigger_level config option

link-mso19: Add probe_factor config option

link-mso19: Setup triggers to basic working state

link-mso19: Add coupling config option

link-mso19: Rework avoid unnecessary endian conversion

Utilizing compile time endian correction instead of using
runtime macros.

link-mso19: Fixup code to get basic analog and digital data working

link-mso19: Remove unused (and invalid) LED function

link-mso19: Fixup the scan function to work properly

This device depends on libserialport to collect the usb serial number
which contains device specific parameters.

link-mso19: Re-enable building this hardware

link-mso19: Remove stale code to get driver to compile

serial_write() doesn't exist anymore, for example.

link-mso19: Update files to match new driver templates better

gwinstek-psp: Initial implementation

gwinstek-psp: Initial driver skeleton

siglent-sdl10x0: Fix compile warning

lecroy-xstream: Fix memory leak

Inspired by https://github.com/sigrokproject/libsigrok/pull/162
Thanks Lucien Murray-Pitts.

siglent-sdl10x0: Implement Siglent SDL10x0 electronic load driver

siglent-sdl10x0: Initial driver skeleton

libsigrok.h: Add SR_CONF_OVER_POWER_PROTECTION_* and SR_CONF_RESISTANCE

These are used for electronic loads like the Siglent SDL10x0.

fluke-dmm: Move device-specific DM handling

With the exception of the Fluke 28x series, device-specific DM
handling is currently handled in protocol.c. Move this to separate
files for clarity.

Signed-off-by: Andreas Sandberg <redacted>

fluke-dmm: Remove device-specific float precision helper

The Fluke DMM driver has a helper function that determines the
precision of a float. This functionality is now provided by strutil.

Signed-off-by: Andreas Sandberg <redacted>

fluke-dmm: Cleanup Fluke 28x QM handling

The code that parses QM responses from Fluke 28x DMMs uses a long
series of if-else statements to determine the type of measurement
reported by the device. Rewrite this to use a conversion table that
contains a mapping between DMM units and Sigrok's internal units.

Signed-off-by: Andreas Sandberg <redacted>

strutil: Add a atof variant that determines precision

There is already a helper to convert a string to a double and
determine its precision. Add a helper that does this for floats as
well.

Signed-off-by: Andreas Sandberg <redacted>

strutil: Add a helper to get the precision of a float

There are a few different instruments where the precision of a
measurement can be determined from the number of decimals in the
reported floats. Add a helper to determine this.

The implemented version of sr_count_digits performs more stringent
error checking than the original version in
sr_atod_ascii_digits. There are two reasons for this: First, we can no
longer rely on atod failing if the provided value is invalid. Second,
there are exotic float formats accepted by atod that we don't support
and we don't want to be in a situation where atod accepts a value and
count digits returns an incorrect value.

Signed-off-by: Andreas Sandberg <redacted>

bindings/python: Use implicit namespaces

The use of 'namespace_packages' is deprecated [1] and for Python >= 3.3,
implicit/native namespaces should be used. There are no incompatibilities
as we require Python >= 3.8. More details can be found in [2] and [3].

[1] https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
[2] https://packaging.python.org/en/latest/guides/packaging-namespace-packages/
[3] https://peps.python.org/pep-0420/

scpi-pps: Minor style fix

scpi-pps: Add Keysight E36300A

- Add E36311A, E36312A, and E36313A
- Tested using E36312A via LAN and smuview

rigol-dg: Announce counter channels for dg1000z only

rigol-dg: Read back parameters for MSO5000

rigol-dg: Reduce unnecessary conversions between waveform and string repr

rigol-dg: Use user name in waveform spec for rigol_dg_waveform_to_string

rigol-dg: Add user name to waveform spec to allow consistent mapping functions

rigol-dg: Add some more waveforms for MSO5000

Known issue: Read-back of parameters does not work.
Caused by changing :SOUR:APPL to :SOUR:FUNC, because APPL only
supports NOIS, PULS, RAMP, SIN, SQU, USER.
When MSO5000 is configured to a different waveform, there is no answer at all.
sr: [00:00.051760] scpi_vxi: Successfully sent SCPI command: 'SOUR1:APPL?'.
sr: [00:02.056361] scpi_vxi: Device read failed for 192.168.1.250 with error 15

rigol-dg: Move string to waveform mapping into separate function

rigol-dg: Add basic commands for MSO5000 models

rigol-dg: Handle unknown waveform types and abort gracefully

configure: quote argument to SR_APPEND

./configure: line 19086: zlib: command not found

because of

sr_deps_avail=${sr_deps_avail}${sr_deps_avail:+' '}crc32 zlib

Signed-off-by: Sven Wegener <redacted>

input/isf: Minor style fixes

input/isf: make WFMTYPE an optional header item

input: add import module for Tektronix ISF file format

Tektronix devices use ISF format to store captured data.
The format varies depending on the device, so the module
tries to be as general as possible.

zeroplus-logic-cube: Add external clock settings

Enables the "external clock mode" in the Zeroplus LAP-C Logic
Cube driver (falling and rising edge) so that this mode also can be used in
sigrok-cli and pulseview

serial-dmm: Add support for Digitech QM1576 serial protocol parser

The protocol is described at https://www.airspayce.com/mikem/QM1578/protocol.txt

You can use this decoder with libsigrok and Digitech QM1578 via ESP32 Bluetooth-Serial converter available from the author at: https://www.airspayce.com/mikem/QM1578/QM1578BluetoothClient.ino which connects to the QM1578 over Bluetooth LE, fetches the data stream and sends it on the serial port to the host, where this driver can read it with this command for example:

sigrok-cli --driver digitech-qm1578:conn=/dev/ttyUSB1 --continuous

openbench-logic-sniffer: Properly set samplerate range

Driver does query max_samplerate from SUMP metadata,
but previously replied to SR_CONF_SAMPLERATE with fixed
1Hz-200MHz range. Update to use max_samplerate as top
of range.

May fix http://sigrok.org/bugzilla/show_bug.cgi?id=1465

openbench-logic-sniffer: Close serial port if device scan fails

openbench-logic-sniffer/protocol: Fix hardcoded unitsize

Some output modules, like srzip, calculate unitsize from maximum number of
channels in the session; and throw an error if the actual datastream uses
bigger size. Remove hardcoded value of 4 and calculate it from maximum number
of channels, supported by the device. Other driver seem to do the same.

This fixes "Unexpected unit size" saving a capture, made by an OLS device with
small number of channels.

The bug was found by using gillham's Arduino analyzer
(https://github.com/gillham/logic_analyzer), which supports a maximum of 8
channels. According to the error message, srzip expected unitsize == 1.

Signed-off-by: Pavel Fedin <redacted>

bindings/swig: Make Device::connection_id() an attribute

Signed-off-by: Marc Schink <redacted>

bindings/swig: Make Device::serial_number() an attribute

Convert Device::serial_number() to an attribute string as for other
functions such as vendor() and model().

Signed-off-by: Marc Schink <redacted>

Add SR_CONF_DELAY_GENERATOR

Signed-off-by: Marc Schink <redacted>

Add SR_CONF_INVERTED

Many device such as oscilloscopes, signal and delay generators are
capable to invert signals or change their polarity.

Add a new configuration to implement such a feature. The term 'inverted'
is chosen in favor of 'polarity' because the former can be unambiguously
represented by a boolean value.

Signed-off-by: Marc Schink <redacted>

backend: Add missing #ifdef for zlib

Add #ifdef statement since zlib is optional. At the moment an empty
version string is used when zlib is not available:

backend: Libs: glib 2.72.4 (rt: 2.72.4/7204:4), zlib , libzip 1.7.3,

Signed-off-by: Marc Schink <redacted>

tcp: Fix memory leak in sr_tcp_dev_inst_free()

Signed-off-by: Marc Schink <redacted>