sigrok-cli.git
12 days agoSet stdout to binary mode on Windows as needed master
Devan Lai [Fri, 11 Oct 2019 03:16:01 +0000 (20:16 -0700)]
Set stdout to binary mode on Windows as needed

On Windows, stdout defaults to text mode, which attempts to perform
newline conversion such that if it sees a bare line feed (hex 0A),
it will insert a carriage return (hex 0D) before the line feed.

For textual output like normal annotation display or ASCII art, this is
desirable, but for binary output, this is likely to garble the output.

On Windows when using stdout as the output file destination or when
using the -B/--protocol-decoder-binary option, change the mode of stdout
to _O_BINARY to disable this newline conversion to ensure that the
binary output is faithfully delivered to stdout without modifications.

On all other platforms, setup_binary_stdout() is a no-op.

This fixes the original issue in bug #1427.

Further follow-up work might be needed to fix the reverse behavior that is
introduced by this change - now textual output sent to stdout on Windows
will only have Unix style line endings, rather than being converted to
Windows style line endings.

5 months ago--list-serial: Reduce indentation level.
Uwe Hermann [Thu, 20 Jun 2019 16:15:44 +0000 (18:15 +0200)]
--list-serial: Reduce indentation level.

This makes some cases of --list-serial output look a bit nicer.

5 months agoshow: support array of floats, unbreaks LCR output frequency listing
Gerhard Sittig [Wed, 1 May 2019 18:59:12 +0000 (20:59 +0200)]
show: support array of floats, unbreaks LCR output frequency listing

5 months agodoc: discuss "colons in conn= specs" in the manpage
Gerhard Sittig [Wed, 1 May 2019 18:41:25 +0000 (20:41 +0200)]
doc: discuss "colons in conn= specs" in the manpage

This should be obvious, but it seems it's not. Extend the paragraph
which introduces conn= specs for USB devices. Mention that colons are
not available within conn= specs because they are used to separate -d
fields from each other (driver name, and options like 'conn').

5 months ago--list-serial: Also mention HID/BT/BLE as possible "ports".
Uwe Hermann [Tue, 4 Jun 2019 17:12:46 +0000 (19:12 +0200)]
--list-serial: Also mention HID/BT/BLE as possible "ports".

5 months agoshow: print list of available serial ports
Gerhard Sittig [Thu, 16 Mar 2017 11:16:58 +0000 (12:16 +0100)]
show: print list of available serial ports

Introduce the --list-serial command line option, which prints the list
of serial ports detected by the sigrok serial.c abstraction.

The code was taken from Carl-Fredrik Sundström's <audio.cf@gmail.com>
message to the mailing list, with minor style fixes applied, and the
option renamed to explicitly mention its "list" nature, not suggesting
it would be a parameter to setup.

6 months agosession: print error message when output file creation fails
Gerhard Sittig [Tue, 16 Apr 2019 18:52:45 +0000 (20:52 +0200)]
session: print error message when output file creation fails

Failure to create an output file from within the sigrok-cli component
previously went unnoticed. Emit an error message to make users aware.

This addresses the sigrok-cli side of bug #1372.

6 months agodoc: update manpage, --show also works with --input-file
Gerhard Sittig [Sat, 20 Apr 2019 10:05:47 +0000 (12:05 +0200)]
doc: update manpage, --show also works with --input-file

6 months agoshow: add support for -i <fn> --show, provide details on input stream
Gerhard Sittig [Fri, 12 Apr 2019 16:50:49 +0000 (18:50 +0200)]
show: add support for -i <fn> --show, provide details on input stream

The combination of an input file spec and --show was not handled before,
and fell through to the case of showing device driver details. Do gather
input stream properties when --show was specified in combination with an
input file, and print those details to stdout.

This implementation does not assume a specific file format, instead the
datafeed callback got modified to watch the packets but not process them
(neither decode nor send data to an output module).

This approach, and the order of option combination checks in main(),
make this implementation work with input files as well as pipes, as well
as with any supported input module or session file, including automatic
format detection.

6 months agosession: make group of 'static' vars more visible
Gerhard Sittig [Fri, 12 Apr 2019 16:41:49 +0000 (18:41 +0200)]
session: make group of 'static' vars more visible

Put the collection of 'static' variables in datafeed_in() to the top of
the declaration block, to raise awareness during maintenance. These are
"semi global" variables with reduced visibility but extended lifetime.

Update a comment while I'm here. Explain what the early "header packet"
check does instead of what it doesn't.

6 months agodoc: weaken and extend the description of the -d option
Gerhard Sittig [Thu, 11 Apr 2019 18:34:34 +0000 (20:34 +0200)]
doc: weaken and extend the description of the -d option

The software implementation supports automatic driver scans, while the
manpage claimed that -d was mandatory. Change the description, suggest
to specify -d for various reasons, but "allow" for the absence of the -d
option as well.

