Remove SR_CONF_MAX_UNCOMPRESSED_SAMPLES again.

The maximum sample size that can be set on a device is now published
by sr_config_list(SR_CONF_LIMIT_SAMPLES). This returns a tuple of
uint64_t representing minimum and maximum number of samples.

sysclk-lwla: Improve message log output.

Report settings at acquisition start as informational messages.
Print a message when the the trigger condition has been met.
Demote some other messages from information to debug, and use
the %zu format for printing size_t values.

sysclk-lwla: Streamline packet output logic.

(process_sample_data): When expanding run-length samples into
session packets, calculate the number of samples to write in
advance while honoring all constraints.  This is cleaner than
checking constraints within the expansion loop.  Also, the new
logic always fills up packets exactly to whatever limit applies
first, thereby removing the need for truncation after the fact.

sysclk-lwla: Implement SR_CONF_LIMIT_MSEC.

Allow the acquisition to be constrained by time in addition to
a sample count limit.  Since the LWLA protocol actually provides
only a duration natively, implement the sample count limit on top
of the new duration limit.

With this change, limiting an acquisition in external clock mode
should finally work properly.

tests: Fix a few warnings.

Fix a bunch of "no previous prototype for ..." warnings exposed
by -Wmissing-prototypes.

demo: Bring analog square wave amplitude into line with other patterns.

demo: Add analog sawtooth pattern.

demo: Add triangle pattern.

demo: Implement sine wave pattern.

This adds sine wave generation capabilities for the analog channels in the demo
driver. The frequency of the sine wave depends on the configured sample rate of
the demo device. The frequency of the sine wave is always 20 times smaller than
the sample rate, in other words we always have 20 samples per period.

uni-t-ut32x: Fix typo that prevents usage with multiple devices.

rigol-ds: Minor whitespace fixes.

rigol-ds: Enable/disable LA pod when (de)selecting digital channels.

rigol-ds: Reset num_frames to zero in dev_acquisition_start.

rigol-ds: Make sure to always send SR_DF_END at end of capture.

rigol-ds: Calculate effective samplerate.

rigol-ds: Separate function to compute analog frame size.

rigol-ds: Advertise SR_CONF_LIMIT_FRAMES support.

rigol-ds: Fix check for probe group validity.

hameg-hmo: Fix two compiler warnings on Windows.

  CC       libsigrok_hw_hameg_hmo_la-protocol.lo
api.c: In function 'auto_find_usb':
api.c:70:39: warning: unused parameter 'vendor_id' [-Wunused-parameter]
 static GSList *auto_find_usb(uint16_t vendor_id, uint16_t product_id)
                                       ^
api.c:70:59: warning: unused parameter 'product_id' [-Wunused-parameter]
 static GSList *auto_find_usb(uint16_t vendor_id, uint16_t product_id)
                                                           ^

usb: Fix two compiler warnings on Windows.

  CC       libsigrok_hw_common_la-usb.lo
usb.c:241:18: warning: no previous prototype for 'usb_thread'
[-Wmissing-prototypes]
 SR_PRIV gpointer usb_thread(gpointer data)
                  ^
usb.c:256:13: warning: no previous prototype for 'usb_callback'
[-Wmissing-prototypes]
 SR_PRIV int usb_callback(int fd, int revents, void *cb_data)
             ^

hameg-hmo: Silence warning about format security.

The command for fetching the sample rate while in single shot mode doesn't take
an argument so the snprintf here is redundant.

hameg-hmo: Reset the number of acquired frames in acquisition stop.

hameg-hmo: Expose more options with config_get().

This patch exposes the TRIGGER_SOURCE and COUPLING options via config_get().

hameg-hmo: Advertise the frame limit option.

hameg-hmo: Update the sample rate when the channel states or the timebase change.

The sample rate on the Hameg scopes changes depending on the number of channels
turned on and on the current timebase.

Update the sample rate if any of the above change.

hameg-hmo: Add support for sample rate fetching.

scpi: Add CMD_GET_SAMPLE_RATE to the command enum.

hameg-hmo: Change the way vdiv and timebase are saved.

The current vertical division setting (per channel) and the timebase are stored
as a floating point number. This is suboptimal since clients expect us to send
this information to them in form of a rational number.

Store only the index of the current setting since all the supported settings are
already stored inside of an array.

hameg-hmo: Send the DF_END packet in acquisition_stop()

The DF_END packet was send out after all configured frames were fetched, but
devices may stop the acquisition at any point in time and an DF_END will not be
send out in this case.

Send the DF_END packet inside of acquisition_stop() so this can't happen.

hameg-hmo: Close the device after initial scan.

The device is after a scan left open and clients that don't call unconditionally
dev_open() will never fetch the initial device state.

Close the device after the scan so clients know they need to open it.

