]> sigrok.org Git - libsigrok.git/log
libsigrok.git
4 years agomanson-hcs-3xxx: Support device IDs with and without "HCS-" prefix.
Uwe Hermann [Tue, 24 Mar 2020 20:07:08 +0000 (21:07 +0100)]
manson-hcs-3xxx: Support device IDs with and without "HCS-" prefix.

Since we've now seen lots of devices in the wild that come with the
"HCS-" prefix in the ID, it's probably safe to assume all of them
could have it.

This fixes bug #1530.

4 years agoasix-sigma: Drop duplicate error message prefixes.
Uwe Hermann [Tue, 24 Mar 2020 18:22:11 +0000 (19:22 +0100)]
asix-sigma: Drop duplicate error message prefixes.

The sr_err() call automatically adds a prefix to all messages, in
this specific case "asix-sigma: " will be added.

4 years agoasix-sigma: move DRAM line buffer allocation closer to its use
Daniel Trnka [Thu, 12 Mar 2020 11:08:35 +0000 (12:08 +0100)]
asix-sigma: move DRAM line buffer allocation closer to its use

Move the allocation of the DRAM line buffer in the sample download code
path closer to the location where that buffer is used and gets released.

4 years agoasix-sigma: check for successful register access in sample download
Daniel Trnka [Thu, 12 Mar 2020 11:08:35 +0000 (12:08 +0100)]
asix-sigma: check for successful register access in sample download

The previous implementation got stuck in an infinite loop when data
acquisition started, but the device got disconnected before the data
acquisition terminates. An implementation detail ignored communication
errors, and never saw the expected condition that was required to
continue in the sample download sequence. Unbreak that code path.

4 years agokorad-kaxxxxp: Fix max. possible current for all devices.
Uwe Hermann [Sun, 22 Mar 2020 15:58:54 +0000 (16:58 +0100)]
korad-kaxxxxp: Fix max. possible current for all devices.

Even though the devices/websites/manuals usually say 0..30V, the
hardware actually accepts up to 31V, both via serial as well as by
simply rotating the knob on the device (and our driver already
reflects that).

The same is true for current, it's usually 0..5A as per docs, but many
(probably all) devices accept 5.1A via serial and knob.

Thus, set the max current of all devices to 5.1A (or 3.1A for 3A
devices). We're assuming they all have this property, and we've seen
this in practice on at least three different versions of the device.

4 years agokorad-kaxxxxp: Add a new ID for KORAD KA3005P V4.2 power supply
David Sastre Medina [Sat, 21 Mar 2020 14:22:28 +0000 (15:22 +0100)]
korad-kaxxxxp: Add a new ID for KORAD KA3005P V4.2 power supply

On a recently acquired Korad KA3005P power supply, the ID supplied by the
device is not known by libsigrok.

$ sigrok-cli --driver=korad-kaxxxxp:conn=/dev/ttyACM0 --scan
sr: korad-kaxxxxp: Unknown model ID 'KORAD KA3005P V4.2' detected, aborting.

This fixes bug #1522.

Thanks to bitaround@gmail.com for the amperage fix.

4 years agorigol-ds: Rigol DS1152E-EDU support fix
jirjirjir [Fri, 17 Jan 2020 13:28:55 +0000 (16:28 +0300)]
rigol-ds: Rigol DS1152E-EDU support fix

4 years agoFix #1509 by providing alternate sample format scales
Soeren Apel [Fri, 28 Feb 2020 21:49:57 +0000 (22:49 +0100)]
Fix #1509 by providing alternate sample format scales

4 years agopython: Apply typemap for VariantBase to VariantContainerBase.
Martin Ling [Sun, 5 Jan 2020 18:25:13 +0000 (18:25 +0000)]
python: Apply typemap for VariantBase to VariantContainerBase.

Fixes #1480.

4 years agoagilent-dmm: Add Agilent U1237AX (completely untested).
Uwe Hermann [Fri, 27 Dec 2019 20:22:18 +0000 (21:22 +0100)]
agilent-dmm: Add Agilent U1237AX (completely untested).

4 years agoagilent-dmm: U127x: Fix mode switch event handling.
Uwe Hermann [Fri, 27 Dec 2019 19:59:04 +0000 (20:59 +0100)]
agilent-dmm: U127x: Fix mode switch event handling.

The DMMs report as an event to which mode the user switched (by turning the
rotary switch): "*0", "*1", etc.

Most other DMMs have few modes, but the U127x DMMs have up to 11 different
modes (i.e., "*10" is a valid event).

4 years agoMakefile.am: Add missing src/serial_hid.h.
Uwe Hermann [Tue, 24 Dec 2019 15:39:01 +0000 (16:39 +0100)]
Makefile.am: Add missing src/serial_hid.h.

4 years agosrc/serial_hid.h: Include guard consistency fix.
Uwe Hermann [Tue, 24 Dec 2019 15:38:56 +0000 (16:38 +0100)]
src/serial_hid.h: Include guard consistency fix.

4 years agoinput/csv: Set default "header" option value to true.
Uwe Hermann [Sun, 22 Dec 2019 22:50:22 +0000 (23:50 +0100)]
input/csv: Set default "header" option value to true.

This makes re-opening files that were saved via the libsigrok CSV
output module slightly more convenient.

4 years agooutput/csv: Set default "time" option value to false.
Uwe Hermann [Sun, 22 Dec 2019 22:49:08 +0000 (23:49 +0100)]
output/csv: Set default "time" option value to false.