6 months agoadd --dont-scan (-D) command line option
Gerhard Sittig [Thu, 11 Apr 2019 18:12:44 +0000 (20:12 +0200)]
add --dont-scan (-D) command line option

Introduce the -D or --dont-scan command line option. This avoids
automatic driver scans, and exclusively uses what -d specifies.

Some device drivers use non-standard protocol features since their
supported devices don't support the corresponding standard mechanisms.
Which can confuse conforming devices. In these situations it can be
desirable to not scan automatically for available devices, instead do
nothing until callers will specify the -d option.

This change is motivated by but does not resolve bugs #1155 and #1373.

6 months agodoc: update manpage, mention -A and annotation rows
Gerhard Sittig [Fri, 19 Apr 2019 15:02:22 +0000 (17:02 +0200)]
doc: update manpage, mention -A and annotation rows

Extend the manpage section which documents the -A option to select
annotation output. Mention that rows are supported in addition to
classes.

Rephrase the paragraph about multiple decoder specs in a single -A
option. Each of them can select multiple annotation classes, while the
previous documentation strongly suggested that only one class would be
available. [ best viewed with 'git diff -U10' ]

6 months agodecode: accept annotation rows for -A specs
Gerhard Sittig [Fri, 19 Apr 2019 14:50:39 +0000 (16:50 +0200)]
decode: accept annotation rows for -A specs

Extend the logic which parses -A annotation selection specs. Lookup
annotation classes first for backwards compatibility. Then lookup rows
when classes don't match.

Users can mix classes and rows in -A specs as they like. This is a
convenience feature, to eliminate the tedium of spelling out many
classes when a row is to be shown. Which is useful to only see the
"summary", the uppermost protocol layer within a protocol decoder, while
the individual bits on lower layers of that same protocol decoder are
not of interest. It's more accessible to specify

  -A ir_rc5=fields
  -A ac97=slots-out
  -A i2c=addr-data

instead of

  -A ir_rc5=startbit1:startbit2:togglebit-0:togglebit-1:address:command
  -A ac97=slot-out-tag:slot-out-cmd-addr:slot-out-cmd-data:slot-out-03:slot-out-04:slot-out-05:slot-out-06:slot-out-07:slot-out-08:slot-out-09:slot-out-10:slot-out-11:slot-out-io-ctrl
  -A i2c=start:repeat-start:stop:ack:nack:address-read:address-write:data-read:data-write

As an additional benefit row specs remain stable across decoder updates
which introduce new annotation classes on existing rows.

6 months agodecode: rephrase logic which looks up -A specs
Gerhard Sittig [Fri, 19 Apr 2019 14:38:11 +0000 (16:38 +0200)]
decode: rephrase logic which looks up -A specs

The previous logic scanned for specific annotation classes listed in -A,
broke the loop when nothing was found, otherwise registered the matching
annotation for display later on.

Rephrase the logic such that classes get registered when found, and then
advance to the next spec. Only mismatch will "fall through" to the error
path and terminate the setup routine with a fatal error.

This prepares for an optional lookup of annotation rows when class
lookup fails, before a fatal error is returned.

7 months agodecode: nits, whitespace and line length fix
Gerhard Sittig [Fri, 19 Apr 2019 13:03:14 +0000 (15:03 +0200)]
decode: nits, whitespace and line length fix

Address nits in the setup_pd_annotations() routine. Fixup an occurance
of too deep an indentation level (introduced in 790b026162332). Rephrase
a long line while we are here, to cut its length and for consistency
with other "visible annotations" manipulations in other locations.

7 months agodecode: improve readability, use intermediate variables
Gerhard Sittig [Fri, 19 Apr 2019 12:55:52 +0000 (14:55 +0200)]
decode: improve readability, use intermediate variables

Replace repeated complex keyval[] references with open coded indices by
respectively named variables. Also use 'ann_id' for an often referenced
annotation ID text string, for consistency with decoder ID.

7 months agodecode: fix memory leaks in annotation setup error paths
Gerhard Sittig [Fri, 19 Apr 2019 12:50:18 +0000 (14:50 +0200)]
decode: fix memory leaks in annotation setup error paths

7 months agoAdd --list-supported-wiki option.
Uwe Hermann [Fri, 29 Mar 2019 15:01:06 +0000 (16:01 +0100)]
Add --list-supported-wiki option.

This is meant for developers only, to easily update the list of
supported protocol decoders in the sigrok wiki.

8 months agoREADME: Bump libsigrokdecode requirement to >= 0.6.0.
Uwe Hermann [Fri, 15 Mar 2019 13:34:53 +0000 (14:34 +0100)]
README: Bump libsigrokdecode requirement to >= 0.6.0.

We now depend on the decoder tags feature introduced in the
libsigrokdecode 0.6.x series.

8 months agoDecoder --show output: List decoder tags.
Uwe Hermann [Fri, 15 Mar 2019 13:34:19 +0000 (14:34 +0100)]
Decoder --show output: List decoder tags.

