configure.ac: Bump package version to 0.4.1.

configure.ac: Bump libtool/library version from 3:0:0 to 3:1:0.

The libtool current:revision:age numbers change from 3:0:0 to 3:1:0.

Details:
http://www.gnu.org/software/libtool/manual/libtool.html#Updating-version-info

This changes the library filename (e.g. on Linux) from
libsigrokdecode.so.3.0.0 to libsigrokdecode.so.3.0.1, the SONAME
(+symlink) stays at libsigrokdecode.so.3, though.

Doxyfile: Set version to 0.4.1.

NEWS: Add list of user-visible changes so far.

configure.ac: Also check for Python 3.6.

build support: sort list of installed items, factor out pretty printer

Alpha-sort the list of installed items, to support developers which scan
the list for newly added items. Factor out the logic which pretty prints
installed items while maintaining a maximum screen output line length.

valgrind: free channels

Signed-off-by: Karl Palsson <redacted>

valgrind: safely iterate lists

Old versions triggered valgrind errors as the API method to remove an
entry modifies the list that was being iterated.

Signed-off-by: Karl Palsson <redacted>

Print error messages when decoders fail load time checks

Several checks get applied when loading externally provided protocol
decoders. Print error messages when checks fail, so that developers can
better resolve issues.

The sequence of tests terminates upon the first failed check, while
decoders may suffer from several issues at the same time. This is
considered acceptable, as it reduces the commit's size and the code's
complexity. This commit only adds error messages, and does not change
logic/behaviour.

Failed API version checks result in two messages: One specific message
which reflects the decoder's version and what's supported by the loader,
and a generic message that the API version check has failed. This is
done to simplify the logic of a rarely used error code path.

This commit addresses libsigrokdecode Bug 704, and allows to identify
decoders from parallel installations of differing version.

This fixes bug #704.

Signed-off-by: Gerhard Sittig <redacted>

srd_inst_decode(): Return SRD_ERR_ARG if unitsize is 0.

srd_inst_decode(): Make the code API version dependent.

Factor out srd_decoder_apiver().

srd_inst_decode(): Improve comments and log messages.

Also, drop @since tag (as the function is not public).

srd_session_send(): Improve Doxygen comments.

Move common/ directory into decoders/.

This simplifies cross-platform handling of the Python/decoder paths
and module installation/dist a bit and also fixes bug #804.

Supply metadata to stacked decoders

Currently only toplevel decoders receive the samplerate, thus stacked
decoders are not able to derive e.g. timestamps from the sample number.

This fixes bug #664.

Add common directory for shared modules.

Backport current PDs from git mainline.

Doxyfile: Set version to 0.4.0.

Bump libtool version (not package version) to 3:0:0.

The last release (0.3.0) had the libtool version (current:revision:age)
set to 2:0:0. Since this release removes and changes interfaces, the new
version is 3:0:0.

http://www.gnu.org/software/libtool/manual/libtool.html#Updating-version-info

This changes the library filename (e.g. on Linux) from
libsigrokdecode.so.2.0.0 to libsigrokdecode.so.3.0.0, the SONAME
(+symlink) becomes libsigrokdecode.so.3.

NEWS: Update for upcoming 0.4.0 release.

configure.ac: Also check for Python 3.5.

This fixes bug #739.

uart: Optimize handling of samples when tx and rx are both idle

Re-enable the fast path for identical samples but only when both
pins are waiting for the start bit. For sparse data sets (I tested
UT61E capture log) the optimization results in a >4x decode
improvement.

arm_itm: Use objdump instead of addr2line.

This improves speed and fixes bug 564. Code is somewhat
of a copypaste from the arm_etm counterpart, but there
are enough differences that it is easier to keep separate
copy of the objdump parsing in each.

usb_power_delivery: Raise SamplerateError instead of Exception.

(for consistency with the other PDs)

Use self.out_binary naming consistently across all PDs.

Use a Python list (not tuple) for OUT_BINARY.

This is more consistent with annotation syntax and looks slightly
better in most cases.

Drop obsolete support for annotation tuples (only support lists).

None of the PDs use this.

em4100: Simplify the code a bit.

em4100: Use ss/es naming for consistency across PDs.

em4100: Eliminate some unnecessary int()s.