hameg-hmo: Handle floating point numbers while ignoring the locale.

scpi: Use sr_atof_ascii() instead of sr_atof().

strutil: Add function to parse floating point numbers while ignoring the locale.

Most of the supported gear uses the ANSI C locale for communication, and if the
client sets up an incompatible locale parsing would fail.

This function ignores the client's locale and parses floating point numbers
using the ANSI C locale. This function should be always used when parsing
floating point numbers coming from the instrument.

device: Pass sdi as an function argument to config_list in dev_has_option()

With drivers that support multiple devices we need to know the device model
while listing options. That information is most of the time saved in the private
part of the dev_inst structure.

Pass the pointer to the dev_inst structure as an function argument so we have
access to this information.

strutil: Correctly parse floating point frequencies in parse_size_string().

parse_size_string() incorrectly parses a real number, e.g. 1.5 kHz ends up
being 1Hz.

This patch fixes parse_size_string() to take the fractional part as well into
account. The fractional part is parsed as an double precision floating point
number while ignoring the locale.

sysclk-lwla: Make use of the probe index again.

(configure_probes): sr_probe::index has been un-deprecated,
so use it instead of relying on the list order.

sysclk-lwla: Bypass divider in external clock mode.

(lwla_setup_acquisition): Set the clock divider bypass
flag to 1 for the external clock modes as well.
(capture_setup): Set the clock divide count to 0 if an
external clock source is selected.

sysclk-lwla: Utility functions clean-up and semantic fixes.

(lwla_send_bitstream): Unref the mapped file earlier in order
to simplify the error handling.
(lwla_receive_reply): Do not treat a reply buffer length of
zero as silent no-op.  That logic was left over from an earlier
iteration, before the distinction between reply buffer size and
expected read length was introduced.

sysclk-lwla: Remove unused dev_context::next_state.

sysclk-lwla: Avoid warning due to bogus range check.

(lwla_set_clock_source): Checking whether an enum value is greater
than or equal to zero apparently results in a warning with some
compilers.  Assign the enum to an unsigned variable to avoid this,
and return SR_ERR_BUG if the range is exceeded, as this indicates
a bug in the driver code itself.

configure: add -Wmissing-prototypes compiler option

This should avoid introduction SR_PRIV functions when static would be enough.

remove unused static functions

change a bunch of functions from SR_PRIV to static

None of those functions are called across compilation units.

move function declarations to the appropriate header

es519xx: add missing declarations for es519xx_2400_11b_altfn

gnuplot: proper error message when using gnuplot output without logic probe

add udev rule for the Rigol DS2000 series

sysclk-lwla: Fix probe name issue.

The g_ascii_formatd() function expects the "format" argument to start
with a '%' character, e.g. it should be "%f" or such (this is not
clearly documented in the glib API docs, but visible from the source code).