9 months agoFix listing of some configuration options.
Guido Trentalancia [Tue, 29 Jan 2019 23:52:41 +0000 (00:52 +0100)]
Fix listing of some configuration options.

The listing of the following configuration options is currently
broken in sigrok-cli (invalid values are displayed):

 - "timebase" in yokogawa-dlm, rohde-schwarz-hameg, hantek-dso,
   lecroy-xstream, siglent-sds and rigol-ds drivers;
 - "vdiv" in hantek-6xxx, yokogawa-dlm, rohde-schwarz-hameg, hantek-dso,
   lecroy-xstream, hung-chang-dso-2100, siglent-sds and rigol-ds drivers;
 - "spl_meas_range" in pce-322a and cem-dt-885x drivers.

Fixes bug #1331.

Signed-off-by: Guido Trentalancia <guido@trentalancia.com>
11 months agodecode: stricter check for integer decoder options
Gerhard Sittig [Sat, 1 Dec 2018 11:26:34 +0000 (12:26 +0100)]
decode: stricter check for integer decoder options

Tighten the check for integer type decoder options. Insist that the
conversion must have consumed all of the input text. Early reject specs
like "1m" or "8.0", but also unusual specs like "1 ". This shall avoid
later confusion when "seemingly accepted" specs won't take effect during
operation.

11 months agodecode: add support for float option data type
Gerhard Sittig [Sat, 1 Dec 2018 11:23:54 +0000 (12:23 +0100)]
decode: add support for float option data type

Accept decoder options where the value is a floating point number. This
fixes bug #1247 (the "unsupported type" issue reported for sigrok-cli).

12 months agomanpage: Clarify annotation classes in -A description.
Uwe Hermann [Wed, 21 Nov 2018 21:34:09 +0000 (22:34 +0100)]
manpage: Clarify annotation classes in -A description.

12 months agomanpage: Fix -A description wrt default annotations shown.
Uwe Hermann [Wed, 21 Nov 2018 21:32:15 +0000 (22:32 +0100)]
manpage: Fix -A description wrt default annotations shown.

12 months agomain: synchronize --get output for MQ with --show output
Gerhard Sittig [Sun, 4 Nov 2018 00:38:42 +0000 (01:38 +0100)]
main: synchronize --get output for MQ with --show output

The --show output used to print human readable text for measured
quantities, --get printed raw numbers for quantity and flags. Adjust
main.c:get_option() to follow the show.c:show_dev_detail() model for
consistency and usability.

12 months agodoc: minor update to -O and -g options in the manpage
Gerhard Sittig [Tue, 23 Oct 2018 17:13:24 +0000 (19:13 +0200)]
doc: minor update to -O and -g options in the manpage

Demonstrate output format options by means of "-O csv:..." which as a
byproduct also demonstrates booleans. Add an example how to combine -g
and -c, configuring an option of a channel group.

13 months agomanpage: Various updates and improvements.
Uwe Hermann [Wed, 8 Aug 2018 15:16:36 +0000 (17:16 +0200)]
manpage: Various updates and improvements.

13 months agodoc: mention lack of decoder signal name matching in manpage
Gerhard Sittig [Wed, 10 Oct 2018 08:13:51 +0000 (10:13 +0200)]
doc: mention lack of decoder signal name matching in manpage

Users may assume that sigrok-cli automatically matches input trace names
to decoder signals like pulseview does, and not be able to learn that
this is not the case. Explicitly state that limitation in the manpage.

13 months agodoc: discuss protocol decoder stacks in the manpage
Gerhard Sittig [Wed, 10 Oct 2018 07:36:13 +0000 (09:36 +0200)]
doc: discuss protocol decoder stacks in the manpage

The previous version mentioned comma-separated lists of decoders, which
is correct but also was easy to miss. Add a discussion and examples on
stacking decoders as well as creating multiple decoder stacks.

13 months agoFix invalid frame limit error reporting
Guido Trentalancia [Sun, 21 Oct 2018 22:15:35 +0000 (00:15 +0200)]
Fix invalid frame limit error reporting

This patch fixes a bug in the sigrok command-line interface error reporting
when an invalid frame limit is selected for the Hameg/Rohde&Schwarz HMO
mixed-signal oscilloscope driver (hameg-hmo).

Signed-off-by: Guido Trentalancia <guido@trentalancia.com>
17 months agoinput: make read from stdin work with the presence of -I specs
Gerhard Sittig [Tue, 15 May 2018 19:18:08 +0000 (21:18 +0200)]
input: make read from stdin work with the presence of -I specs

The previous implementation supported reading from stdin, but only when
-I was not specified (i.e. with auto detectable input formats). Accept
stdin also when -I was specified, i.e. with the full set of supported
input formats (given they can get streamed).

18 months agomain.c: Minor cosmetic fix.
Uwe Hermann [Wed, 16 May 2018 20:09:00 +0000 (22:09 +0200)]
main.c: Minor cosmetic fix.