em4100: Change annotation setup a bit, decode more protocol details.

Add RFID EM4100 protocol decoding

usb_power_delivery: Use "if not self.samplerate" form.

Use the "if not self.samplerate" form, which catches both the case
where self.samplerate is None, as well as the case where it is 0.

This is also consistent with all the other PDs.

usb_power_delivery: convert double quotes to single quotes

For consistency with other decoders, convert all the double quotes around
strings to single quotes.

Signed-off-by: Vincent Palatin <redacted>

init: Don't let Python override signal handlers

This fixes bug #461.

usb*: Minor cosmetics.

usb_signalling: detect PREamble PID

PID decoding is normally done in the packet layer, but the signalling
layer has to switch behaviour on detection of an PREamble PID.

usb_signalling: add signaling states needed after LS PREamble PID

usb_{packet,request}: handle PREamble transmissions

usb_signalling: fix SOP detection after an error condition

If the oldsym is not set to 'J' before entering IDLE state, the SOP
detection will never trigger

usb_packet: handle errors from usb_signalling

If a packet had any errors, the usb_signalling PD will not send an EOP
but an ERR message. Wait for the next SOP in both cases.

usb_request: Handle transmission timeouts

If a device or host did not receive a handshake 18 bit times after the
EOP, there was a transmission error and the host may repeat the
transmission

usb_signalling: add option to automatically set signalling speed

In automatic mode, the bus is assumed to be in IDLE state. After a
RESET, the bus state is checked again.

usb_signalling: decode RESET and Keep-Alive signalling conditions

Resets are at least 10ms at the root hub downstream facing port and
2.5us at the device (root hub reset may be shortened by itermediate
hubs).
Keep-alive is a low-speed only signalling condition, as low-speed has no
SOFs to inhibit devices to enter suspend

usb_signalling: use explicit positions for packet start/end

Instead of centering the block around the sampleposition, which shows
some visual glitches due to rounding, use the edge positions already
known. Remove unused halfbit symbol.

usb_signalling: handle symbol errors in EOP state

Only SE0 and J are valid symbols during EOP

usb_signalling: remove unneeded syms array

Do not call get_eop() from get_bit(), but directly issue the symbol. As
get_eop() is only called during the GET EOP state, the SE0 is implicit and
there is no need to save it into the syms array.

usb_signalling: remove unused ss_sop member

usb_signalling: detect bit stuffing errors

More than six consecutive 1's are an error

usb_signalling: consolidate symbol output

No need to replicate the symbol formatting in several places

spi: Add a 'TRANSFER' output type.

Fix documentation comment in SPI decoder

usb_power_delivery: Minor typo and whitespace fixes.

usb_power_delivery: Use the same license header format as other PDs.

USB Power Delivery protocol decoder

Decode and packetize the Biphase Mark Coding (aka differential Manchester)
as used in the Universal Serial Bus Power Delivery Specification Revision 2.0
v1.1, then decode the packet content.

Signed-off-by: Vincent Palatin <redacted>

spiflash: Fix incorrect 'inputs' field.

(this PD is indeed stacked on top of SPI)

Initial attempt at MAX7219/MAX7221 decoder

srd_decoder_unload(): Remove PD from the internal list.

This issue was discovered via the newly added unit tests.

tests/decoder.c: Add some srd_decoder_unload*() tests.

UART: Handle framing errors better

1. Show Frame Error on the Start bit
2. Don't overwrite framing errors with (valid) start/stop bit info

exception: Decorate catch function with format attribute

build: Require GLib 2.28.0

This is needed for e.g. g_slist_free_full(), which incidentally
has been in use for a while already.

util: Factor out Python module load

Create a utility function for loading a Python module by its name
in UTF-8.

decoder: Refactor loading code and plug leaks

Python: Restrict code to stable ABI subset

Limit usage of the Python C API to the stable ABI subset as defined
by PEP 384. This removes some type definitions and functions which
libsigrokdecode made use of. Convert all affected code to suitable
API alternatives. Also fix a few leaks that became apparent while
working on the code.

The most visible change is that PyTypeObject is now an opaque type.
Thus, the custom Decoder and srd_logic types are now created on the
heap via an alternative API. Unfortunately, since tp_name is now
inaccessible, type names had to be removed from the log output.

