asix-sigma: Remove stack-based alloc in download_capture()

Avoid allocating 32KiB of data on stack in download_capture(). Instead,
do a glib-variant of calloc(1, ) to allocate the data for the samples
which will be downloaded. This avoids explosions of stack on systems
with tight stack limits.

Furthermore, define structures describing the organisation of Sigma's
DRAM memory and start using those instead of ad-hoc preprocessor macros
defining the sizes of various structures in memory.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Localize variables

The trigger position, stop position and chunk in which the trigger happened
are no longer needed in the global scope. Make those variables local to the
download_capture() function.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Move all register I/O into download_capture()

Move all the register I/O that is necessary to do the download of samples
from Sigma into download_capture() function. This makes the downloading
code contained a bit more again.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Clearly separate the sample download

Reorder the sigma_capture_mode() function so that the part which handles
the download of samples from Sigma is clearly separated from the tests if
the download should be started.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Pull out the CAPTURE mode handler

Pull out the code handling the Sigma which is in CAPTURE mode into
a separate function. This is so we can start reworking this entire
code easily soon.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Read position only in CAPTURE state

Check the position of ForceStop and Trigger events only in case we
are in CAPTURE state, it's useless to do this unconditionally when
receive_data() is called.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Decrypt the LA mode switch sequence

Decode the logic mode start sequence into a series of FPGA instructions
instead and get rid of this sequence of magic numbers.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Encode the opcodes as hex values

Just encode the opcodes as hexadecimal values. This makes for better
readability when mapping the communication dump with the sigma to the
code.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Pull out the logic-mode switching

Pull out the code which switches Sigma from the FPGA programming mode
into Logic-Analyzer mode into separate function. Also, given the reply
is only ever 3-byte long, do not allocate 32 byte big buffer, but only
a 3-byte long one.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Squash yoda-condition in upload_firmware()

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Move the sigma_fw_2_bitbang()

Move the sigma_fw_2_bitbang() function closer to the upload_firmware()
function so there's not such a horrible mess in the file.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Cleanup the bit2bitbang

Rework the bit2bitbang function. Remarkable changes are:
- The function was renamed to sigma_fw_2_bitbang()
- We use glib function calls to work with the file
- We mmap the file containing firmware instead of opening the file
  and then reading it into a buffer.
- The magic firmware transformation is now type-safe.
- Documentation and comments were added where applicable.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Weed out in-condition assignments

Remove all those if ((ret = foo(bar)) < 0) constructs from upload_firmware()
function. This is just a confusing programming practice, kill it. While at it,
replace all the uses of &devc->ftdic with plain ftdic , which is defined at
the begining.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Fix firmware path construction

The code silently assumed the firmware path can be no longer than 128 bytes.
This doesn't scale. This patch fixes it in such a way that it completely
rips out the run-time computation of firmware path and instead replaces it
with compile-time computation. It's true this makes the library grow by a
couple bytes, but makes the code cleaner.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Cleanup FPGA initialization

Rework the pre-firmware-upload FPGA initialization sequence so it matches
the documentation. Also, since this sequence is documented in a separate
section, wrap it into separate function.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Remove NUM_CHANNELS macro

The NUM_CHANNELS macro is inflexible, since in 100MHz and 200MHz modes
we don't support 16 channels. Moreover, it's only used to limit the size
of array of channel labels, which can be done in much cleaner way.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Document sampling rate table

Add comments to the sampling rate table explaining how the frequencies
are selected and where do those numbers come from.

Signed-off-by: Marek Vasut <redacted>

asix-sigma: Acquisition fixes.

output/vcd: Support getting samplerate from meta packets.

output/gnuplot: Support getting samplerate from meta packets.

output/csv: Support getting samplerate from meta packets.

output/ascii: Support getting samplerate from meta packets.

output/hex: Support getting samplerate from meta packets.

output/bits: Support getting samplerate from meta packets.

session: Show unitsize of the received data.

Don't include NO_LOG_WRAPPERS in the Doxygen output.

Don't include LOG_PREFIX in the Doxygen output.

(files in hardware/, input/, and output/ don't need this since none of
those directories contain API stuff and are thus not included in the
list of input files for Doxygen at all).

configure.ac: Separate library flags from other CFLAGS.

Give type names to all enumerations.

Remove orphaned prototype for sr_filter_channels from proto.h.

fx2lafw: Don't send more samples than requested.

It's the new law.

WIP