18 months agoMinor fix that makes Clang Static Analyzer stop complaining
Aleksander Alekseev [Wed, 2 May 2018 12:30:41 +0000 (15:30 +0300)]
Minor fix that makes Clang Static Analyzer stop complaining

18 months agoBugfix in order to make sr_session_stop() actually work.
Uwe Hermann [Wed, 25 Apr 2018 05:53:42 +0000 (07:53 +0200)]
Bugfix in order to make sr_session_stop() actually work.

The sr_session_stop() calls were previously invoked with the session
pointer being NULL, which didn't actually stop the session. This could
lead to samples still being processed and (e.g.) passed to libsigrokdecode
for decoding, even though the session should have been stopped.

Fix this by passing the session pointer along in the
sr_session_datafeed_callback_add() function, so that the callbacks
can make use of it.

19 months agosession: Don't run nop transform unless explicitly requested.
Uwe Hermann [Thu, 19 Apr 2018 19:44:43 +0000 (21:44 +0200)]
session: Don't run nop transform unless explicitly requested.

19 months agooptions: --help output cosmetics.
Uwe Hermann [Thu, 19 Apr 2018 06:16:00 +0000 (08:16 +0200)]
options: --help output cosmetics.

19 months agoinput.c: Rename BUFSIZE to CHUNK_SIZE.
Uwe Hermann [Thu, 19 Apr 2018 06:25:42 +0000 (08:25 +0200)]
input.c: Rename BUFSIZE to CHUNK_SIZE.

19 months agoinput: Increase chunk size from 16KB to 4MB.
Uwe Hermann [Mon, 16 Apr 2018 22:23:19 +0000 (00:23 +0200)]
input: Increase chunk size from 16KB to 4MB.

This not only slightly speeds up loading files, but more importantly leads
to larger chunks being passed to input modules (and later potentially output
modules), some of which can have a large performance penalty for small chunks.

In the specific case of e.g. binary input and an .sr output file this change
can lead to a speedup of 220x or more (due to larger chunks for srzip).

19 months agosigrok-cli.h: Drop unused functions and #defines.
Uwe Hermann [Mon, 16 Apr 2018 22:23:00 +0000 (00:23 +0200)]
sigrok-cli.h: Drop unused functions and #defines.

19 months agoAdd a .desktop file and an icon.
Uwe Hermann [Sat, 7 Apr 2018 17:27:45 +0000 (19:27 +0200)]
Add a .desktop file and an icon.

19 months agomanpage: Don't list specific output formats, mention -L.
Uwe Hermann [Mon, 26 Mar 2018 07:02:56 +0000 (09:02 +0200)]
manpage: Don't list specific output formats, mention -L.

The list is going to be outdated eventually (e.g. "gnuplot" no longer
exists), it's better to just mention that -L can be used to get a list.

23 months agoForce backup analog traces only if needed
Vincent Palatin [Tue, 24 Oct 2017 12:00:29 +0000 (14:00 +0200)]
Force backup analog traces only if needed

With the current conditions, when using the 'srzip' output module with
device generating SR_DF_ANALOG packets, they are processed both by
'srzip' and the backup 'analog' module (as 'out' is always null for
srzip which does its own file operation).
For a moderately fast continuous analog capture (e.g. 2.4MSamples/s),
the overhead of the 'analog' module is important (e.g. 80% of the whole
processing of my workstation) while we are not using its output,
so disable it when the user actually selected an output format/file
but the module doesn't generate text output.

Also remove the second part of the condition as 'opt_output_format'
never seems to be null or at least not in the case where the user has
omitted output parameters.

Tested with a device generating analog packets both with '-o test.sr'
and no output option.

2 years agoUpdate manpage references from -V to -L
yahvuu [Wed, 13 Sep 2017 17:45:50 +0000 (19:45 +0200)]
Update manpage references from -V to -L

Since bug #604, the supported drivers, decoders and input/output formats
are listed by -L|--list-supported.

Update the references in the manpage to -L, accordingly.

2 years agodecode: Always print colon after protocol decoder ID
Gerhard Sittig [Sun, 9 Jul 2017 12:09:31 +0000 (14:09 +0200)]
decode: Always print colon after protocol decoder ID

Remove the "bug compatibility" support. It's assumed that the omission
of the colon after the protocol decoder ID in log level 3 was less of a
design choice, but instead a byproduct of the redundancy in a previous
implementation (multiple code paths that got out of sync).

2 years agodoc: add --protocol-decoder-samplenum option to the manpage
Gerhard Sittig [Mon, 31 Jul 2017 17:32:05 +0000 (19:32 +0200)]
doc: add --protocol-decoder-samplenum option to the manpage

2 years agodecode: Optionally print sample numbers with protocol decoder annotations
Gerhard Sittig [Sun, 9 Jul 2017 11:56:45 +0000 (13:56 +0200)]
decode: Optionally print sample numbers with protocol decoder annotations

Introduce the "--protocol-decoder-samplenum" command line option (no
short form available), which emits sample numbers with textual output
from protocol decoder annotations, regardless of a log level value.

