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.

Replace another -D_DEFAULT_SOURCE with -D_POSIX_C_SOURCE=200112L.

Build: Go back to _POSIX_C_SOURCE=200112L

configure: Enable largefile support on 32-bit systems

Build: Define feature test macro _DEFAULT_SOURCE

This basically makes glibc expose the same set of features as
if gcc was invoked without any restricting -std=c* option. Unlike
_GNU_SOURCE however, it does not enable GNU-specific extensions.

So, with this macro defined the behavior of Linux with glibc
should match that of other platforms.

Build: Use TESTS prefix instead of CHECK for flag variables

In order to avoid confusion of the flags-gathering pkg-config
result with the actual test for the availability of "check",
change the pkg-config output variable prefix from CHECK to TESTS.

Clean up .gitignore

Build: Adopt new Autotools magic

Build: Reduce autogen.sh to trivial stub

Use autoreconf instead of invoking the various Auto tools
separately. Get rid of the Darwin-specific guesswork -- it does
not make sense to handle this at the level of libsigrokdecode.

People should set up their ACLOCAL_PATH themselves as appropriate
for their own system; just as they already need to set up various
other paths.

Build: Add copy of libsigrok Autoconf macro file

nunchuk: Fix inverted button press logic.

The buttons are pressed if the respective bit is 0.

Pass unitsize per sample chunk.

Don't pass unitsize to srd_inst_channel_set_all(), have that only
set the channel map. Instead, srd_session_send() now has a parameter
for the unitsize which is passed with every new chunk to be decoded.

This is in preparation to fix issues with devices or files which
have a unitsize != 1 and where the "guessed" unitsize based on the
number of channels is not correct.

This also allows for (potential) future changes where every chunk can
indeed have a different unitsize.

This fixes (parts of) bug #352.

mrf24j40: Fix a register address typo.

tlc5620: Add options for per-DAC Vref, show voltages.

Instead of showing raw values in the annotations, calculate the
actual voltage (based on raw value, gain, and Vref) and use that.

tlc5620: Handle the case of less than 11 bits in a command.

tlc5620: Properly handle LOAD and LDAC based operations.

- If LOAD goes low *and* LDAC is already low, the voltage is set
  immediately.

- If LOAD goes low but LDAC is high, the value is only written into
  an internal register, but the voltage is not changed (yet).

- If LDAC goes low, all four DAC voltages (DAC A/B/C/D) are set
  according to the respective register values at the same time.

tlc5620: Only decode the last 11 bits, ignore the rest.

tlc5620: Fix incorrect DAC selection decode, add more annotations.

usb_packet: Handle invalid packets more gracefully.

Previously the PD was blindly trying to decode any packet, no matter
whether it was too short (and/or bogus) or not, causing issues like this:

  srd: Calling usb_packet decode(): IndexError: list index out of range

Such cases of invalid input are now handled better by emitting "UNKNOWN"
and "Invalid packet" annotations.

This fixes bug #186.

usb_packet: Fix incorrect DATA*/MDATA handling.

Thanks to Pavel Sukortsev for the bug report and fix.

This fixes bug #623.

Add initial SD card (SD mode) decoder.

In the current state the decoder can decode the following commands:
CMD0, CMD2, CMD3, CMD6, CMD7, CMD8, CMD9, CMD10, CMD13, CMD16, CMD55,
ACMD6, ACMD13, ACMD41, ACMD51.

Other commands (and more details for existing commands) will be added later.

Slightly more verbose logging in srd_inst_decode().

Bump version to 0.4.0 (the upcoming next major release).

Append "-git" to the version string.

This avoids confusion with the released tarballs.

Add protocol decoder for Modbus RTU.

qi: Add Qi charger protocol decoder

Add an MDIO/SMI decoder.