Stack traces after Python exceptions are now formatted by calling
into Python, since the trace object C API is no longer available.

usb_request: USB transaction decoder and PCAP generator

The new decoder stacks on top of the usb_packet PD. It adds one new
annotation row, and is able to save the decoded data as PCAP trace.

It has been successfully tested against all traces in sigrok-dumps and
some more traces.

usb_packet: Document CRC5 ERROR and CRC16 ERROR packets.

usb_{signalling,packet}: Minor cosmetics.

usb_signalling: Track USB symbol width to compensate frequency errors

USB low/full speed allows for frequency tolerance of 1.5%/0.25%. At
maximum packet size (sync + PID + data + CRC16) of 12 bytes/1027 bytes
this amounts to 1.4 bits/20 bits, so the decoder has to lock to the
actual symbol frequency to avoid any symbol misdetections.

The signal is sampled twice, once at the symbol center and once at
the expected edge position. Comparing the symbol at both positions gives
an indication if the current bit width is too low or too high. Adjust
accordingly.

usb_packet: calculate and check CRC5/CRC16

In case CRC validation fails, an crc5-err/crc16-err annotation is issued,
instead of crc5-ok/crc16-ok, and the error is noted in the annotation
text.

rfm12: fix small typo

There is a small typo in the rfm12 protocol decoder that I just noticed:

tests: Do not reference parent directory in includes

Rely on the compiler include path instead.

tests: Rename DECODERS_DIR to DECODERS_TESTDIR

Avoid confusion resulting from using the same macro name for
different purposes.

build: Do not hard-code decoders location on Windows

init: Also look for decoders in XDG data directories

init: Simplify srd_decoder_searchpath_add()

Don't mess with colon-separated search path strings -- instead,
simply prepend items to the sys.path list object. Also keep the
internal decoder path list in the same order as the items added
to sys.path.

jitter, pwm: Avoid Unicode string literals

It seems the Python we ship along our Windows binaries does not
support u"..." strings, even though it's been a language feature
since Python 2. Remove the "u" prefix to avoid a syntax error.

Also, consistently use format "%.1f" at all scales.

This fixes bug #569.

can: Improve stuff bit annotation placement.

can: Emit bit value annotations.

Makefile.am: Shorten ChangeLog target a bit.

log: Use GLib printf for consistency

Just as in libsigrok, use g_vfprintf() in the log callback to make
sure we are linking to a standard-conforming printf implementation.

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.

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.

Fix srd_log() format warnings

log: Enable varargs format warnings

Decorate srd_log() with the GCC format printf attribute so that the
compiler can check the format string and arguments for consistency.

Also remove the srd_err(), sr_warn(), etc. functions and replace
them with macro wrappers around srd_log().

log: Remove srd_log_logdomain_{get,set} from the API

The confusingly named srd_log_logdomain_set() simply set a global
string prefixed to the log message by the default log callback.
This is pretty much useless, misleadingly named, and not used by
either sigrok-cli or PulseView.

instance.c: Don't use 'di' before NULL-checking it.

Build: Put --no-print-directory into GNUMAKEFLAGS

Also output the compiler version in the configuration summary.

sigrok.m4: Update with latest version from libsigrok

libsigrokdecode.pc: Publish decodersdir variable

This is so that downstream modules can find the install location
of the decoders.

Build: Show CC and CFLAGS in configuration summary

jtag_stm32: Decode IDCODE contents as strings (not just hex vals).

jtag_stm32: Decode more fields, improve IDCODE handling.

jtag_stm32: Cleanups, shorter code.

jtag_stm32: Update to reflect change in jtag OUT_PYTHON.

jtag: Submit bit samplenumbers together with the bits.

jtag_stm32: Fix incorrect handling of registers.

The STM32F10xxx has two serially connected JTAG TAPs, the boundary scan tap
(5 bits) and the Cortex-M3 TAP (4 bits). See UM 31.5 "STM32F10xxx JTAG TAP
connection" for details.
Due to this, we need to ignore the last bit of each data shift (and we
currently ignore the 5 bits of the boundary scan tap).

jtag: Add more annotations, fix a SHIFT-IR/-DR issue.

The code was previously not decoding a bit for the first state change
to SHIFT-IR/-DR, which was incorrect.

jtag: Fix/enable OUT_PYTHON output.