sae_j1850_vpw: update for API v3, adjust to recent requirements

IRC user pman92 reported that this decoder exists, and started migration
to the v3 API. This commit completes the migration, and adds missing
decoder infrastructure which has become mandatory recently.

Adjust the boilerplate: Drop FSF postal address. No Python output, add
category tag, unambiguous annotation class and row names. Add reset()
method. Use common code for edge detection.

This commit also addresses minor style nits. Pass the most recent
pulse's edges as ss and es to the data bit handling routine. Adjust
whitespace to unbreak editor navigation and to improve readability.

Use a more verbose name for the decoder, "vpw" appears a little short
and collision happy, and is not found when users search for "j1850".

[ Indentation changed, see whitespace ignoring diff for the essence. ]

Reported-By: pman92 <redacted>

vpw: introduce SAE J1850 (VPW) decoder

Introduce a protocol decoder for the GM VPW 1x and 4x Vehicle Bus
(SAE J1850, or VPW for variable pulse width).

pjon: show "communication relations", and nits

Do track the RX and TX information, including their bus IDs. Present bus
numbers as dotted quads. Emit another summary annotation for completed
frames which presents receiver, transmitter, payload, and ACK details at
even higher zoom levels. Rename the last remaining "init CRC" instance
for consistency.

pjdl: prepare for "stretched" timings

Since the spec is vague on the subject, and real world captures were
found to occassionally run on odd clocks, internally prepare to inspect
traffic and interpret its content although the input data is invalid in
the strictest sense. Keep this hack internal, don't suggest to users
that invalid traffic would be perfectly acceptable.

pjon: use underscore in input/output names for stacked decoders

Rename 'pjon-link' to 'pjon_link' for consistency with other decoders.

pjon: introduce decoder for PJON frame inspection

Introduce a protocol decoder which accepts 'pjon-link' Python input and
interprets PJON frames. The implementation is assumed to be operational
but most of the protocol's flexibility (optionally present and variable
width fields) has not yet been tested due to lack of example captures.
During development of the PJON decoder only the PJDL link layer decoder
was available, other link layers were not tested.

pjdl: introduce decoder for PJON single wire serial link layer

Introduce a protocol decoder which generates 'pjon-link' output from
'logic' input by interpreting the PJDL single wire serial communication
link layer of the PJON protocol stack. This decoder extracts frame
markers, data bytes, as well as their pad/sync decoration. Inspection of
data values, or checks for frame validity remain the responsibility of a
stacked decoder which is shared among several link layer types.

This implementation "violates" the PJDL spec in those places where the
spec is incomplete or vague, and real world traffic would not decode at
all when the strict letter of the spec is applied instead of its spirit.
When in doubt, the decoder implementation errs to the usability side.

Carrier sense detection is incomplete in this version. Data extraction
works for all currently available captures. Recovery from synchronization
loss after glitches is acceptable. Glitch filtering is missing (the spec
is silent on this subject).

jtag: Use list.append instead of insert

Improve processing time by appending bits
instead of inserting them to the lists.

nrf905: Drop non-existent OUTPUT_PYTHON output.

nrf905: More readable annotation byte values.

Drop the 0x prefix for each byte in annotations (for improved readability).

Also, use 02X instead of 02x (printf-style formats).

nrf905: Drop all print() calls.

If those are useful for the decoder user, they should be annotations
using the Ann.WARN annotation class.

nrf905: Simplify a few code snippets.

nrf905: Use SrdIntEnum for annotation classes.

nrf905: Add missing 'tags' item.

ntf905: Add/rename required self.reset() method.

Add decoder for Nordic Semiconductor nRF905 chip

Use PyLong_FromUnsignedLongLong() where needed.

There were a few places where PyLong_FromLong() was used for uint64_t
numbers. Properly use PyLong_FromUnsignedLongLong() there, and also
fix a few additional size/signedness issues while we're here.

Reported (and partial patch provided) by "The Count" on Bugzilla, thanks!

This fixes bug #1499.