4 years agomanson-hcs-3xxx: support new HCS-3200 / PPS-13610 model string.
Uwe Hermann [Sun, 22 Dec 2019 22:18:24 +0000 (23:18 +0100)]
manson-hcs-3xxx: support new HCS-3200 / PPS-13610 model string.

This fixes bug #1441.

4 years agomastech-ms6514: Add missing string.h #include.
Uwe Hermann [Sun, 22 Dec 2019 20:06:46 +0000 (21:06 +0100)]
mastech-ms6514: Add missing string.h #include.

4 years agoREADME.devices: Document MASTECH MS6514 interface enabling.
Uwe Hermann [Sun, 22 Dec 2019 14:52:46 +0000 (15:52 +0100)]
README.devices: Document MASTECH MS6514 interface enabling.

4 years agoconfigure.ac: mastech-ms6514: Add serial_comm flag.
Uwe Hermann [Sun, 22 Dec 2019 14:33:45 +0000 (15:33 +0100)]
configure.ac: mastech-ms6514: Add serial_comm flag.

4 years agoInitial support for MASTECH MS6514 thermometer
Dave Buechi [Sat, 17 Aug 2019 21:36:14 +0000 (23:36 +0200)]
Initial support for MASTECH MS6514 thermometer

4 years agoinput/csv: Consistently use a newline before the last return statement.
Uwe Hermann [Sun, 22 Dec 2019 13:42:34 +0000 (14:42 +0100)]
input/csv: Consistently use a newline before the last return statement.

4 years agoinput/csv: style nits, drop @brief and DIAG in debug output
Gerhard Sittig [Sun, 22 Dec 2019 13:11:15 +0000 (14:11 +0100)]
input/csv: style nits, drop @brief and DIAG in debug output

The @brief keyword is not needed since JAVADOC_AUTOBRIEF is enabled in the
Doxygen configuration. Remove a remaining "DIAG" prefix in a debug
message, the output already is filtered according to the log level, and
prefixed by the module name.

4 years agoinput/csv: re-calculate samplerate after file re-import
Gerhard Sittig [Sat, 19 Oct 2019 19:20:36 +0000 (21:20 +0200)]
input/csv: re-calculate samplerate after file re-import

Don't clobber the user provided samplerate (specified by input module
options). This allows to re-determine the samplerate from a potentially
changed file on disk upon reload.

4 years agoinput/csv: expand cleanup (resource release) and unbreak file reload
Gerhard Sittig [Sat, 19 Oct 2019 14:09:15 +0000 (16:09 +0200)]
input/csv: expand cleanup (resource release) and unbreak file reload

The list of previously created channels is kept across file reloads. We
also need to keep (or re-create) the list of channels that are used for
datafeed submission of analog data.