output/ascii/bits/hex: Remove stray g_free().

ols: Allow disabling all channels.

This fixed bug 316.

output: Introduce output module API wrappers.

Removed ALSA driver.

This driver has been unmaintained for years, and was never good code
to begin with. It's also questionable whether it was ever useful,
particularly with the demo driver now supporting various analog
signalling.

Removed obsolete sr_filter_channels().

Remove obsolete output API.

output/ascii: Rewrite for new output API.

output/hex: Rewrite for new output API.

output/bits: Rewrite for new output API.

This also fixes the performance problems in the old module.

output/binary: Use new output API.

output/chronovu_la8: Use new output API.

This somewhat naively copies whatever it gets into the output, regardless
of how many channels are in there, or which ones are enabled. Not sure
what the best way to deal with that is, but for now you have to feed it
a channel setup the Chronovu software can read.

output/csv: Use the logic stream's unitsize.

This also fixes the channel index, instead of counting on the logic
data being arranged according to enabled channels only.

output/vcd: Use the logic stream's unitsize.

Also code cleanup and some memory leaks fixed.

output/gnuplot: Use new output API.

.gitignore: Add test-suite.log.

sr_dev_inst_new(): Use NULL (not "") if version unknown/nonexisting.

Unify the whole code-base to use NULL there, not a mix of "" and NULL.

chronovu-la: Show errors other than "device not found".

This can be useful for the user to see, e.g. for permission related
issues (device is there, but cannot be accessed).

configure.ac: Clearly mark required and optional libs.

rigol-ds: fix long memory depth setting on DS1000

scpi_usbtmc_libusb: disable all clear_halt for Rigol DS1000

scpi_usbtmc_libusb: disable bulk_out clear_halt for Rigol DS1000

fix hangup issue with Rigol firmware version 00.02.02.02.00

fx2lafw: fix possible use of uninitialized variable (gcc-4.9 warning fix)

hardware/fx2lafw/protocol.c: In function 'fx2lafw_command_start_acquisition':
hardware/fx2lafw/protocol.c:113:7: warning: 'cmd.flags' may be used uninitialized in this function [-Wmaybe-uninitialized]
   (cmd.flags & CMD_START_FLAGS_CLK_48MHZ) ? "48" : "30");
       ^

Various drivers: Expose SR_CONF_TRIGGER_TYPE.

Without this, sigrok-cli --show (for example) will not show the
supported trigger types.

chronovu-la: LA16 has fewer max. samples than LA8.

Each LA16 sample is 16bits wide (LA8 samples are 8 bits wide). Adapt the
number returned for SR_CONF_LIMIT_SAMPLES in config_list() accordingly.

chronovu-la: Fix byte order for LA16 samples sent to the bus.

chronovu-la: Document that streaming is not possible.

This closes bug #261.

Rename 'chronovu-la8' driver to 'chronovu-la'.

The driver now also supports the LA16 device (and possibly others in the
future).

la8: Add support for the ChronoVu LA16.

The ChronoVu LA16 is a new logic analyzer from ChronoVu with some
differences in features compared to the LA8, e.g.

 - Supports 16 channels (instead of 8).
 - Max. 200MHz samplerate (instead of 100MHz).
 - Supports state triggering (low and high channel value) and edge triggering
   (rising or falling edge), the LA8 only supports state triggering.

This driver now supports both the LA8 and LA16, but it needed a few
changes:

 - Add support for detecting multiple device instances at all.
 - Add support for both LA8 and/or LA16 devices being detected.
 - Add a device profile struct for LA8-/LA16-specific device properties.
 - Move the samplerates list to devc (it's different for LA8 and LA16).
 - Split scan() into two functions, one for scanning, one for adding a device.
 - Expand some variables and fields from uint8_t to uint16_t in order to
   support 16 channels.
 - Update the samplerate related functions to support the LA16's 200MHz.
 - Various other minor updates in order to better handle both device types.
 - Various error handling improvements and simplifications.
 - Also, replace time() with g_get_monotonic_time() everywhere.

This also fixes bug #247 (which was related to incorrect handling of
resources during scan and open of the device, which was exposed by
PulseView allowing multiple consecutive scan/close/open calls).

la8: Cleanups, cosmetics, simplifications.

Also, make some LA8 references more generic in preparation of
LA16 support.

demo: Fix typos.

Various drivers: Drop some unneeded lines.

rigol-ds: Don't close SCPI handle we didn't open.