The usage of "CH%f" for example will trigger an assertion and thus make the
LWLA device unusable in practice (e.g. in PulseView on Windows no probenames
would be shown, and sampling wouldn't work).

Example:
  GLib-CRITICAL **: g_ascii_formatd: assertion 'format[0] == '%'' failed

(not exposed in all glib versions or builds of glib on all distros
apparently, some may need G_MESSAGES_DEBUG=all or other measures)

From the glib g_ascii_formatd() code:
  g_return_val_if_fail (format[0] == '%', NULL);

We now use g_snprintf() instead for simplicity. This has been tested to
fix this specific issue (i.e. the probenames now do show up in PulseView).

This closes bug #270.

configure.ac: Don't build sysclk-lwla if libusb-1.0 is not found.

configure.ac: Move sysclk-lwla chunk to correct location.

sysclk-lwla: Honor SR_CONF_CONN at scan time.

sysclk-lwla: Fix calculation of the running sample count.

Field 7 of the status response is actually a duration in
milliseconds at all samplerates but 125 MHz.

sysclk-lwla: Implement support for the LWLA1034.

sysclk-lwla: Initial driver skeleton.

rigol-ds: Use correct digital channel numbers when fetching config.

cem-dt-885x: Don't use char as signed type.

scpi: properly check for HAVE_RPC (which is always defined)

rigol-ds: remove lonely break

rigol-ds: prevent config_list() to return empty g_variant

This fixes the following glib message:
$ ./sigrok-cli -d rigol-ds --show
[...]
g_variant_builder_end: assertion '!GVSB(builder)->uniform_item_types || GVSB(builder)->prev_item_type != NULL || g_variant_type_is_definite (GVSB(builder)->type)' failed

Use PRIu64 for format in pretty-printer.

Thanks to Marcus Comstedt for the fix.

rigol-ds: does not depend on libserialport

scpi_usbtmc: does not depend on libserialport

udev: Add the Rigol DG4000 series to the udev rules

atten-pps3xxx: Fix options reporting.

atten-pps3xxx: Fix output channel mode get.

demo: Properly handle logic vs. analog when setting the pattern.

demo: Split supported device options by probe group.

scpi_vxi: ensure the vxi link was properly opened before closing it

scpi_tcp: split into scpi_tcp_raw and scpi_tcp_rigol

The current implementation is renamed to tcp-rigol as it seems to be
a Rigol proprietary protocol used only on Rigol VS5000 series.

A new tcp-raw implementation is introduced which simply carries raw SCPI
commands over TCP. It is probably a much more common protocol and it is
at least available on Rigol DS2000 series on port 5555.

scpi: make the scpi_dev_inst_new more generic

scpi: factorize dev_inst_new calls out of individual drivers

scpi: add VXI transport support

import VXI RPC Language description

along with the corresponding rpcgen generated source code files

configure.ac: Disable atten-pps3xxx driver if libserialport is not found.

std: use #ifdef rather than #if where the constant may not be defined

atten-pps3xxx: Push configured settings even without acquisition.

atten-pps3xxx: Use configured values to construct packet.

This used the values previously received from the device, not very useful.

atten-pps3xxx: Full support for the PPS3203T-3S.

Add config keys for programmable power supplies.

atten-pps3xxx: Initial driver skeleton.

chronovu-la8: Publish SR_CONF_MAX_UNCOMPRESSED_SAMPLES.

ikalogic-scanalogic2: List all keys in device options.

zeroplus-logic-cube: List all keys in device options.

zeroplus-logic-cube: Publish SR_CONF_MAX_UNCOMPRESSED_SAMPLES.

ikalogic-scanalogic2: Publish SR_CONF_MAX_UNCOMPRESSED_SAMPLES.

Add support for SR_CONF_MAX_UNCOMPRESSED_SAMPLES.

Add SR_CONF_MAX_UNCOMPRESSED_SAMPLES.

Add sr_session_save_init().

This allows a frontend to initialize a session file, providing the
required samplerate and probe names, without having a proper
struct sr_dev_inst handy.

sr_session_append() is then used to add captured data to the session
file, as usual.

The existing sr_session_save() function works the same way as always.

serial-dmm: Drop unused DMM_COUNT.

hardware/common/dmm: Fix debug output level.

Most messages from the DMM parsers are not hard errors, lower to
sr_dbg() so that the sigrok-cli output doesn't get cluttered (by default)
with debug output such as:

 P1: 0.001100 V DC AUTO
 sr: fs9721: Sync nibble in byte 0 (0x00) is invalid.
 P1: 0.001100 V DC AUTO

(using -l 4 or -l 5 will still allow the user to see such messages)

serial-dmm: Support the Tenma 72-7745 via UT-D02 cable.

(it was already supported in uni-t-dmm via UT-D04 cable)

Add support for the Tenma 72-7750 (UNI-T UT60G rebadge).

Add initial support for the UNI-T UT60G.

README.devices: Cosmetics, consistency fixes, updates.

serial: Improved docs.

scpi_usbtmc: fix reading of blocks bigger than the 2048 bytes buffer

rigol-ds: fix waveform reception on DS2000 series

The ":WAV:DATA?" scpi command must be sent before calling
sr_scpi_read_begin().

rigol-ds: add a few more DS2000 models to the supported list

zeroplus: Add voltage threshold support

It doesn't currently mesh well with libsigrok, but at least the support is there.

zeroplus: Major rework of sample buffer processing

The sample buffer is a still a bit of a mystery, but this should help.
The variables in play:

triggerbar/ramsize_trigger - These two variables added together indicate
how many samples we want captured. ramsize_trigger - triggerbar
indicades how many samples must be captured. The ratio between the two
is determined by capture ratio.

memory_size - This indicates the number of samples in the circular
capture buffer. stop_address, now_address, and trigger_address are
pointers within the zeroplus that wrap based on this size.

now_address - The address that the zeroplus was about to write to when
it finished capturing, and now the address that will be read from when
reads are done from the capture buffer

stop_address - The address that the zeroplus last wrote to when it
completed capture.

trigger_address - The sample address for which the trigger occured.

status - This one is a bit tricky. Some testing has shown that if the
zeroplus has captured memory_size or less samples, the STATUS_READY bit
is set. For all captures generated with more samples than this,
STATUS_READY was cleared. However, boundary conditions are difficult to
test and values such as, memory_size + 1 have not been tested. We use
this to determine if the capture has wrapped through the sample buffer.

More testing is required, but this improves behavior in a number of
cases, specifically capturing sample amounts that are not a power of 2
of the sample buffer size. Before, random data was passed to libsigrok.

Signed-off-by: Russ Dill <redacted>