This shall increase usability of sigrok-cli output with external
postprocessors, as the option exclusively adds sample numbers to the
output but does not affect the presence or layout of any other line.

This commit prepends "text" and "meta" annotations with sample number
information. It does not change "binary" output.

2 years agodecode: Rework logic for text output of protocol decoder annotations
Gerhard Sittig [Sun, 9 Jul 2017 11:44:39 +0000 (13:44 +0200)]
decode: Rework logic for text output of protocol decoder annotations

Fold the previous three code paths which formatted text output for
annotations into one code path, which inspects the flags that were
determined before the actual output, and then always formats the fields
in identical ways regardless of why a field is shown.

Remove the assert checks that were introduced in a previous commit. The
logic for the field selection was proven to be identical to the previous
behaviour.

2 years agodecode: Prepare to rework text output for protocol decoder annotations
Gerhard Sittig [Sun, 9 Jul 2017 11:10:31 +0000 (13:10 +0200)]
decode: Prepare to rework text output for protocol decoder annotations

When the current implementation receives a protocol decoder annotation,
one out of several code paths prints one of several different variations
of text layout. The current implementation exclusively checks the active
log level to determine the layout variant.

Prepare to rework that text layout logic. Separate the determination of
which fields to display from the logic which does emit those fields in a
specific format.

This commit does not yet adjust the print statements. It is supposed to
verify that the "which fields to display?" logic results in identical
behaviour to the previous implementation.

2 years agodecode: Nit, avoid 'class' identifier for local variable
Gerhard Sittig [Sun, 9 Jul 2017 08:50:42 +0000 (10:50 +0200)]
decode: Nit, avoid 'class' identifier for local variable

2 years agosession: Adjust scan for non-demo device
Gerhard Sittig [Sat, 17 Jun 2017 16:19:34 +0000 (18:19 +0200)]
session: Adjust scan for non-demo device

The "demo?" condition is a property of the driver not a device instance.
Adjust the check accordingly, by not passing the 'sdi' when retrieving
the list of options. Rename the involved variable to reflect its purpose.
Adjust an error message text.

This fixes bug #848.

2 years agoHACKING: Prefer git pull requests over mailing list patches.
Uwe Hermann [Tue, 13 Jun 2017 15:08:21 +0000 (17:08 +0200)]
HACKING: Prefer git pull requests over mailing list patches.

Also, drop the reference to gitorious.org (no longer available).

2 years agoconfigure.ac: Bump package version to 0.8.0.
Uwe Hermann [Tue, 13 Jun 2017 20:52:53 +0000 (22:52 +0200)]
configure.ac: Bump package version to 0.8.0.

0.8.0 will be the next major release. Bump now, so that
there is no confusion of tarball 0.7.0 and 0.8.0-git snapshots.

2 years agoNEWS: Update for upcoming 0.7.0 release.
Uwe Hermann [Thu, 8 Jun 2017 18:28:02 +0000 (20:28 +0200)]
NEWS: Update for upcoming 0.7.0 release.

2 years agoFix various clang warnings. sigrok-cli-unreleased
Uwe Hermann [Thu, 8 Jun 2017 21:16:18 +0000 (23:16 +0200)]
Fix various clang warnings.

../show.c:668:30: warning: variable 'q' is uninitialized when used here [-Wuninitialized]
                                        s = sr_period_string(p, q);
                                                                ^
../show.c:324:30: note: initialize the variable 'q' to silence this warning
        const uint64_t *uint64, p, q, low, high;
                                    ^
                                     = 0
../show.c:668:27: warning: variable 'p' is uninitialized when used here [-Wuninitialized]
                                        s = sr_period_string(p, q);
                                                             ^
../show.c:324:27: note: initialize the variable 'p' to silence this warning
        const uint64_t *uint64, p, q, low, high;
                                 ^
                                  = 0
../show.c:458:58: warning: variable 'high' is uninitialized when used here [-Wuninitialized]
                                printf("    Maximum number of samples: %"PRIu64"\n", high);
                                                                                     ^~~~
../show.c:324:41:   CC       input.o
note: initialize the variable 'high' to silence this warning
        const uint64_t *uint64, p, q, low, high;
                                               ^
                                                = 0
../show.c:589:33: warning: variable 'low' is uninitialized when used here [-Wuninitialized]
                                printf("%"PRIu64"-%"PRIu64, low, high);
                                                            ^~~
../show.c:324:35: note: initialize the variable 'low' to silence this warning
        const uint64_t *uint64, p, q, low, high;
                                         ^
                                          = 0

2 years agoHACKING: Update URL to Linux kernel coding style.
Uwe Hermann [Tue, 6 Jun 2017 12:45:37 +0000 (14:45 +0200)]
HACKING: Update URL to Linux kernel coding style.

2 years agoREADME: Drop trailing whitespace.
Uwe Hermann [Tue, 6 Jun 2017 12:05:12 +0000 (14:05 +0200)]
README: Drop trailing whitespace.