Release more allocated resources in the .cleanup() routine, and do reset
internal state such that a .reset() thus .cleanup() then .receive() call
sequence will work. This code path is taken for re-import of files (see
bug #1241, CSV was affected).

4 years agoinput/csv: improve reliabilty of text line isolation
Gerhard Sittig [Sat, 19 Oct 2019 09:47:45 +0000 (11:47 +0200)]
input/csv: improve reliabilty of text line isolation

Slightly unobfuscate the "end of current input chunk" marker in the data
processing loop. Make the variable's identifier reflect that it's not a
temporary, but instead something worth keeping around until needed again.

Unbreak the calculation of line numbers in those situations where input
chunks (including previously accumulated unprocessed data) happens to
start with a line termination. This covers input files which start with
empty lines, as well as environments with mutli-byte line termination
sequences (CR/LF) and arbitrary distribution of bytes across chunks.

This fixes bug #968.

Accept when there is no line termination in the current input chunk. We
cannot assume that calling applications always provide file content in
large enough chunks to span complete lines. And any arbitrary chunk size
which applications happen to use can get exceeded by input files (e.g.
for generated files with wide data or long comments).

4 years agoinput/csv: update developer comments and TODO list
Gerhard Sittig [Fri, 18 Oct 2019 23:14:29 +0000 (01:14 +0200)]
input/csv: update developer comments and TODO list

Mention the required synchronization of default option values and format
match logic in a prominent location where options are discussed.

Update the TODO list for the CSV input module. Mixed signal handling got
fixed by rearranging channel creation. Samplerate can get determined
from timestamp columns already. The double data type for analog data
remains.

4 years agoinput/csv: move channel creation to after format parsing
Gerhard Sittig [Fri, 18 Oct 2019 23:07:28 +0000 (01:07 +0200)]
input/csv: move channel creation to after format parsing

The previous implementation incompletely handled arbitrary data type
oders in mixed signal input files. Rearrange the logic such that all
format specs get parsed first, then all channel creation details get
determined, then all channels get created. It appears to be essential
that all logic channels get created first, resulting in index numbers
starting at 0 and addressing the correct position in bitfields. The
analog channels get created after all logic channels exist. Adjacent
number ranges for channel types also results in more readable logic in
other locations.

This was tested with -I csv:column_formats=t,2a,l,a,x4,a,-,b3 example
data, and works as expected.

4 years agoinput/csv: add automatic format match support
Gerhard Sittig [Fri, 18 Oct 2019 20:26:22 +0000 (22:26 +0200)]
input/csv: add automatic format match support

Implement .format_match() support in the CSV input module. Simple
multi-column files will automatically load without an "-I csv" spec.
Non-default option values still do require the module selection before
options can get passed to the module.

4 years agoinput/csv: another stab at option help texts
Gerhard Sittig [Fri, 18 Oct 2019 05:48:41 +0000 (07:48 +0200)]
input/csv: another stab at option help texts

Try to balance a compact format and completeness/accuracy of content for
builtin help texts for the CSV input module's options. Assume a technical
audience (this is signal analysis software after all).

Rename a few internal identifiers which help organize the list of options
and their help texts. Too short names became obscure.

4 years agoinput/csv: add support for timestamp columns, auto detect samplerate
Gerhard Sittig [Thu, 17 Oct 2019 23:11:26 +0000 (01:11 +0200)]
input/csv: add support for timestamp columns, auto detect samplerate

Accept 't' format specs for timestamp columns. Automatically derive the
samplerate from input data when timestamps are available and the user
did not provide a rate. Stick with a simple approach for robustness
since automatic detection is easy to override when it fails. This
feature is mostly about convenience.

4 years agoinput/csv: robustness nits in column format dispatching
Gerhard Sittig [Thu, 17 Oct 2019 21:54:02 +0000 (23:54 +0200)]
input/csv: robustness nits in column format dispatching

The previous implementation open coded type checks by comparing an enum
value against specific constants. Which was especially ugly since there
are multiple types which all are logic, and future column types neither
get ignored nor have channels associated with them.

Improve readability and robustness by adding helpers which check classes
(ignore, logic, analog) instead of the multitude of -/x/o/b|l/a variants
within the classes.

Also comment on the order of channel creation, and how to improve it.

4 years agoinput/csv: update comments/helptext for analog input data
Gerhard Sittig [Thu, 17 Oct 2019 20:14:04 +0000 (22:14 +0200)]
input/csv: update comments/helptext for analog input data

Expand the developer comment that's inline in the source file. There is
enough room to explain things. Try to come up with a "one-line" help text
that is precise yet compact, and does inform the user of available format
choices including modifiers. Chances are that longer descriptions start
reducing the usefulness of the help or the visibility of options when
users are in a hurry. Those who care can access the manual.

Mark more options as obsolete, and mention more default values in the
builtin help text. Also tweak a comment on getting channel names from
header lines.

4 years agoinput/csv: work around undesired logic/analog group "bleeding"
Gerhard Sittig [Thu, 17 Oct 2019 19:52:50 +0000 (21:52 +0200)]
input/csv: work around undesired logic/analog group "bleeding"

Support for mixed signal CSV input data is desirable and should be
possible. The current implementation just happens to not fully cope with
arbitrary mixes of data types in columns yet. Add a quick workaround,
but also a TODO item to properly address the topic later.

4 years agoinput/csv: accept user provided analog resolution in column formats
Gerhard Sittig [Thu, 17 Oct 2019 18:53:41 +0000 (20:53 +0200)]
input/csv: accept user provided analog resolution in column formats

Stick with the (arbitrary) default of 3 digits for analog data. Accept
user specified digit counts in the column_formats= option, like "a4".

4 years agoinput/csv: add support for analog input data
Gerhard Sittig [Wed, 16 Oct 2019 22:20:56 +0000 (00:20 +0200)]
input/csv: add support for analog input data

Extend the CSV input module which was strictly limited to logic data so
far. Add support for analog data types. Implement the 'a' column format,
and feed analog data to the session bus.

This implementation feeds data of individual analog channels to the
session bus in separate packets each. This approach was found to work
most reliably, not all recipients support the submission of multiple
samples for multiple channels in a single packet.

A fixed 'digits' value is used. This needs to get addressed later.

Local experiments suggest that the 'double' data type for analog data
can result in erroneous visual presentation (observed with sigrok-cli).
Use 'float' for now, until the issue is understood and got fixed.
Support for double is prepared internally and is easily enabled.

4 years agoinput/csv: address unassorted nits
Gerhard Sittig [Wed, 16 Oct 2019 20:19:24 +0000 (22:19 +0200)]
input/csv: address unassorted nits

Address several minor nits. Eliminate unneeded variables. Update text to
number conversion comments including wildcard handling. Remove empty
lines in init() which used to spill out a set of lines which all do the
same thing (evaluate a set of options) and shall belong together.

4 years agoinput/csv: move channel creation to column processing details creation
Gerhard Sittig [Wed, 16 Oct 2019 20:16:24 +0000 (22:16 +0200)]
input/csv: move channel creation to column processing details creation

Move the creation of logic channels to the location where formats fields
get iterated, and column processing details get derived. This reduces a
lot of redundancy, and simplifies the addition of more data formats.

4 years agoinput/csv: update TODO comments
Gerhard Sittig [Wed, 16 Oct 2019 18:50:03 +0000 (20:50 +0200)]
input/csv: update TODO comments

Update the list of TODO items at the top of the CSV input module's
source. Text line handling (counting line numbers) got fixed. Adding
support for analog channels was prepared, as are timestamp columns.

4 years agoinput/csv: rework user accessible options for consistency
Gerhard Sittig [Wed, 16 Oct 2019 18:40:36 +0000 (20:40 +0200)]
input/csv: rework user accessible options for consistency

Rename the CSV input module's option keywords. To better reflect their
purpose, and for consistency across the rather complex set of options
and how they interact. Rearrange the list of options (not that the order
matters from the outside, but it's good to have during maintenance).

Update builtin help texts which will show up in applications, as well as
the source code comments which discuss these options in greater detail.
Would be nice to have a "general" help text for input modules which is
not tied to one single option, to provide an overview or use examples.
Arrange the option keys, short and long help texts such that the source
better reflects the applications' screen layout. To better support
future maintenance, again.

Consistently separate multi-work keywords for improved readability.
Prefer underscores over dashes for consistency with common keys in
shared infrastructure in other project sources (device options, MQ
items, etc).

4 years agoinput/csv: extend column-formats support for backwards compatibility
Gerhard Sittig [Wed, 16 Oct 2019 15:57:46 +0000 (17:57 +0200)]
input/csv: extend column-formats support for backwards compatibility

Extend the "column-formats" option support in the CSV input module to
also support wildcards and automatic channel count detection. Move the
format string interpretation to the location where the first data line
or the optional header line are seen. Map the simple options (single
column number and channel count, or first column number and optional
channel count) to a format string, to unify internal code paths. Remove
code paths for the previous specific yet limited scenarios.

Rephrase the condition which keeps executing the "initial receive"
phase. The text line termination sequence gets derived from the first
complete text line, but other essential information is only gathered
later, potentially after skipping a large (user specified) amount of
input data. Keep checking for this essential setup data until data or
the header actually were seen, before regular processing of input data
starts.

4 years agoinput/csv: introduce column-formats option (flexible logic import)
Gerhard Sittig [Tue, 15 Oct 2019 21:39:59 +0000 (23:39 +0200)]
input/csv: introduce column-formats option (flexible logic import)

Extend the CSV input module, introduce support for the "column-formats="
option. This syntax can express the previous single- and multi-column
semantics, as well as any arbitrary order of to-get-ignored, and single-
and multi-bit columns in several formats.

The previous "simple" keywords for single and multi column modes still
are in place, it's yet to get determined whether to axe them. Depends on
whether users can handle the format strings for these simple cases.

4 years agoinput/csv: fixup input file line number handling
Gerhard Sittig [Tue, 15 Oct 2019 19:43:55 +0000 (21:43 +0200)]
input/csv: fixup input file line number handling

The previous implementation allowed CSV input files to use any line
termination in either CR only, LF only, or CR/LF format. The first EOL
was searched for and was recorded, but then was not used. Instead any of
CR or LF were considered a line termination. "Raw data processing" still
was correct, but line numbers in diagnostics were way off, and optional
features like skipping first N lines were not effective. Fix that.

Source code inspection suggests the "startline" feature did not work at
all. The user provided number was used in the initial optional search
for the header line (to get signal names) or auto-determination of the
number of columns. But then was not used when the file actually got
processed.

4 years agoinput/csv: unassorted adjustment, mostly "column processing" related
Gerhard Sittig [Tue, 15 Oct 2019 19:11:28 +0000 (21:11 +0200)]
input/csv: unassorted adjustment, mostly "column processing" related

Reduce "state" in the CSV input module's context. Stick with variables
that are local to routines when knowledge of details need not be global.
Really base the processing of a column's input text on the column's
processing information which was gathered in the setup phase.

Rename few identifiers, to explicitly refer to logic channels (the only
currently supported data type of the CSV input module). Cease feeding
logic data to the session bus when there are no logic channels at all
(currently not really an option). Prepare for simpler dispatching of
parse routines should more data types get added in a future version.

Reduce some "clutter" (overly fragmented stuff that should go together
since it forms logical groups and is not really standalone). Address a
few more minor style nits (sizeof() redundancy, "seemingly inverse"
string comparison phrases).

4 years agoinput/csv: improve "channel name from header line" logic
Gerhard Sittig [Tue, 15 Oct 2019 16:27:34 +0000 (18:27 +0200)]
input/csv: improve "channel name from header line" logic

Improve the code paths which determine logic channels' names from an
optional CSV file header line. Strip optional quotes from the column's
input text (re-use a SCPI helper routine for that). Also use the channel
name for multi-bit fields, append [0] etc suffixes in that case. Comment
on the manipulation of input data, which is acceptable since that very
data won't get processed another time in another code path.

4 years agoinput/csv: introduce generic "column processing" support
Gerhard Sittig [Mon, 14 Oct 2019 21:43:11 +0000 (23:43 +0200)]
input/csv: introduce generic "column processing" support

Rephrase the CSV input module's implementation such that generic support
to "process a column" becomes available. All columns of an input file's
text line get inspected, a column can either get ignored, or converted
to logic data. A future version can then remove the current limitations
of single- and multi-column modes (either one single multi-bit cell, or
multiple single-bit cells which must be adjacent).

Combine the bin/oct/hex parse routines into one routine which handles up
to four bits per input number digit with common logic. Availability of
more data than channels (according to user specs) is not fatal.

Drop the counter intuitive "first-channel" option, use "first-column"
instead. Warn when comment leader and column separator are identical
(was silent before, may be unexpected). Extend diagnostics and address
minor readability nits, update comments. Rephrase logic channel name
assignment.

Use simple scalar options to derive generic processing details: Either
'single-column' and 'numchannels' are required, with an optional
'format' spec (resulting in single-column mode). Or 'first-column' with
an optional 'numchannels' (multi-column mode with fixed format, using
all available columns by default). The default is multi-column mode with
one logic channel per column and spanning all columns on a text line.

4 years agoinput/csv: improve robustness of "use header for channel names"
Gerhard Sittig [Mon, 14 Oct 2019 17:33:29 +0000 (19:33 +0200)]
input/csv: improve robustness of "use header for channel names"

Don't clobber the value of the user provided 'header' option. Use a
separate flag to track whether the header line was seen before, or
needs to get skipped when it passes by.

4 years agoinput/csv: move samplerate meta packet to logic data feed submission
Gerhard Sittig [Sun, 13 Oct 2019 15:51:35 +0000 (17:51 +0200)]
input/csv: move samplerate meta packet to logic data feed submission

Move the communication of the samplerate meta packet to the very spot
where logic sample data gets sent. This allows to optionally determine
late the samplerate, potentially from input data instead of user specs.

4 years agoinput/csv: rearrange text to logic data conversion and datafeed
Gerhard Sittig [Sun, 13 Oct 2019 15:33:53 +0000 (17:33 +0200)]
input/csv: rearrange text to logic data conversion and datafeed

Move the helper routines which arrange for the data feed to an earlier
spot, so that references resolve without forward declarations. Rename
routines to reflect that they deal with logic data.

Slightly unobfuscate column text to logic data conversion, and reduce
redundancy. Move sample data preset to a central location.

Rephrase error messages, provide stronger hints as to why the input text
for a conversion was considered invalid.

4 years agoinput/csv: stricter input data test for multi column mode
Gerhard Sittig [Sun, 13 Oct 2019 15:20:16 +0000 (17:20 +0200)]
input/csv: stricter input data test for multi column mode

The previous implementation assumed that in multi-column mode each cell
communicates exactly one bit of input (a logic channel). But only the
first character got tested. Tighten the check, to cover the whole input
text. This rejects fully invalid input, as well as increases robustness
since multi-bit input like "100" was mistaken as a value of 1 before.

4 years agoinput/csv: slightly shuffle text routines, add bin/hex/oct doc
Gerhard Sittig [Sun, 13 Oct 2019 15:14:49 +0000 (17:14 +0200)]
input/csv: slightly shuffle text routines, add bin/hex/oct doc

Add documentation to the bin/hex/oct text parse routines, and move the
bin/hex/oct dispatcher to the location where its invoked routines are.
Stick with a TODO comment for parse_line() to reduce the diff size.

4 years agoinput/csv: unobfuscate text line to column splitting
Gerhard Sittig [Sun, 13 Oct 2019 14:39:51 +0000 (16:39 +0200)]
input/csv: unobfuscate text line to column splitting

The parse_line() routine is rather complex, optionally accepts an upper
limit for the number of columns, but unconditionally assumes a first one
and drops preceeding fields. The rather generic n and k identifiers are
not helpful.

Use the 'seen' and 'taken' names instead which better reflect what's
actually happening. Remove empty lines which used to tear apart groups
of instructions which are strictly related. This organization neither
was helpful during maintenance.

4 years agoinput/csv: trim whitespace after eliminating comments
Gerhard Sittig [Sun, 13 Oct 2019 14:28:39 +0000 (16:28 +0200)]
input/csv: trim whitespace after eliminating comments

Accept when comments are indented, trim the whitespace from text lines
after stripping off the comment. This avoids the processing of lines
which actually are empty, and improves robustness (avoids errors for a
non-fatal situation). Also results in more appropriate diagnostics at
higher log levels.

4 years agoinput/csv: eliminate magic numbers in options declaration
Gerhard Sittig [Sun, 13 Oct 2019 10:35:18 +0000 (12:35 +0200)]
input/csv: eliminate magic numbers in options declaration

The CSV input module has grown so many options, that counting them by
hand became tedious and error prone. Eliminate the magic numbers in the
associated code paths.

This also has the side effect that the set is easy to re-order just by
adjusting the enum, no other code is affected. Help text and default
values is much easier to verify and adjust with the symbolic references.

[ see 'git diff --word-diff' for the essence of the change ]

4 years agoinput/csv: data type nits (sizes, enums)
Gerhard Sittig [Sun, 13 Oct 2019 10:25:28 +0000 (12:25 +0200)]
input/csv: data type nits (sizes, enums)

Use size_t for things that get counted: column indices, channel numbers
(line numbers already used size_t). De-anonymize an enum to avoid 'int'
where it gets referenced. Adjust printf(3) format strings. Get unsigned
values from option lookups (stick with 32bits, should be acceptable for
spreadsheet columns and channel counts).

Address other minor nits while we are here: Also terminate the last item
in an enum declaration. Add a doxygen comment for parse_line(). Rename a
parameter to achieve tabular doc text layout.

4 years agoinput/csv: include section nits
Gerhard Sittig [Sun, 13 Oct 2019 09:58:49 +0000 (11:58 +0200)]
input/csv: include section nits

Rephrase the #include statements in the CSV input module. "config" is
not a system header but is provided by the application source code.
Separate the config and system and application groups (their order is
essential). Alpha-sort the files within their group for simplified
maintenance.

4 years agoinput/csv: add channel list checks for file re-read
Gerhard Sittig [Sun, 28 Oct 2018 07:51:27 +0000 (08:51 +0100)]
input/csv: add channel list checks for file re-read

Do for the CSV input module what commit 08f8421a9e82 did for VCD. Check
the channel list for consistency across re-imports of the same file.
This addresses the CSV part of bug #1241.

4 years agoinput/csv: improve cleanup code path, unbreak re-import
Gerhard Sittig [Sun, 28 Oct 2018 08:36:11 +0000 (09:36 +0100)]
input/csv: improve cleanup code path, unbreak re-import

The cleanup() routine gets invoked upon shutdown, as well as before
re-importing another file. The cleanup() routine must not release
resources which get allocated in the init() routine, as the init()
routine won't run again in the module's lifetime. The cleanup() routine
must void those context fields which get evaluated in the next receive()
calls.

4 years agooutput/vcd: support smaller timescales with higher resolution
Gerhard Sittig [Fri, 12 Oct 2018 10:35:50 +0000 (12:35 +0200)]
output/vcd: support smaller timescales with higher resolution

The previous implementation inspected the input stream's samplerate, and
simply used the next 1kHz/1MHz/1GHz timescale for VCD export. Re-import
of the exported file might suffer from rather high an overhead, where
users might have to downsample the input stream. Also exported data
might use an "odd" timescale which doesn't represent the input stream's
timing well.

Rephrase the samplerate to VCD timescale conversion such that the lowest
frequency is used which satisfies the file format's constraints as well
as provides high enough a resolution to communicate the input stream's
timing with minimal loss. Do limit this scaling support to at most three
orders above the input samplerate, to most appropriately cope with odd
rates.

As a byproduct the rephrased implementation transparently supports rates
above 1GHz. Input streams with no samplerate now result in 1s timescale
instead of the 1ms timescale of the previous implementation.

4 years agooutput/vcd: use larger data type to internally store frequency
Gerhard Sittig [Sun, 21 Oct 2018 13:40:55 +0000 (15:40 +0200)]
output/vcd: use larger data type to internally store frequency

The 'period' member of the VCD output module's context is supposed to
hold frequencies that correspond to the timescale used during export.
An 'int' (in combination with VCD's 1/10/100 constraint) thus would
result in a 1GHz limit, use uint64_t instead to support higher rates.

4 years agooutput/wavedrom: rephrase accumulation of output data
Gerhard Sittig [Sat, 21 Dec 2019 11:31:56 +0000 (12:31 +0100)]
output/wavedrom: rephrase accumulation of output data

Iterate over the received sample set first, before iterating over the
respective sample's number of channels. This avoids redundant extraction
of sampled bits (which saves only little), but also increases locality
of processed data (though string accumulation still may be expensive).

It also adds the future option of RLE compression during accumulation of
output data, which perfectly matches the WaveDrom syntax for repeated
bit patterns.

4 years agooutput/wavedrom: separate data processing logic from init/cleanup
Gerhard Sittig [Sat, 21 Dec 2019 11:27:18 +0000 (12:27 +0100)]
output/wavedrom: separate data processing logic from init/cleanup

Rearrange the order of routines in the wavedrom output module. Keep the
flow of .receive() -> .process_logic() -> .wavedrom_render() in one common
group of routines, which is not disrupted by the .init() and .cleanup()
routines which are kind of boilerplate in the source file. This increases
readability and maintainability.

4 years agooutput/wavedrom: address style nits
Gerhard Sittig [Sat, 21 Dec 2019 10:29:57 +0000 (11:29 +0100)]
output/wavedrom: address style nits

Adjust brace style, use C language comments, drop camel case. Use size_t
for indices and offsets. Unobfuscate the open/close logic of rendered
output. Allocate zero-filled memory, reduce sizeof() redundancy. Don't
SHOUT in the module's .name property.

[ Changes indentation, see 'git diff -w -b' for review. ]

4 years agooutput/wavedrom: introduce a wavedrom output module
Marc Jacobi [Sun, 29 Sep 2019 07:07:47 +0000 (09:07 +0200)]
output/wavedrom: introduce a wavedrom output module

WaveDrom provides a textual description of digital timing diagrams,
using the JSON syntax. See https://wavedrom.com/ for more details.

4 years agohp-3478a: Check for overflow.
Frank Stettner [Tue, 17 Dec 2019 18:55:28 +0000 (19:55 +0100)]
hp-3478a: Check for overflow.

4 years agohp-3478a: Add get/set/list of digits.
Frank Stettner [Tue, 17 Dec 2019 16:59:44 +0000 (17:59 +0100)]
hp-3478a: Add get/set/list of digits.

4 years agohp-3478a: Add get/set/list of measurement ranges.
Frank Stettner [Fri, 13 Dec 2019 17:23:09 +0000 (18:23 +0100)]
hp-3478a: Add get/set/list of measurement ranges.

4 years agoAdd config key SR_CONF_DIGITS.
Frank Stettner [Tue, 17 Dec 2019 13:21:06 +0000 (14:21 +0100)]
Add config key SR_CONF_DIGITS.

4 years agoAdd config key SR_CONF_RANGE.
Frank Stettner [Tue, 17 Dec 2019 13:17:55 +0000 (14:17 +0100)]
Add config key SR_CONF_RANGE.

4 years agobeaglelogic: Fix mismatched printf specifiers.
Michał Janiszewski [Wed, 31 Oct 2018 21:17:20 +0000 (22:17 +0100)]
beaglelogic: Fix mismatched printf specifiers.

4 years agoREADME.devices: Various updates.
Uwe Hermann [Thu, 19 Dec 2019 21:15:43 +0000 (22:15 +0100)]
README.devices: Various updates.

 - Mention missing drivers that require (or not) firmware uploads.

 - Update HID "chip" spec options.

 - Remove rarely useful and hard to maintain lists of drivers requiring
   serial port, HID, Bluetooth, BLE, or network conn specs.

4 years agoREADME: Clarify bluez/libbluetooth name.
Uwe Hermann [Thu, 19 Dec 2019 21:13:51 +0000 (22:13 +0100)]
README: Clarify bluez/libbluetooth name.

4 years agoasix-sigma: comment on trigger/stop position, silence warning
Gerhard Sittig [Thu, 19 Dec 2019 19:30:00 +0000 (20:30 +0100)]
asix-sigma: comment on trigger/stop position, silence warning

Add a comment on the logic which skips the upper 64 bytes of a 512 bytes
chunk in the Asix Sigma's sample memory. Move the initial assignment and
the subsequent update from a value which was retrieved from a hardware
register closer together for awareness during maintenance. Pre-setting a
high position value that will never match when the feature is not in use
is very appropriate.

Adjust the sigma_read_pos() routine to handle triggerpos identically to
stoppos. The test condition's intention is to check whether a decrement
of the position ends up in the meta data section of a chunk. The previous
implementation tested whether a pointer to the position variable ended in
0x1ff when decremented -- which is unrelated to the driver's operation.
It's assumed that no harm was done because the trigger feature is
unsupported (see bug #359).

This silences the compiler warning reported in bug #1411.

4 years agoscpi-dmm: Added minimal support for HP34401A (PR #36)
Adrian Godwin [Tue, 3 Dec 2019 22:25:37 +0000 (22:25 +0000)]
scpi-dmm: Added minimal support for HP34401A (PR #36)

4 years agokorad-kaxxxxp: Add support for RND 320-KD3005P (PR #35)
Peter Åstrand [Sun, 17 Nov 2019 06:27:47 +0000 (07:27 +0100)]
korad-kaxxxxp: Add support for RND 320-KD3005P (PR #35)

Since this model replies with a serial number, truncate before that

4 years agolecroy-xstream: Remove header read (PR #33)
Sylvain Pelissier [Wed, 30 Oct 2019 15:24:32 +0000 (16:24 +0100)]
lecroy-xstream: Remove header read (PR #33)

4 years agofluke-dmm: Fix use-after-free bugs
Andreas Sandberg [Sun, 29 Sep 2019 15:38:17 +0000 (16:38 +0100)]
fluke-dmm: Fix use-after-free bugs

The handler for fluke 18x and 28x DMMs allocates several data
structures on the stack that are used after they have been freed when
creating a data feed packet.

Restructure the code so that all handlers send their own packets. As a
bonus, this avoid a couple of small heap allocations.

4 years agokorad-kaxxxxp: Reword debug output for the status byte.
Frank Stettner [Thu, 11 Apr 2019 09:54:31 +0000 (11:54 +0200)]
korad-kaxxxxp: Reword debug output for the status byte.

4 years agokorad-kaxxxxp: Send META packet when states have changed.
Frank Stettner [Sat, 6 Apr 2019 22:03:49 +0000 (00:03 +0200)]
korad-kaxxxxp: Send META packet when states have changed.

4 years agokorad-kaxxxxp: Fix bug when setting values while acquisition is running.
Frank Stettner [Tue, 9 Apr 2019 12:20:41 +0000 (14:20 +0200)]
korad-kaxxxxp: Fix bug when setting values while acquisition is running.

By separating the variables that holds the get and set values, the output
state, OVP and OCP can now be set while the acquisition is running.
Also some variables are named more clearly.

4 years agordtech-dps: Send META package when states have changed.
Frank Stettner [Wed, 22 May 2019 13:40:08 +0000 (15:40 +0200)]
rdtech-dps: Send META package when states have changed.

4 years agordtech-dps: Handle different current/voltage digits for the various models.
Frank Stettner [Sun, 19 May 2019 10:42:04 +0000 (12:42 +0200)]
rdtech-dps: Handle different current/voltage digits for the various models.

4 years agordtech-dps: Retry sr_modbus_read_holding_registers() up to 3 times.
Frank Stettner [Wed, 1 May 2019 21:18:13 +0000 (23:18 +0200)]
rdtech-dps: Retry sr_modbus_read_holding_registers() up to 3 times.

The communication with the rdtech power supplies is not very reliable,
especially during the start. Because of that, the driver tries to read the
modbus registers up to three times.
Nevertheless there is the chance, that the communication fails.

4 years agordtech-dps: Use SR_MQFLAG_DC only for voltage and current channels.
Frank Stettner [Tue, 9 Apr 2019 17:14:04 +0000 (19:14 +0200)]
rdtech-dps: Use SR_MQFLAG_DC only for voltage and current channels.

4 years agordtech-dps: Synchronize read and write operations.
Frank Stettner [Tue, 9 Apr 2019 17:08:52 +0000 (19:08 +0200)]
rdtech-dps: Synchronize read and write operations.

4 years agoAdd udev rule for Lecroy WaveRunner oscilloscope
Sylvain Pelissier [Mon, 4 Nov 2019 13:00:23 +0000 (14:00 +0100)]
Add udev rule for Lecroy WaveRunner oscilloscope

4 years agools: introduce metadata quirks support, unbreak Logic Shrimp
Gerhard Sittig [Sat, 5 Oct 2019 11:45:49 +0000 (13:45 +0200)]
ols: introduce metadata quirks support, unbreak Logic Shrimp

Introduce quirks support for devices which provide incomplete metadata.
Add conservative logic to unbreak the Logic Shrimp. Amend previously
received information when it was incomplete, but don't interfere if a
future firmware version fixes the issue.

Without this change, the device gets detected but "has zero channels"
and would be unusable. Because when a device provides metadata, these
details are used exclusively, no fallbacks apply.

4 years agostd: Remove call to sr_dev_close from std_serial_dev_acquisition_stop
Sylvain Munaut [Sat, 13 Oct 2018 13:51:15 +0000 (15:51 +0200)]
std: Remove call to sr_dev_close from std_serial_dev_acquisition_stop

There is no reason to close the entire device in acquisition_stop and
this actually breaks pulseview on serial devices using this callback
when running multiple acquisition cycles

This fixes bug #1271.

Signed-off-by: Sylvain Munaut <redacted>
4 years agodemo: Fix memory leak
Marc Schink [Wed, 30 Oct 2019 09:11:34 +0000 (10:11 +0100)]
demo: Fix memory leak

How to reproduce:

 $ G_SLICE=always-malloc G_DEBUG=gc-friendly valgrind --leak-check=full sigrok-cli --scan --driver demo

Signed-off-by: Marc Schink <redacted>
4 years agomooshimeter-dmm: silence compiler warning (memset() prototypes)
Gerhard Sittig [Tue, 29 Oct 2019 16:48:31 +0000 (17:48 +0100)]
mooshimeter-dmm: silence compiler warning (memset() prototypes)

The Mooshimeter driver uses mem*() and str*() library calls. Include the
<string.h> header file to silence compiler warnings.

  ...
  ./src/hardware/mooshimeter-dmm/protocol.c: In function 'lookup_tree_path':
  ./src/hardware/mooshimeter-dmm/protocol.c:275:3: warning: implicit declaration of function 'strchr' [-Wimplicit-function-declaration]
     end = strchr(path, ':');
     ^
  ./src/hardware/mooshimeter-dmm/protocol.c:275:9: warning: incompatible implicit declaration of built-in function 'strchr' [enabled by default]
     end = strchr(path, ':');
           ^
  ...

4 years agobt: apply 20s timeout to BLE connect(2) attempts
Gerhard Sittig [Tue, 29 Oct 2019 16:42:34 +0000 (17:42 +0100)]
bt: apply 20s timeout to BLE connect(2) attempts

Even working BLE devices won't immediately succeed in the first call to
connect(2), which is why the "in progress" phase was added. But absent
peers made the previous implementation try forever, getting stuck in the
sr_bt_connect_ble() call.

Try to balance these two constraints. Do terminate BLE connect attempts
after a generous timeout. When this 20s period passed, there probably is
a configuration error or unresponsive peer. Yet the timeout needs to be
in this ballpark to not erroneously fail for working setups.

4 years agobt/bt_bluez: Adjust some log levels.
Uwe Hermann [Sat, 26 Oct 2019 19:10:32 +0000 (21:10 +0200)]
bt/bt_bluez: Adjust some log levels.

4 years agobt/bt_bluez: Remove some no longer needed verbose log messages.
Uwe Hermann [Sat, 26 Oct 2019 19:08:59 +0000 (21:08 +0200)]
bt/bt_bluez: Remove some no longer needed verbose log messages.

4 years agoREADME: Mention new (optional) libgio dependency.
Uwe Hermann [Sat, 26 Oct 2019 19:30:42 +0000 (21:30 +0200)]
README: Mention new (optional) libgio dependency.

libgio may or may not be part of a larger libglib package, depending on
OS and distro.

4 years agomooshimeter-dmm: Only report successful scan upon working connection.
Uwe Hermann [Fri, 25 Oct 2019 14:58:27 +0000 (16:58 +0200)]
mooshimeter-dmm: Only report successful scan upon working connection.

4 years agoAdd support for the Mooshimeter DMM
Derek Hageman [Tue, 19 Feb 2019 23:26:27 +0000 (16:26 -0700)]
Add support for the Mooshimeter DMM

This adds support for the Mooshim Engineering BLE based Mooshimeter.
Because the meter requires raw BLE packets, the driver uses the BLE
layer directly. Since the meter has no physical way of configuring it,
the actual configuration is set entirely with sigrok device options.

4 years agoFix link errors when compiling with LTO enabled
Stefan Brüns [Sat, 14 Sep 2019 20:13:13 +0000 (22:13 +0200)]
Fix link errors when compiling with LTO enabled

When libsigrok is compiled with link-time-optimization, the linker
stumbles over the section named the same as the function. Whether this
is a compiler/linker bug or a coding error is unknown.

Renaming the special section (to the same as already used on OS X) avoids
the problem.

This fixes bug #1416.

4 years agobuild: unbreak C++ binding compilation with newer Doxygen (1.8.16)
Gerhard Sittig [Sun, 6 Oct 2019 08:08:09 +0000 (10:08 +0200)]
build: unbreak C++ binding compilation with newer Doxygen (1.8.16)

Doxygen version 1.8.16 introduced an issue which breaks the compilation
of sigrok's C++ binding (https://github.com/doxygen/doxygen/issues/7190).
Don't set the FILE_PATTERNS variable at all, instead of assigning an
empty value. This resolves bug #1422.

4 years agofx2lafw: allow for sampling at 48MHz, matching a fw change
Kate J. Temkin [Thu, 22 Aug 2019 01:12:33 +0000 (19:12 -0600)]
fx2lafw: allow for sampling at 48MHz, matching a fw change

4 years agomanson-hcs-3xxx: support new firmware for HCS-3202
Jan Luebbe [Wed, 11 Sep 2019 14:30:19 +0000 (16:30 +0200)]
manson-hcs-3xxx: support new firmware for HCS-3202

A recently bought device seems to use a different model string, but
still speaks the same protocol.

4 years agorigol-ds: Add initial Rigol MSO5000 support.
Sebastian Reichel [Wed, 15 May 2019 20:20:55 +0000 (22:20 +0200)]
rigol-ds: Add initial Rigol MSO5000 support.

This adds basic support for the Rigol MSO5000 series. It has
the same problems as the DS4000 series: Live capture provides
one digital channel per byte. Buffered memory returns the data
compressed (one byte has 8 digital channels), but the banks are
read separately. It's not possible to read uint16.