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.

adns5020: Minor tweak to annotation format.

mrf24j40: Factor out register lists to lists.py.

Add a Microchip MRF24J40 802.15.4 2.4GHz RF tranceiver decoder.

No interpretation of register bits, nor attempting to decode packet contents,
but all shifted register addresses and long register memory regions are
decoded.

Signed-off-by: Karl Palsson <redacted>

adns5020: Add manufacturer name to description.

Add Avago ADNS-5020 optical mouse sensor decoder.

Decodes reads/writes of registers. Understands resetting on chip select if
provided. Does not understand burst mode.

Signed-off-by: Karl Palsson <redacted>

am23xx: explicitly mention RHTxx devices.

Include their compatibility in the dropdown options too.

timing: Shorten annotations (time units).

Add a decoder that calculates timings between transitions.

spiflash: Add lists.py and a 'chips' PD option.

Rename the 'mx25lxx05d' PD to 'spiflash'.

This is in preparation of making the PD more generic so it can support
a lot more xx25 series SPI (NOR) flash chips.

Fix a bunch of typos.

autogen.sh: Drop obsolete MinGW/MSYS items.

For Windows builds (which require MinGW-w64) we currently support:

 - cross-builds using MXE (mxe.cc) and possibly other cross-compile setups

 - native builds using MSYS2 (sf.net/projects/msys2/)

Neither of those require explicitly specifying ACLOCAL_DIR.

onewire_link: Fix a bug when the samplerate is too low for the PD.

The code was trying to emit annotations before
self.register(srd.OUTPUT_ANN) was run.

This fixes bug #357.

Makefile.am: Use libtool's -no-undefined option.

spi: Don't decode data lines if CS isn't asserted

Avoid decoding and outputting data from the SPI bus if the CS pin
hasn't been asserted. This avoids confusing both users and stacked
decoders which otherwise end up seeing traffic intended for other
chips (or just noise).

Note: The old behavior of decoding all traffic is still in place if
no CS pin has been wired up to the decoder.

This fixes bug #559.

Slightly shorten a few PD descriptions.

Add a decoder for stepper motor direction/speed signals.

Fix a few decoding bugs with arm_etmv3.

Fixed:
- wrong address decoding for some branches
- error in annotation lists for exception entry/exit
- exception was being thrown for 0E0N p-header packets
- fixes to objdump output parsing, better function name & source code matching

New features:
- added decoding of IRQ names for branch exception info
- added support for -C option to objdump to decode C++ symbol names
- put branches on the "Exceptions" annotation row when they contain exception info.

pwm: add period annotation

Use g_malloc*() consistently, simplify error handling.

Use g_malloc*() for small allocations and assume they always
succeed. Simplify error handling in a few places accordingly.

Document the rules in the README file.

unit tests: Drop unneeded check_ filename prefix.

mx25lxx05d: Implement FAST READ command.

Fix typo in swd decoder

Add ARM TPIU/ITM/ETMv3 decoders

nrf24l01: Change 'xn297_extensions' option to 'chip'.

This allows for easily extending the PD to support other clones or
compatibles of the nRF24L01(+) chip as well.

nrf24l01: Add option to support the XN297's extra registers

Improve uart decoder sample positions at high data rates.

At 3 samples per bit, the uart decoder took the value at the last sample
instead of the middle one. Improve calculations so that sampling is more
accurate at odd number of samples per bit.

unit tests: Disable timeout for one of the test cases.

This one can take a while, thus disable the timeout.

README: Update build requirements list.

Add 24xx I2C EEPROM protocol decoder.

spi: Add binary output facilities for MISO/MOSI.

This fixes bug #424.

edid: Add annotation rows.

eded: Add all sections.

jitter: Simplify ascii-float binary out.

This also outputs one item of OUT_BINARY per jitter value (not one
item per ASCII character in the ASCII floating point number).

jitter: Rename "jitter" binary out to "ascii-float".

jitter: add a binary output to report the raw jitter value.

Signed-off-by: Sebastien Bourdelin <redacted>

jitter: fixes an error when a miss clock/signal is detected

Signed-off-by: Sebastien Bourdelin <redacted>