2 years agoFix build when using --without-libsigrokdecode.
Uwe Hermann [Tue, 6 Jun 2017 09:15:45 +0000 (11:15 +0200)]
Fix build when using --without-libsigrokdecode.

2 years agoInstaller: Whitespace consistency fixes.
Uwe Hermann [Mon, 5 Jun 2017 15:43:32 +0000 (17:43 +0200)]
Installer: Whitespace consistency fixes.

2 years agoInstaller: Add "sigrok-cli" to uninstaller/Zadig links.
Soeren Apel [Mon, 5 Jun 2017 15:16:30 +0000 (17:16 +0200)]
Installer: Add "sigrok-cli" to uninstaller/Zadig links.

Windows 10 merges all links of the same vendor ("sigrok" here) into one
section in the Start menu. If PulseView and sigrok-cli are installed,
there will be two "Uninstall" links and the user will not know which one
uninstalls which program.

2 years agoInstaller: Place set of example .sr files in separate section
Soeren Apel [Mon, 5 Jun 2017 15:13:21 +0000 (17:13 +0200)]
Installer: Place set of example .sr files in separate section

2 years ago--list-supported: Show detailed library/feature info.
Uwe Hermann [Sat, 3 Jun 2017 15:17:04 +0000 (17:17 +0200)]
--list-supported: Show detailed library/feature info.

2 years agoList supported devices, modules, decoders via --list-supported.
Uwe Hermann [Sat, 3 Jun 2017 15:03:44 +0000 (17:03 +0200)]
List supported devices, modules, decoders via --list-supported.

Move this data from -V to -L|--list-supported for usability reasons.

This fixes bug #604.

2 years agoBump requirements to libsigrok/libsigrokdecode 0.5.0.
Uwe Hermann [Wed, 31 May 2017 21:09:57 +0000 (23:09 +0200)]
Bump requirements to libsigrok/libsigrokdecode 0.5.0.

We use various new API calls from the upcoming 0.5.0 releases and are
not compatible with the respective 0.4.x anymore.

2 years agoshow: Also print the possible decoder input/output IDs.
Uwe Hermann [Wed, 31 May 2017 21:00:39 +0000 (23:00 +0200)]
show: Also print the possible decoder input/output IDs.

2 years agooptions: expand builtin help, discuss samples/time/continuous constraint
Gerhard Sittig [Tue, 23 May 2017 17:41:41 +0000 (19:41 +0200)]
options: expand builtin help, discuss samples/time/continuous constraint

It might be surprising when the sigrok-cli(1) program does nothing (yet
spends user noticable time) before quitting while it spews the --help
text to the screen. Expand the builtin help text, explicitly mention
that either of --samples or --frames or --time or --continuous are
essentially required for data acquisition.

2 years agoUpdate sr_period_string usage
Soeren Apel [Sat, 25 Feb 2017 21:56:36 +0000 (22:56 +0100)]
Update sr_period_string usage

2 years agoconfigure summary: Show linker flags.
Uwe Hermann [Sun, 5 Mar 2017 16:08:54 +0000 (17:08 +0100)]
configure summary: Show linker flags.

2 years agoShow annotations based on decoder ID
Karl Palsson [Tue, 29 Nov 2016 16:21:30 +0000 (16:21 +0000)]
Show annotations based on decoder ID

Originally, decoder ID was equal to decoder instance ID, so it didn't
matter what you were comparing with. However, with distinct decoder
instances, this assumption fails. Because instance IDs can't be known
in advance, and because it would make the CLI arguments even more
complex, we simply let the -A option continue to be per decoder, not per
instance. This maintains compatibility with prior behaviour, but
doesn't allow the full flexibility possible with the library.

Behaviour preserved:
Single PD, no -A flag => all annotations from PD
Single PD, -A pdid => all annotations from PD
Single PD, -A pdid=annX => annX from PD

Behaviour now added:
Duplicate PDs, no -A flag => all annotations from all PDs
Duplicate PDs, -A pdid => all annotations from all PDs of type pdid
Duplicate PDs, -A pdid=annX => all annX from all PDs of type pdid

This fixes parts of bug #868.

Signed-off-by: Karl Palsson <karlp@etactica.com>
2 years agovalgrind: Clear more unfreed memory issues
Karl Palsson [Tue, 29 Nov 2016 14:32:29 +0000 (14:32 +0000)]
valgrind: Clear more unfreed memory issues

Command line used for checking:

  $ valgrind -v --leak-check=full ./sigrok-cli \
    -i rockband_wireless_keyboard_wii/midi_key4.sr \
    -P uart:baudrate=31250,midi -P uart:baudrate=31250,midi

Signed-off-by: Karl Palsson <karlp@etactica.com>
2 years agoInclude decoder instance ID in normal outputs
Karl Palsson [Tue, 29 Nov 2016 13:19:46 +0000 (13:19 +0000)]
Include decoder instance ID in normal outputs