type_decoder.c: Workaround for a compiler warning.

  type_decoder.c:1040:16: warning: cast between incompatible function types from ‘PyObject * (*)(PyObject *, PyObject *, PyObject *)’ {aka ‘struct _object * (*)(struct _object *, struct _object *, struct _object *)’} to ‘PyObject * (*)(PyObject *, PyObject *)’ {aka ‘struct _object * (*)(struct _object *, struct _object *)’} [-Wcast-function-type]
   1040 |  { "register", (PyCFunction)Decoder_register, METH_VARARGS|METH_KEYWORDS,
        |                ^

adxl345: Update register meaning for 0x21, 0x22, 0x23 regs.

On the Data row, the content of the single-byte registers is decoded as
follows: '<Meaning> <Value> <Unit>'. Initially, the meaning for these
registers was misplaced. This commit updates these meanings as they
really are.

Signed-off-by: Teo Perisanu <redacted>

ad79x0: Rename the 'ref' option to 'vref'.

ad79x0: Reduce unnecessarily many voltage formats.

ad79x0: Fix the tags metadata item.

ad79x0: Avoid duplicated annotation class/row IDs.

Add AD79x0 decoder.

Signed-off-by: Teo Perisanu <redacted>

ad5626: Always use the same number of decimal places.

ad5626: Rename an annotation class.

ad5626: Drop unnecessary annotation_rows item.

ad5626: Fix the tags metadata item.

Add AD5626 decoder.

Signed-off-by: Teo Perisanu <redacted>

ltc242x: Reduce unnecessarily many voltage formats.

ltc242x: Put channel voltages on different annotation rows.

Drop the pure channel "marking" annotations, they're unneeded.

ltc242x: Use plural for annotation rows, avoid duplicates.

Since recent libsigrokdecode changes, annotation classes and rows must
not have overlapping IDs.

ltc242x: Rename the 'ref' option to 'vref'.

ltc242x: Fix the tags metadata item.

Add LTC242x decoder.

Signed-off-by: Teo Perisanu <redacted>

ltc26x7: Put channel voltages on different annotation rows.

ltc26x7: Reduce unnecessarily many voltage formats.

ltc26x7: Fix the default for the 'chip' option.

ltc26x7: Option renames for consistency.

The 'part' option is renamed to 'chip' (and 'ref' to 'vref') to be more
consistent with the naming used in other decoders.

ltc26x7: Fix the tags metadata item.

ltc26x7: Drop unnecessary annotation_rows item.

All annotation classes are on the same row anyway, and the row name
"LTC26x7 data" isn't all that much more useful than no row name at all.

Add LTC26x7 decoder.

Signed-off-by: Teo Perisanu <redacted>

adxl345: Fix incorrect rate value in handle_reg_0x2c().

adxl345: Use all-lowercase method named for handle_reg_*.

adxl345: Use SrdIntEnum for the state machine.

adxl345: Use SrdIntEnum for annotation classes.

adxl345: Fix scale factor in handle_reg_0x22().

According to the datasheet, this should be 1.25 (ms/LSB).

adxl345: Shorten/simplify self.putbs() invocations.

adxl345: Shorten/simplify a few code snippets.

Add ADXL345 decoder.

Signed-off-by: Teo Perisanu <redacted>

sdcard_sd: Automatically generate token field annotation classes.

sdcard_sd: Put 0/1 bits in different annotation classes.

This will allow for usage of different colors in UIs, and for
showing/hiding them independently in UIs.

sdcard_sd: Put card status fields in their own annotation classes.

This also fixes the CURRENT_STATE and RSVD_TESTMODE fields, which are
not single-bit fields.

sdcard_sd: Add basic support for CSD register fields.

sdcard_sd: Add basic support for CID register fields.

sdcard_sd: Rename some annotation classes.

The responses were simply named "R1" etc., but this becomes inconvenient
when Ann.prefixeѕ('R') is used and other annotation classes also have
names that start with 'R'. Hence, rename respose annotation classes to
"RESPONSE_R1" etc.

sdcard_sd: Use a Bit class to improve readability.

Usually we'd use namedtuple(), but in this case we need to
retro-actively modify the items, so that's not possible.

sdcard_sd: Replace a hardcoded number for better readability.

sdcard_sd: Clarify some variable/argument names.

Differentiate cmd (e.g. 55 or CMD55) vs. cmd_pin (the value of the CMD pin).

sdcard_sd: Remove a list that needs manual maintenance.

Instead of checking if a command is in a fixed list/tuple (that needs
to be manually updated every time a new command becomes supported),
simply check if the respective handling method for the command exists.

This fixes the bug of the CMD16 handler not being called, and will
prevent similar bugs in the future.

sdcard_sd: Use correct annotation class for R2.

cjtag: Use SrdIntEnum for cJTAG states.

sda2506: Use SrdIntEnum for pins.

tlc5620: Use SrdIntEnum for pins.

maple_bus: Use SrdIntEnum for pins.

sdcard_sd: Use SrdIntEnum for pins.

sdcard_sd: Simplify self.putr() invocations.

sdcard_sd: Put responses in their own annotation classes.

Also, rename "reply" to "response" to use the wording from the spec.

sdcard_sd: Use SrdIntEnum for annotation classes.

cjtag: Use SrdStrEnum for the state machine.

jtag: Use SrdStrEnum for the state machine.

edid: Use SrdIntEnum for the state machine.

sdcard_sd: Use SrdStrEnum for the state machine.

srdhelper: Add SrdStrEnum with various helper methods.

usb_signalling: Use SrdIntEnum for the state machine.

jtag_ejtag: Use SrdIntEnum for annotation classes.

mcs48: Use SrdIntEnum for annotation classes.

ac97: Rename Pins class to Pin.

ac97: Use SrdIntEnum for annotation/binary/pin classes.

spiflash: Use SrdIntEnum for annotation classes.

Also, automate construction of the Ann SrdIntEnum.
This avoids having to remember to manually keep two lists in sync.

sdcard_spi: Use ternary operator where possible.

sdcard_spi: Use SrdIntEnum for annotation classes.

This also fixes incorrect annotation classes for bit, bit warning, and R1.

Also, auto-generate some more parts of the annotation class listing.

ds1307: Use SrdIntEnum for annotation classes.

ds1307: Consistently use _ instead of - for various IDs.

cc1101: Use SrdIntEnum for annotation classes.

amulet_ascii: Use SrdIntEnum for annotation classes.

Also, automate construction of the Ann SrdIntEnum.
This avoids having to remember to manually keep two lists in sync.

srdhelper: Add SrdIntEnum with various helper methods.

cjtag: Drop no longer needed _real variable name suffix.

cjtag: Give each cJTAG state its own annotation class.

cjtag: Use correct TCKC/TMSC channel names.

cjtag: Drop some unused method arguments.

cjtag: Drop non-existing channels from the decoder.

cJTAG only has two wires/channels.

cjtag: Use ternary operator where possible.

cjtag: Use += operator where possible.

cjtag: Drop various unneeded parenthesis.

cjtag: Use plural for annotation row IDs/names.

cjtag: Add cJTAG OSCAN1 decoder.

[Note: This is a commit from Kongou Hikari ("diodep" on GitHub) that was
"rebased" by Uwe Hermann on top of the current libsigrokdecode mainline JTAG
decoder. There are various reasons for this, including avoiding non-mainline
or outdated decoder changes, as well as making it easily visible what the
changes vs. the current JTAG decoder are, in case we later need to apply some
changes to both decoders or in case both decoders might be merged later on.
Minor cosmetic and naming changes were also squashed in (no functional
changes, though).]

cjtag: Use an exact copy of the jtag PD as basis for cjtag.

spiflash: Don't use ambiguous annotation class names.

This is not technically a bug since (at least some) datasheets refer
to this command as "Chip erase", it just happens to have two different
command codes (0x60 and 0xc7).

In order to not confuse users with two annotation classes with the
same name, we'll call the second one "Chip erase 2" to match the
"CE2" short name.

This fixes bug #1482.

dmx512: stack DMX512 on top of UART and improve usability

It was unfortunate that the previous DMX512 decoder implementation
re-invented UART decoder features and introduced new issues in the
process. Automatic polarity detection is just impossible when the full
set of valid DMX timings is to get supported. Sample numbers suffered
from floating point rounding errors.

Introduce a stacked decoder on top of UART which exclusively deals with
DMX512 details, and transparently benefits from all available UART
features (adjustable polarity, robust and correct sampling, data byte
accumulation, BREAK detection while silencing false STOP violations). On
one hand this requires users to specify the bitrate (250kbps), on the
other hand it results in reliable operation for all captures that have
become available so far.

Provide Python output for stacked decoders which can process protocol
extensions like RDM. Prepare the DMX512 decoder itself to handle simple
cases of protocol extensions (start codes other than 0). Add support for
additional constraint checks, emit warnings when user specified limits
are violated: short BREAK, long MARK, long RESET to RESET and short
BREAK to BREAK intervals. This shall speedup the identification of bus
health issues (data loss or corruption) or improper controller timing.

Also hide all-zero values by default, to make used channels visually
stand out, and help users focus their attention. For special cases (like
16bit data, or zero being a valid set-point for the channels) users can
enforce the display of all values. Provide an option to present byte
values to users in the most appropriate format for the use case.

This resolves bug #1442.