runtc: print errors at more failure points

run_testcase() can fail in many ways and just cause runtc to return 1,
at least now it doesn't do so silently.

Add test for mcs48 decoder

jtag_stm32: Update testcases

Due to bug #1066, the output of all DR/IR bits changed in
the underlying JTAG decoder.

jtag: Fix shifting of registers

As reported in bug #1066, the decoding of IR/DR bits was incorrect.

spiflash: Add an Adesto AT45DB161E test-case.

dcf77: adjust test expectation (PD emits more "invalid" annotations)

The dcf77 decoder recently started emitting "invalid bit timing" and
"Invalid DCF77 bit number" annotations, and does display bit positions
of the 59th and subsequent pulse within a minute.

can: Update tests wrt the fix for bug #1085.

Add a few test-cases for rc_encode.

Add a few sda2506 test-cases.

ir_nec: add test for the optional carrier detect feature

The IR-NEC decoder optionally can handle input streams that still have
the carrier signal. Add a test for this configuration, derived from the
test which uses the signal that had the carrier removed in hardware.

The same data is seen, just at different sample numbers. The content was
verified by means of the "cut -d ' ' -f 2-" and "diff" commands.

graycode: add test for stream without sample rate

Use a smaller input stream which lacks a samplerate spec. This results
in output that lacks the timing information (motion speed), but step
count and direction are identical to the other existing test.

stepper_motor: add test for input stream without sample rate

The stepper motor decoder can still determine the steps' directions
and count. Missing input sample rates just prevent the decoder from
outputting speed annotations. All other decoder output is identical.

stepper_motor: lack of samplerate is not fatal any longer

i2s: add test for stream without sample rate

Use a smaller input stream which lacks a samplerate spec. This results
in identical decoder output (the existing test did not cover throughput
annotations).

i2s: lack of samplerate is not fatal any longer

i2c: add test for input stream without sample rate (identical decoder output)

The existing test did not check the throughput annotation. Which is why
the decoder output is identical for input streams with and without a
sample rate specification.

i2c: lack of samplerate is not fatal any longer

ac97: introduce a basic test with the realtek alc655 snippet

counter: use HD44780 dump to check counter's reset_edge option

Resetting counters depending on the LCD controller's R/S signal yields
numbers that correspond to byte sequences that either "control" hardware
(register access) or carry "data" (display text).

Provide a test case for the rising edge of RESET. Existing cases already
covered the falling edge of RESET which is the default.

counter: adjust test config for renamed 'data_edge' option

parallel: use HD44780 dump to test parallel's word demux feature

Add some counter test-cases.

parallel: adjust hex number format of expected PD output

The parallel decoder got adjusted to emit annotations for "items" and
"words" in a unified format (consistent width, leading zeroes). Adjust
the test expectation to match the PD implementation.

usb_power_delivery: fixup order of combined annotation text fragments

The USB power delivery decoder got adjusted to construct complex
annotations that consisted of several text fragments in a reproducible
order. Adjust the test expectation to match the PD implementation.

Add some graycode test-cases.

maple_bus: Add a simple test-case.

i2s: Update wav output (PD now emits 32bit samples).

Adjust ds2432 test cases to the rename to ds243x

Add a DS2432 test-case.

This fixes (parts of) bug #1019.

usb_{signalling,packet,request}: Add another test-case.

can: catch up with sample number changes after clock sync introduction

The CAN decoder got adjusted to synchronize to the input stream's clock.
Catch up in the test expectation.

onewire_network: Adjust test expectation for sample number fix in the PD

The start sample number of onewire network decoder annotations got fixed
in the decoder implementation (used to start one bit late). Catch up in
the test expectation.

pwm: Adjust binary output test for sample number fix in the PD

The previous implementation used the period counter instead of the
period's actual start and end sample number. The implementation got
fixed, catch up in the test expectation.

dsi: Update due to recent minor PD changes.

pwm: Add a few more test-cases.

spi: Missing samplerate spec is not fatal

Remove the "exception_samplerate" test for the SPI decoder. Lack of a
samplerate specification only breaks the emission of the throughput
annotations, but still allows decoding the clocked signal.

Bump requirements to libsigrok/libsigrokdecode 0.5.0.

Rename eeprom93cxx to eeprom93xx.

Add support for initial pin settings.

Adapt all tests that require it to use the old behaviour of assuming an
initial pin value of 0 (or whatever was previously hardcoded in the PD).
for now (this will likely see some changes later).

swd: Add missing channels.

Fix issue in -s output.

Option values are strings, not numbers.

This fixes:

$ ./decoder/pdtest -s jitter
Testcase: jitter/toim4243
  Protocol decoder: jitter
    Channel clk=3
    Channel sig=2
Error: %d format: a number is required, not str

ds28ea00: Update due to decoder rename.

can: Update test results wrt recent PD changes.

can: Update wrt new PD warnings.

sdcard_sd: add CMD23 test

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

sdcard_sd: correct cmd numbers when prefixing responses

Signed-off-by: Wolfram Sang <wsa@the-dreams.de>

timing: Update tests, add some more.