Without this, it's nigh impossible to determine what lines of output are
from what decoder. While -l 4 and above will include this information,
this feels like something that is essential even at the basic level.

Old:
System Realtime: active sensing
System Realtime: active sensing
System Realtime: active sensing
UNHANDLED DATA: 0x00
Channel 1: note on (note = 51 'D#3', velocity = 70)
System Realtime: active sensing
System Realtime: active sensing
System Realtime: active sensing
System Realtime: active sensing
UNHANDLED DATA: 0x00
Channel 1: note on (note = 51 'D#3', velocity = 70)
System Realtime: active sensing

New:

midi1: System Realtime: active sensing
midi1: System Realtime: active sensing
midi1: System Realtime: active sensing
midi1: UNHANDLED DATA: 0x00
midi1: Channel 1: note on (note = 51 'D#3', velocity = 70)
midi1: System Realtime: active sensing
midi2: System Realtime: active sensing
midi2: System Realtime: active sensing
midi2: System Realtime: active sensing
midi2: UNHANDLED DATA: 0x00
midi2: Channel 1: note on (note = 51 'D#3', velocity = 70)
midi2: System Realtime: active sensing

This fixes parts of bug #868.

Signed-off-by: Karl Palsson <karlp@etactica.com>
2 years agoSupport multiple protocol decoder stacks
Karl Palsson [Fri, 25 Nov 2016 16:28:17 +0000 (16:28 +0000)]
Support multiple protocol decoder stacks

Requires libsigrokdecode changes to actually work.
This allows multiple -P options to be specified, each of which is
registered as a full stack with libsigrokdecode.

At this point, there's no way to specify annotations per instance, the
annotations are simply global per decoder (as before) rather than per
instance.

This fixes parts of bug #868.

Signed-off-by: Karl Palsson <karlp@etactica.com>
2 years agoDrop -S stack option
Karl Palsson [Thu, 24 Nov 2016 12:29:09 +0000 (12:29 +0000)]
Drop -S stack option

The -P option already stacks. The -S option was only used for changing the
stacking order. Simply require the correct order with -P. This reduces
the confusion in stacking.

Signed-off-by: Karl Palsson <karlp@etactica.com>
2 years agosigrok-cli_cross.nsi.in: Use Python 3.4 (Windows XP support).
Uwe Hermann [Fri, 17 Feb 2017 08:24:18 +0000 (09:24 +0100)]
sigrok-cli_cross.nsi.in: Use Python 3.4 (Windows XP support).

The last Python version to officially support Windows XP was 3.4.x.

We'll keep the Windows installers at that version for the time being,
until Windows XP support is no longer feasible (e.g. because important
sigrok requirements such as Qt, glib, or libusb drop XP suppport).

2 years agosigrok-cli_cross.nsi.in: Bump Python version to 3.6.
Uwe Hermann [Thu, 2 Feb 2017 21:26:52 +0000 (22:26 +0100)]
sigrok-cli_cross.nsi.in: Bump Python version to 3.6.

2 years agolicense: remove FSF postal address from boiler plate license text
Gerhard Sittig [Thu, 20 Oct 2016 07:49:00 +0000 (09:49 +0200)]
license: remove FSF postal address from boiler plate license text

Remove the FSF postal address as it might change (it did in the past).
Reference the gnu.org website instead which is more stable.

3 years ago--help output: More specific --show help text.
Uwe Hermann [Thu, 20 Oct 2016 21:09:24 +0000 (23:09 +0200)]
--help output: More specific --show help text.

Mention that --show also works for showing input/output format options,
as well as protocol decoder options.

This fixes bug #851.

3 years agoshow: print binary classes as well in protocol decoder details
Gerhard Sittig [Sun, 16 Oct 2016 16:25:30 +0000 (18:25 +0200)]
show: print binary classes as well in protocol decoder details

The output of `sigrok -P <decoder> --show` omitted the binary classes,
users could not learn which specs are available for the -B option.  Add
a "Binary classes:" section after the list of annotations.

Signed-off-by: Gerhard Sittig <gerhard.sittig@gmx.net>
3 years agoconfigure.ac: Bump package version to 0.7.0.
Uwe Hermann [Mon, 15 Aug 2016 12:52:18 +0000 (14:52 +0200)]
configure.ac: Bump package version to 0.7.0.

0.7.0 will be the next major release. Bump now, so that
there is no confusion of tarball 0.6.0 and 0.7.0-git snapshots.

3 years agoNEWS: Update for upcoming 0.6.0 release. sigrok-cli-0.6.x sigrok-cli-0.6.0
Uwe Hermann [Fri, 29 Jan 2016 20:51:13 +0000 (21:51 +0100)]
NEWS: Update for upcoming 0.6.0 release.

3 years agoUpdate API calls due to recent libsigrok renames.
Uwe Hermann [Thu, 31 Dec 2015 17:55:29 +0000 (18:55 +0100)]
Update API calls due to recent libsigrok renames.

 - sr_driver_scan_options() -> sr_driver_scan_options_list()

 - sr_dev_config_capabilities() -> sr_dev_config_capabilities_list()