rigol-ds: Add the TRIGGER_SLOPE entry to config_get() and config_list()

hameg-hmo: Change some PG_/pg_ names to CG_/cg_.

This fixes parts of bug #259.

A few more random 'probe' to 'channel' changes.

This fixes parts of bug #259.

Rename a few more API calls with 'probe' in their name.

This fixes parts of bug #259.

Rename various *PROBE* macros to *CHANNEL*.

This fixes parts of bug #259.

Rename sr_probe_new() to sr_channel_new().

This fixes parts of bug #259.

Replace 'probe' with 'channel' in most places.

Also, consistently use 'ch' for channel variables. This matches how we
consistently use sdi, devc, and so on all over the code-base.

This fixes parts of bug #259.

Rename 'struct sr_probe' to 'struct sr_channel' everywhere.

This fixes parts of bug #259.

Consistently use 'cg' for channel group variables.

This matches how we consistently use sdi, devc, and so on all over
the code-base.

This fixes parts of bug #259.

Replace channel_group.probes with channel_group.channels.

This fixes parts of bug #259.

Replace some 'pg' with 'cg'.

(since 'probe groups' are now called 'channel groups')

This fixes parts of bug #259.

Replace 'probe group' with 'channel group' everywhere.

The name 'probe' (and thus 'probe group') is a relic from the times when
sigrok was mostly about logic analyzers. Nowadays we support a lot more
device types where 'probe' is not really a good term and 'channel' is
much better suited.

This fixes parts of bug #259.

hameg-hmo: Add missing cases in config_get().

hameg-hmo: Add the TRIGGER_SLOPE entry to config_list().

hameg-hmo: Fix the logic for the horizontal trigger position in config_set()

This patch fixes the logic calculating the trigger position as well as the
(hopefully) last remaining locale issue.

serial-dmm: Implemented support for Voltcraft M-3650CR (driver voltcraft-m3650cr).

serial-dmm: Implemented support for Voltcraft ME-42 (driver voltcraft-me42).

Makefile.am: Add missing vxi.h.

If it's not listed as SOURCE, it won't end up in the tarball and a build
from tarball would thus break.

Switch to a non-recursive automake setup.

Instead of >= 44 Makefile.am's we now only have one top-level
Makefile.am, and use the 'subdir-objects' automake option to
handle the build via non-recursive (auto)make.

This has the advantage of fewer (boilerplate or other) files and less
clutter in general, as well as performance advantages since the new
setup can build many files in parallel (with 'make -j'), not only 2 or 3
files within the same (e.g. hardware/xxxx/* subdirectory) and also since
we no longer need to build intermediate libtool helper libs per subdirectory.

A quick, non-scientific test build on a quad-core laptop with 'make -j 4'
yields a build time reduction from 35s to 19s.

All autotools features that worked before are still intact without any
regressions, including the Make targets 'install', 'uninstall', 'check',
'dist', 'clean', 'distclean' and so on, as well as all the usual portability
handling (build works on any OS, with any Make implementation such as
GNU Make or BSD Make, with any shell such as sh/ksh/zsh/bash/dash, etc. etc.)
and features such as out-of-tree build support, cross-compile support,
testsuite support (also with colored output), "silent make rules", etc. etc.

scpi_usbtmc_libusb.c: Fix two error checks.

Two error checks had a missing "ret = ", which lead to an incorrect
value being passed to libusb_error_name().

Also, lower the level for those messages from sr_err() to sr_dbg()
since they're not fatal.

scpi_usbtmc_libusb.c: Consistently check for < 0 (libusb calls).

Fix 'aquisition' typo in a few places.

brymen-bm86x: disable driver when libusb is not present

gmc-mh-1x-2x: Whitespace fixes (cosmetic).

visa: Fixed missing header warnings/uninitialized variable.

gmc-mh-1x-2x: Cosmetic fixes for Metrahit 25S.

gmc-mh-1x-2x: Fixed 300 µA range scaling.

Fluke189 support

I can confirm that the 189 uses the same "QM" command as the 187:
<-- QM\r
--> QM,+03.225 mV AC

Fixed session_driver so that it can run more than one session.

brymen-bm86x: Remove std_dev_clear() wrapper.

configure.ac: Move Brymen BM86X snippet to correct place.

brymen-bm86x: actual driver implementation

brymen-bm86x: Initial driver skeleton.

session: Always read a whole number of samples.

Remove std_dev_clear() wrappers from drivers.

session: Implement dev_clear().