timing: add tests for noaverage/edge/delta modes

Signed-off-by: Karl Palsson <karlp@tweak.net.au>

args: make pdtest -l behave as advertised.

Help says -l should list tests. Make it do so.
You can still do "-l uart" to see only uart tests, but -l by itself now
does what you expect.

Signed-off-by: Karl Palsson <karlp@tweak.net.au>

pdtest: say that '-f' creates initial output for new tests

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>

Add an eeprom93cxx test-case.

Add a microwire test-case.

sdcard_sd: Update to reflect recent PD changes.

onewire_network: Add more test-cases.

onewire_link: Add more test-cases.

Adjust tests for the new onewire_link decoder

Time slots and reset pulse/presence detect end timestamps were off by 1 us.

Additional warnings have been added for glitches.

Incomplete starting patterns are ignored.

Add a Sensirion SHT3x test-case.

Add a Sensirion SHT2x test-case.

eeprom24xx: Add Xicor X24C02 test-case.

Add a DALI test-case.

Add a DSI test-case.

configure summary: Show linker flags.

runtc: unbreak the protocol decoder output match logic (type vs instance)

Recent libsigrokdecode adjustment changed decoder output. The ID of a
protocol decoder instance need no longer match the decoder type's name
even for simple setups.

Lookup the selected decoder's instance ID during construction of the
decoder stack. Such that the selected decoder's data can get identified
in the output of the stack which is received by runtc(1).

Note that this implementation passes the decoder type's name to the
runtc(1) output file, so that it keeps matching the expected .output
files when pdtest(1) compares those two streams. This approach avoids
the necessity to adjust test.conf or .output files. It will no longer
be appropriate when more complex test configurations shall be supported
which involve multiple instances of the same decoder type or multiple
stacks of decoders.

runtc: fixup coding style (conditionals, braces)

When a conditional has one complex branch (multiple instructions), both
branches are supposed to have braces around them. Instead of putting
extra braces around single instructions, eliminate the "else" branch
since the "if" branch already ends in break or return respectively.

runtc: minor nits in main()

Pass NULL instead of 0 where a pointer is expected. Fixup whitespace.

runtc: minor nits in usage(), add missing options

Adjust the screen layout of usage() output: Align parameters to options
in identical ways, to separate them from descriptions for options which
don't take parameters. Add previously not listed options.

iec: Add a simple test-case.

Use PD ID (not PD instance ID) for now.

This is related to the libsigrokdecode change regarding bug #868.

adf435x: Add an initial test-case.

arm_tpiu: Fix confusing file and test-case naming.

runtc, pdtest: Fixup minor style nits

Address several minor style nits: Remove trailing empty lines in source
files. Declare 'static' auto variables before non-static variables.
Re-use the result of a division instead of running the calculation again.

pdtest: Make exit code communicate the termination cause not counts

On one hand the exit(3) API might limit the range of values that can get
communicated to callers (a single byte is used in some implementations).
On the other hand wait(2) might interpret specific meaning into some of
the returned values (abnormal cause of termination, like unhandled
signals). So the use of error counts as exit codes for processes can be
severely constrained. For specific non-zero values of error counts, an
exit code of zero might be seen at the caller's side (modulo calculation
or truncation).

Don't pass error or difference counts to exit(3). Instead communicate
the kind of termination (error or difference or success) in exit codes
which reliably can get evaluated by callers.

pdtest: Exit non-zero for errors as well as PD output differences

Make the pdtest(1) executable return a non-zero exit code for errors
during test execution (as before) as well as for differences in the
decoder output (that's the change). Return either the number of errors
seen, or the number of tests which saw different output. Only return
zero in the absence of errors and differences.

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.

avr-pdi: Add initial AVR PDI test case

spiflash: Update due to recent PD changes.

spiflash: Enforce chip selection for all test-cases.

ps2: Add an initial PS/2 keyboard test-case.

pdtest: Minor help text improvements.

spi: Add some test-cases for wordsize != 8.

uart: Update test-cases for recent UART PD changes.

uart: Make rxtx_overlapped test-case use format=hex.

uart: Explicitly select a format option for the tests.

uart: Add test-cases for 5n1, 6n1, 7n1, 8n1, 9n1.

Add a test-case for the GPIB decoder.

midi: Add 3 new test-cases, update existing ones.

midi: chromatic_notes in list.py was off by 1 octave

(ex: note 60 is middle C which is C4, not C3)

uart: Add a test-case for overlapping RX/TX bytes.

midi: Update each annotation with multiple strings.

(ranging from long, medium, and short lengths)

Add a lot more MIDI test-cases.

Code coverage increased to 98%, only 5 lines missed. 4 are from
handle_channel_msg_generic().

Add some SSI32 tests.

README: Add missing autogen.sh call.

Add Python 3.5 support.

Add a few DMX512 test-cases.

timing: Update to latest PD changes.

spiflash: Add a few more test-cases for dual I/O reads.

spiflash: Add a few more test-cases.

Add a few em4305 test-cases.

usb_request: Update PCAP files due to recent changes.

See bug #664 for details.

Add wiegand 36bit test