4 years agoUpdate for configuration API changes.
Martin Ling [Tue, 3 Nov 2015 11:42:30 +0000 (11:42 +0000)]
Update for configuration API changes.

4 years agoinput: Use own main loop instead of sr_session_run()
Daniel Elstner [Fri, 9 Oct 2015 16:45:03 +0000 (18:45 +0200)]
input: Use own main loop instead of sr_session_run()

Again, this is mainly for demonstration purposes, and to make
sigrok-cli consistently use the same "modern" API everywhere.

4 years agobuild: Require GLib 2.32.0
Daniel Elstner [Fri, 9 Oct 2015 16:33:14 +0000 (18:33 +0200)]
build: Require GLib 2.32.0

This is needed for the G_SOURCE_REMOVE symbolic constant. Although
it would be trivial to make do without it, there is no reason not to
bump the requirement since libsigrok depends on GLib 2.32.0 anyway.

4 years agoanykey: Use GLib I/O channel watch
Daniel Elstner [Fri, 9 Oct 2015 16:26:00 +0000 (18:26 +0200)]
anykey: Use GLib I/O channel watch

Watch for input on a GLib I/O channel using the standard GLib main
loop API instead of injecting an FD event source into the sigrok
session loop.

Also run the session in a main loop created by sigrok-cli instead
of using sr_session_run(). This is mainly for demonstration; the
GLib I/O watch change would work even with sr_session_run() since
it is using the same main context in this case.

4 years agooptions: Type of G_OPTION_ARG_NONE is gboolean
Daniel Elstner [Fri, 9 Oct 2015 15:22:21 +0000 (17:22 +0200)]
options: Type of G_OPTION_ARG_NONE is gboolean

Change mistaken gchar * variables to gboolean type.

4 years agoUse g_strerror() in favor of strerror().
Uwe Hermann [Wed, 7 Oct 2015 00:05:00 +0000 (02:05 +0200)]
Use g_strerror() in favor of strerror().

4 years agowindows: Install data files into share subdirectory
Daniel Elstner [Sat, 3 Oct 2015 15:50:28 +0000 (17:50 +0200)]
windows: Install data files into share subdirectory

This allows these files to be found on standard paths relative
to the executable location.

4 years agoMakefile.am: Shorten ChangeLog target a bit.
Uwe Hermann [Mon, 14 Sep 2015 15:31:59 +0000 (17:31 +0200)]
Makefile.am: Shorten ChangeLog target a bit.

4 years agoBuild: Move _POSIX_C_SOURCE definition to config.h
Daniel Elstner [Sun, 13 Sep 2015 20:45:24 +0000 (22:45 +0200)]
Build: Move _POSIX_C_SOURCE definition to config.h

Do not redefine it though when already set, so that it can be
overridden by the user, or indirectly by the compiler settings.

4 years agoBuild: Include <config.h> first in all source files
Daniel Elstner [Sun, 13 Sep 2015 20:42:51 +0000 (22:42 +0200)]
Build: Include <config.h> first in all source files

Since Autoconf places some important feature flags only into the
configuration header, it is necessary to include it globally to
guarantee a consistent build.

4 years agoUse PRIu64 instead of %ld for uint64_t printing.
Uwe Hermann [Sat, 12 Sep 2015 15:06:43 +0000 (17:06 +0200)]
Use PRIu64 instead of %ld for uint64_t printing.

4 years agoClean up .gitignore
Daniel Elstner [Sat, 5 Sep 2015 22:51:10 +0000 (00:51 +0200)]
Clean up .gitignore

4 years agoBuild: Adopt modernized Autotools build system
Daniel Elstner [Sat, 5 Sep 2015 21:12:09 +0000 (23:12 +0200)]
Build: Adopt modernized Autotools build system

4 years agoBuild: Add copy of sigrok.m4 from libsigrok
Daniel Elstner [Sat, 5 Sep 2015 21:08:51 +0000 (23:08 +0200)]
Build: Add copy of sigrok.m4 from libsigrok

4 years agodevice: Use g_ascii_strcasecmp() for portability
Daniel Elstner [Sat, 5 Sep 2015 21:20:21 +0000 (23:20 +0200)]
device: Use g_ascii_strcasecmp() for portability

4 years agoFree trigger after acquisition session.
Bert Vermeulen [Tue, 8 Sep 2015 12:26:22 +0000 (14:26 +0200)]
Free trigger after acquisition session.

4 years agoNSIS installer fix for Windows
Daniel Elstner [Wed, 2 Sep 2015 13:55:41 +0000 (15:55 +0200)]
NSIS installer fix for Windows

4 years agoSupport SR_T_MQ.
Bert Vermeulen [Sun, 23 Aug 2015 21:21:14 +0000 (23:21 +0200)]
Support SR_T_MQ.