2 years agoMinor whitespace/typo fixes.
Uwe Hermann [Tue, 6 Mar 2018 22:55:34 +0000 (23:55 +0100)]
Minor whitespace/typo fixes.

2 years agodali: Avoid CamelCase in most places.
Uwe Hermann [Fri, 23 Feb 2018 15:57:22 +0000 (16:57 +0100)]
dali: Avoid CamelCase in most places.

2 years agoir_nec: optionally accept input signals that include the carrier
Gerhard Sittig [Sat, 2 Sep 2017 10:52:32 +0000 (12:52 +0200)]
ir_nec: optionally accept input signals that include the carrier

Introduce optional detection of a carrier signal. Immediately "go active"
when edges are seen. "Go inactive" again in the absence of edges in a
specified period of time. Cope with input signals that already had the
carrier removed.

By default carrier detection is disabled, to remain pixel compatible to
the previous implementation. When a carrier frequency is specified and
thus detection is enabled, edges of already filtered input are shifted
by one carrier period, and thus changes the output of the decoder. For
unfiltered inputs that still contain the carrier, detection of activity
is reliable and immediate, but the active phase is extended by one
period of the carrier frequency (which is considered acceptable).

2 years agoparallel: expand 'wordsize' description (bits vs cycles)
Gerhard Sittig [Thu, 28 Dec 2017 17:08:51 +0000 (18:08 +0100)]
parallel: expand 'wordsize' description (bits vs cycles)

Users may not know which unit the "wordsize" is supposed to get
specified in. Especially when it's not a number of bits, but instead
the number of bus cycles. Expand the description text accordingly.

2 years agostepper_motor: make samplerate meta data optional
Gerhard Sittig [Sun, 17 Dec 2017 22:59:16 +0000 (23:59 +0100)]
stepper_motor: make samplerate meta data optional

Only emit the speed annotations when a sample rate was specified. Cope
with the absence of a sample rate for the input stream. Decoding is
still possible, it's just that no timing information is available.

2 years agoi2s: make samplerate meta data optional
Gerhard Sittig [Sun, 17 Dec 2017 23:05:40 +0000 (00:05 +0100)]
i2s: make samplerate meta data optional

Only emit sound samplerate information when an input stream sample rate
was specified. Cope with the absence of a sample rate for the input
stream. Decoding is still possible, it's just that no timing information
is available.

2 years agoi2c: make input samplerate optional
Gerhard Sittig [Fri, 15 Dec 2017 18:26:29 +0000 (19:26 +0100)]
i2c: make input samplerate optional

Absence of a samplerate for the input stream should not be fatal. The
protocol decodes fine, we just cannot determine a bitrate for frames.

This addresses part of bug 1076.

2 years explicitly require pkg-config related macro
Gerhard Sittig [Sat, 10 Feb 2018 11:24:17 +0000 (12:24 +0100)] explicitly require pkg-config related macro

Make sure the PKG_PROG_PKG_CONFIG macro has become available before it
gets used. This unbreaks configuration in the poky environment (rocko).

2 years agograycode, morse, pwm, usb_request, wiegand: cope with absent sample rate
Gerhard Sittig [Sun, 28 Jan 2018 19:49:48 +0000 (20:49 +0100)]
graycode, morse, pwm, usb_request, wiegand: cope with absent sample rate

Improve robustness of some more protocol decoders. Few of them never
checked for the availability of a sample rate in the first place, others
checked for the presence of a spec but would not cope with a value of 0.
Some checked the value only after processing it, which could result in
runtime errors.

This change is motivated by bug #1118.

2 years agospi: cope when a sample rate of 0 was configured
Gerhard Sittig [Sun, 28 Jan 2018 19:02:09 +0000 (20:02 +0100)]
spi: cope when a sample rate of 0 was configured

The explicit test for None was not good enough. Change test conditions
such that sample rates only get processed when they got specified _and_
were not zero.

This fixes bug #1118.

2 years agoac97: introduce first AC'97 decoder implementation (bits, slots, binary)
Gerhard Sittig [Sat, 2 Sep 2017 16:48:03 +0000 (18:48 +0200)]
ac97: introduce first AC'97 decoder implementation (bits, slots, binary)

Introduce an "audio and modem control for PC systems" protocol decoder
(referred to as AC'97).

This implementation extracts bits and identifies frames, and annotates
the slots of a frame with mere integer values. Bit fields get decoded
depending on the slot numbers. Bit patterns in audio/modem data slots
can get exported as binary streams.

Some TODO items remain. Register access (read/write) gets annotated, but
neither gets interpreted nor affects the decoding of subsequent frames.
The RESET# line status does not get evaluated.

2 years agoi2c: Drop obsolete 10bit slave address TODO.
Uwe Hermann [Sat, 13 Jan 2018 17:07:57 +0000 (18:07 +0100)]
i2c: Drop obsolete 10bit slave address TODO.

Nobody has seen any such slave in the wild, yet. In the very unlikely
case that someone actually sees or needs this, patches are welcome
though (together with sample .sr files).

2 years agocounter: make 'reset' signal polarity an option
Gerhard Sittig [Wed, 3 Jan 2018 18:41:37 +0000 (19:41 +0100)]
counter: make 'reset' signal polarity an option

Allow configuration of the 'reset' signal polarity. Reset counters on
either falling (default) or rising edges.

2 years agocounter: Make "word reset" an explicit annotation class+row.
Uwe Hermann [Wed, 3 Jan 2018 16:20:49 +0000 (17:20 +0100)]
counter: Make "word reset" an explicit annotation class+row.

Also, add long and short annotation string versions.

2 years agoAdd a trivial "counter" decoder
Stefan Brüns [Tue, 2 Jan 2018 01:17:07 +0000 (02:17 +0100)]
Add a trivial "counter" decoder

This decoder just counts the number of falling and/or rising edges. This
is especially useful for diagnosing protocols with a clock signal or a
fixed number of transitions per bit, e.g. pulse length coded.

It also provides a divider, which can be used to e.g. count the number
of words in I²C or SPI transfers.

2 years agoparallel: implement consistent number formatting
Gerhard Sittig [Fri, 22 Dec 2017 20:07:28 +0000 (21:07 +0100)]
parallel: implement consistent number formatting

Print all annotations for individual data bit items and for the
de-multiplexed words in a consistent style with leading zeros and
constant width. This shall lend itself better to quick navigation
during visual inspection, as well as automatic processing.

2 years agoparallel: unbreak and improve "word" de-multiplexing
Gerhard Sittig [Fri, 22 Dec 2017 14:16:27 +0000 (15:16 +0100)]
parallel: unbreak and improve "word" de-multiplexing

The previous implementation prepared but never fully enabled the
accumulation of several multi-bit items into words that span multiple
bus cycles (think: address or data de-multiplexing on memory busses).

Complete the accumulation, and fixup the end samplenumber for word
annotations. Fixup the endianess logic (the condition was inverted).
Rephrase calculation to be more Python idiomatic.

Default to word size zero, and only emit word annotations for non-zero
word size specs. This keeps the implementation backwards compatible and
still passes the test suite. Default behaviour is most appropriate for
interactive use in GUI environments, while automated processing will
find consistent behaviour across all setups (non-multiplexed busses, and
multiplexed busses with "words" that span one or multiple cycles).

2 years agoparallel: cope with sparse input mappings, assume zero for not-connected pins
Gerhard Sittig [Fri, 22 Dec 2017 13:48:37 +0000 (14:48 +0100)]
parallel: cope with sparse input mappings, assume zero for not-connected pins

Cope when users only provide e.g. input lines D0 and D2 to the parallel
decoder. Assume that not-connected pins are "always zero".

Rephrase the .decode() logic which determines .wait() conditions while
we are here, to slightly unobfuscate the implementation.

This fixes bug #1088.

2 years agoparallel: address minor style nits, no change in behaviour
Gerhard Sittig [Fri, 22 Dec 2017 14:12:07 +0000 (15:12 +0100)]
parallel: address minor style nits, no change in behaviour

Remove the redundant .itemcount variable, which exactly corresponds to
the length of the .items array.

Arrange retrieval of options and their evaluation closer to each other
for improved readability.

Use common logic to construct "words" from several multi-bit "items".
Arrange for endianess support by optionally reversing the array before

2 years agoparallel: unify decode() code paths with and without clock signal
Gerhard Sittig [Fri, 22 Dec 2017 12:55:45 +0000 (13:55 +0100)]
parallel: unify decode() code paths with and without clock signal

Instead of implementing two main loops for operation in the presence and
in the absence of a clock line, use a common main loop which operates on
pre-determined wait conditions.

2 years agousb_power_delivery: enforce check order for start-of-packet sequences
Gerhard Sittig [Fri, 22 Dec 2017 17:51:00 +0000 (18:51 +0100)]
usb_power_delivery: enforce check order for start-of-packet sequences

The list of a dictionary's keys need not reproduce in identical order
everywhere. Make sure to run all start-of-packet sequence checks in the
decoder implementation in a specific order on each machine, such that
annotations get emitted with identical content and in the same order for
each execution of the decoder.

This fixes the remaining part of bug #1090.

2 years agousb_power_delivery: fixup a Python style nit (dict lookup fallback)
Gerhard Sittig [Fri, 22 Dec 2017 17:37:29 +0000 (18:37 +0100)]
usb_power_delivery: fixup a Python style nit (dict lookup fallback)

Use the dictionary's .get() method in combination with a default result
parameter, instead of an explicit "k in dictvar" test and a conditional

2 years agousb_power_delivery: enforce numerical order of RDO/PDO flag annotations
Gerhard Sittig [Fri, 22 Dec 2017 17:33:58 +0000 (18:33 +0100)]
usb_power_delivery: enforce numerical order of RDO/PDO flag annotations

Annotations of the USB power delivery decoder contain multiple text
fragments that correspond to several flags in bit fields. The Python
runtime did not guarantee an order of emission and made the test suite

Sort the order in which RDO and PDO flags related text fragments get
constructed and concatenated. Print text for higher bit positions first
as this might feel more intuitive to users.

This fixes part of bug #1090.

2 years agoall decoders: introduce a reset() method
Gerhard Sittig [Mon, 20 Feb 2017 20:37:05 +0000 (21:37 +0100)]
all decoders: introduce a reset() method

Move initialization code of protocol decoders from the constructor to a
new reset() helper method. The libsigrokdecode backend could run this
method several times to clear the decoder's internal state, before new
data from another acquisition gets fed to decode() calls.

2 years agossi32: rename reset() helper method
Gerhard Sittig [Fri, 22 Dec 2017 02:55:16 +0000 (03:55 +0100)]
ssi32: rename reset() helper method

The ssi32 decoder implements a reset() method which clears internally
accumulated data during decoding. Rename the method before all decoders
will grow a new reset() method that will be used for a different purpose.

2 years agomrf24j40: rename reset() helper method
Gerhard Sittig [Fri, 22 Dec 2017 02:54:21 +0000 (03:54 +0100)]
mrf24j40: rename reset() helper method

The mrf24j40 decoder implements a reset() method which clears internally
accumulated data during decoding. Rename the method before all decoders
will grow a new reset() method that will be used for a different purpose.

2 years agoade77xx: rename reset() helper method
Gerhard Sittig [Fri, 22 Dec 2017 02:51:20 +0000 (03:51 +0100)]
ade77xx: rename reset() helper method

The ade77xx decoder implements a reset() method which clears internally
accumulated data during decoding. Rename the method before all decoders
will grow a new reset() method that will be used for a different purpose.

2 years agosdcard_sd: remove unused .reset() method
Gerhard Sittig [Fri, 22 Dec 2017 01:35:12 +0000 (02:35 +0100)]
sdcard_sd: remove unused .reset() method

The decoder's .reset() method seems to partially implement constructor's
assignments, but is not referenced anywhere. There is neither a direct
call site in the remainder of which uses the "reset" name, and
runtime computation only references "handle_*()" methods for commands
and responses.

Drop the unused .reset() method in the individual decoder before the
introduction of a common .reset() method approach for all decoders.

2 years agoam230x: rename reset() helper method
Gerhard Sittig [Mon, 20 Feb 2017 19:58:09 +0000 (20:58 +0100)]
am230x: rename reset() helper method

The am230x decoder implements a reset() method which clears internal
decoder state. Rename the method before all decoders will grow a new
reset() method that will be used for a different purpose.

2 years agoeeprom 24xx: rename reset() helper method
Gerhard Sittig [Mon, 20 Feb 2017 20:09:07 +0000 (21:09 +0100)]
eeprom 24xx: rename reset() helper method

2 years agoir_nec: Make timing margin relative
Grant Miller [Tue, 14 Nov 2017 09:31:05 +0000 (03:31 -0600)]
ir_nec: Make timing margin relative

This replaces the fixed timing margin with a percentage based tolerance
to better allow for timing inaccuracies, especially for longer timings
like the Leader and Repeat codes.

2 years agograycode: Move bitpack/bitunpack to common/.
Uwe Hermann [Sat, 9 Dec 2017 16:25:53 +0000 (17:25 +0100)]
graycode: Move bitpack/bitunpack to common/.

2 years agoAdd a gray code and rotary encoder protocol decoder.
Christoph Rackwitz [Mon, 18 Sep 2017 19:15:09 +0000 (21:15 +0200)]
Add a gray code and rotary encoder protocol decoder.

2 years agoAdd a morse code protocol decoder.
Christoph Rackwitz [Thu, 21 Sep 2017 01:28:33 +0000 (03:28 +0200)]
Add a morse code protocol decoder.

2 years agoi2s: Fix unportable use of struct.pack
Marcus Comstedt [Mon, 27 Nov 2017 09:54:35 +0000 (10:54 +0100)]
i2s: Fix unportable use of struct.pack

Using the format string 'I' with the default (implied) prefix '@'
results in both word size and endianness being platform dependent.
In this case standard size (32 bits) and little endianness is
required, so the prefix '<' needs to be used.

2 years agomaple_bus: Add helpers to make the call sites a bit shorter.
Uwe Hermann [Sat, 9 Dec 2017 12:29:02 +0000 (13:29 +0100)]
maple_bus: Add helpers to make the call sites a bit shorter.

2 years agoAdd an initial Maple bus decoder.
Marcus Comstedt [Mon, 20 Nov 2017 19:01:24 +0000 (20:01 +0100)]
Add an initial Maple bus decoder.

3 years agoinstance.c: Removed redundant variable
Joel Holdsworth [Thu, 28 Sep 2017 03:15:04 +0000 (21:15 -0600)]
instance.c: Removed redundant variable

3 years Added AC_C_BIGENDIAN to autoconf
Joel Holdsworth [Thu, 5 Oct 2017 15:00:30 +0000 (09:00 -0600)] Added AC_C_BIGENDIAN to autoconf

This is needed so that WORDS_BIGENDIAN is defined on big-endian
systems, which is needed by version.c .

3 years agoi2s: Set file size to 4Gb
Joel Holdsworth [Wed, 8 Nov 2017 02:55:50 +0000 (19:55 -0700)]
i2s: Set file size to 4Gb

3 years agoi2s: Make 32-bit wav files
Joel Holdsworth [Wed, 8 Nov 2017 02:55:15 +0000 (19:55 -0700)]
i2s: Make 32-bit wav files

3 years agods243x: Add binary output for data read from EEPROM
Soeren Apel [Wed, 25 Oct 2017 07:28:56 +0000 (09:28 +0200)]
ds243x: Add binary output for data read from EEPROM

3 years agoRename ds2432 to ds243x
Soeren Apel [Wed, 25 Oct 2017 07:22:58 +0000 (09:22 +0200)]
Rename ds2432 to ds243x

3 years agods2432: Adapt to allow for DS2433 as well
Soeren Apel [Tue, 31 Oct 2017 20:17:53 +0000 (21:17 +0100)]
ds2432: Adapt to allow for DS2433 as well

3 years agods2432: Simplify MAC printing.
Uwe Hermann [Thu, 19 Oct 2017 07:36:17 +0000 (09:36 +0200)]
ds2432: Simplify MAC printing.

3 years agods2432: Add a Maxim DS2432 protocol decoder.
Kevin Redon [Thu, 19 Oct 2017 07:08:42 +0000 (09:08 +0200)]
ds2432: Add a Maxim DS2432 protocol decoder.

This fixes (parts of) bug #1019.

3 years agosrd_init(): Print list of decoder search paths in debug output.
Uwe Hermann [Fri, 22 Sep 2017 09:45:47 +0000 (11:45 +0200)]
srd_init(): Print list of decoder search paths in debug output.

3 years agoClarify and improve channel map debug output.
Uwe Hermann [Tue, 5 Sep 2017 22:37:35 +0000 (00:37 +0200)]
Clarify and improve channel map debug output.

3 years agousb_request: Remove code duplication, save end sample in central location
Stefan Brüns [Sat, 22 Jul 2017 18:17:10 +0000 (20:17 +0200)]
usb_request: Remove code duplication, save end sample in central location

3 years agousb_request: Handle CONTROL transfer protocol stalls
Stefan Brüns [Sat, 22 Jul 2017 17:54:59 +0000 (19:54 +0200)]
usb_request: Handle CONTROL transfer protocol stalls

According to the USB 2.0 spec,, a protocol stall condition lasts
until the next SETUP transfer. On reception of the SETUP, adjust the end
sample accordingly, and flush the previous CONTROL transfer.

3 years agoFix a unit test.
Uwe Hermann [Thu, 6 Jul 2017 06:23:34 +0000 (08:23 +0200)]
Fix a unit test.

Only try to acquire the GIL if the Python interpreter is actually
initialized, otherwise there'll be a segfault.

3 years agocan: introduce clock synchronization (simple implementation)
Gerhard Sittig [Sun, 2 Jul 2017 10:39:08 +0000 (12:39 +0200)]
can: introduce clock synchronization (simple implementation)

Check for falling edges (i.e. changes to dominant state) between bits of
a CAN frame, and adjust subsequent bit slots' sample points accordingly.
This is a simple implementation which could get improved later. But it
improves the decoder's reliability when the input signal's rate differs
from the nominal rate.

This fixes bug #990.

Reported-By: PeterMortensen via IRC
3 years agocan: rename 'bitpos' variable
Gerhard Sittig [Sun, 2 Jul 2017 10:23:09 +0000 (12:23 +0200)]
can: rename 'bitpos' variable

Consistently use .sample_point for the member variable that corresponds
to the 'sample_point' property. Use 'samplenum' in .get_sample_point()
to reflect that the routine returns a sample number. This eliminates the
'bitpos' identifier which was used for two different things.

3 years agoself.put(): Eliminate an unneeded malloc/free pair.
Uwe Hermann [Fri, 30 Jun 2017 16:42:13 +0000 (18:42 +0200)]
self.put(): Eliminate an unneeded malloc/free pair.

3 years agoAcquire/release the Python GIL where needed to avoid threading issues.
Uwe Hermann [Wed, 28 Jun 2017 20:29:06 +0000 (22:29 +0200)]
Acquire/release the Python GIL where needed to avoid threading issues.

With these additions, frontends can now call libsigrokdecode API
functions from different threads without running into threading issues.

The backend releases the GIL when it is performing tasks that might take
a while and it doesn't need to run Python/C API calls during that time.

This allows frontends to run multiple PD stacks (in multiple frontend
threads) "at the same time" in a time-sharing, "interlocked" manner.
Whenever one of the decoders is inside e.g. self.wait() it releases the
GIL and thus allows other decoders to do some work in the mean time.

The user-visible effect is that for use-cases such as running 3 different
decoder stacks at the same time for an acquisition, the user will not
have to wait for PD 1 to finish decoding, then wait for PD 2 to finish
decoding, and only *then* being able to see annotations from PD 3.

Instead, all three PDs will decode some chunks of data from time to
time, thus the user is able to inspect annotations from all 3 PDs while
the acquisition and decoding is still going on.

3 years agoHACKING: Prefer git pull requests over mailing list patches.
Uwe Hermann [Fri, 16 Jun 2017 18:48:07 +0000 (20:48 +0200)]
HACKING: Prefer git pull requests over mailing list patches.

Also, drop the reference to (no longer available).

3 years agopwm: Drop unused 'pins' variable.
Uwe Hermann [Wed, 21 Jun 2017 06:32:43 +0000 (08:32 +0200)]
pwm: Drop unused 'pins' variable.

3 years agoir_rc5: Rephrase open coded value for start bit 1
Gerhard Sittig [Sun, 18 Jun 2017 18:02:46 +0000 (20:02 +0200)]
ir_rc5: Rephrase open coded value for start bit 1

Make obvious that the start bit's value is 1 in the IDLE stage.

3 years agolpc: Improve robustness when decoding unexpected input data
Gerhard Sittig [Sun, 18 Jun 2017 10:24:32 +0000 (12:24 +0200)]
lpc: Improve robustness when decoding unexpected input data

The 'fields' table of state and descriptions is not fully populated,
some slots are missing. Cope with lookup misses when unexpected input
data is not found in the table. Use different error text in annotations
for described but invalid states (the previous implementation used
"reserved / invalid"), and for states that are not described in the
table (introduce the "reserved / unknown" text for conditions that are
not covered by the decoder implementation).

The previous implementation missed the emission of some warnings. When a
"reserved / invalid" description was found, the subsequent exact match
for "reserved" failed and the warning was not emitted. Weaken the test
to emit warnings for either description that has "reserved" in it,
regardless of whether the text was found in the table or is not part of
the table at all.

3 years agolpc: Nit, remove an unused variable
Gerhard Sittig [Sun, 18 Jun 2017 10:19:32 +0000 (12:19 +0200)]
lpc: Nit, remove an unused variable

3 years agoonewire_network: Fixup start samplenumber for annotations
Gerhard Sittig [Fri, 17 Mar 2017 19:28:59 +0000 (20:28 +0100)]
onewire_network: Fixup start samplenumber for annotations

Annotations generated by the onewire_network decoder started where bit 1
began, while it should align with the start of bit 0.

3 years agoguess_bitrate: Eliminate internal state of the decoder
Gerhard Sittig [Sun, 18 Jun 2017 18:46:40 +0000 (20:46 +0200)]
guess_bitrate: Eliminate internal state of the decoder

Getting the first edge outside of the loop in .decode() greatly
simplifies the logic, any other edge then is just an update of the
previously collected data, determined by always equal code.

3 years agoi2c: Don't drop the first sample of input data
Gerhard Sittig [Sun, 18 Jun 2017 17:32:57 +0000 (19:32 +0200)]
i2c: Don't drop the first sample of input data

The previous implementation of the I2C decoder used to retrieve and not
process the first sample of the input stream. Remove this instruction.

3 years agodecoders: Rephrase condition-less .wait() calls (self documentation)
Gerhard Sittig [Sun, 18 Jun 2017 17:27:47 +0000 (19:27 +0200)]
decoders: Rephrase condition-less .wait() calls (self documentation)

Telling .wait() to "skip one sample" slightly obfuscates the intent of
getting the next samples while no condition applies. Explicitly pass no
condition arguments instead, to better reflect the purpose. Coincidently
these .wait() calls will execute in slightly less expensive code paths
in the common code.

3 years agodecoder: Accept more forms of "unconditional wait()" (None, no args)
Gerhard Sittig [Sun, 5 Mar 2017 17:31:18 +0000 (18:31 +0100)]
decoder: Accept more forms of "unconditional wait()" (None, no args)

The Decoder.wait() method expects a list of dicts, or as a special form
a single dict, to specify the conditions to wait for. An empty dict or
an empty list mean "unconditional wait", requesting the very next sample.

Accept None as well as no arguments at all in Decoder.wait() calls. This
shall better reflect the intent and slightly unobfuscate PD code, as well
as avoid creation of potentially expensive Python objects at the call site.

3 years agodecoder: Unbreak execution of Decoder.wait() with empty condition
Gerhard Sittig [Sun, 18 Jun 2017 17:12:11 +0000 (19:12 +0200)]
decoder: Unbreak execution of Decoder.wait() with empty condition

The Decoder.wait() method works as expected when non-empty conditions
are specified by the caller. For empty conditions the implementation was
incomplete, and ended up in an infinite loop because the sample number
got stuck in the current location. Code review revealed more issues like
not getting more input data chunks when needed.

Detect when empty wait() conditions were specified, and re-use existing
code paths for match handling as much as possible. This is achieved by
the manual creation of a SKIP term with the appropriate count value.
It's assumed that naive decoder implementations will run this kind of
condition-less query for every individual sample, which means that
efficiency is important.

Make sample number 0 available to condition-less calls, too. Don't skip
the first sample in the input stream.

3 years agousb_signalling: Move another edge detection to common backend code
Gerhard Sittig [Sun, 18 Jun 2017 13:51:35 +0000 (15:51 +0200)]
usb_signalling: Move another edge detection to common backend code

Rephrase how the 'WAIT IDLE' stage skips over all-low input signals.
Have the next high level on either line detected in common code.

3 years agopwm: Fixup sample numbers for binary output
Gerhard Sittig [Sun, 18 Jun 2017 11:50:48 +0000 (13:50 +0200)]
pwm: Fixup sample numbers for binary output

Do reference the period's start and end sample numbers when the
duty cycle is written to the binary output stream. The previous
implementation inappropriately used the period counter instead.

3 years agopwm: Eliminate more decoder "state"
Gerhard Sittig [Sun, 18 Jun 2017 11:38:15 +0000 (13:38 +0200)]
pwm: Eliminate more decoder "state"

Move more items from the Python object's members to local variables of the
decode() method, as they are used there exclusively.

Mark a spot where the binary output references suspicious "sample numbers".

3 years agopwm: Rephrase edge and period detection, eliminate internal state
Gerhard Sittig [Sun, 18 Jun 2017 10:59:31 +0000 (12:59 +0200)]
pwm: Rephrase edge and period detection, eliminate internal state

With the simplicity of PWM waveforms and the convenience of the v3 API,
there is no need to maintain internal state. After the start of the
first period was determined based on the input signal's polarity, any
period is defined by just the next two edges, and the next period is
immediately to follow.

3 years agoDrop support for PD API version 2.
Uwe Hermann [Fri, 16 Jun 2017 14:32:24 +0000 (16:32 +0200)]
Drop support for PD API version 2.

All decoders must be of PD API version 3 now.

3 years agoMark all stacked decoders as being PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 14:27:08 +0000 (16:27 +0200)]
Mark all stacked decoders as being PD API version 3.

This is not really relevant for stacked PDs currently (they can be used
unmodified with either PDv2 or PDv3 low-level decoders), but it'll allow
us to drop PDv2 support completely.

3 years agoir_rc5: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 14:23:10 +0000 (16:23 +0200)]
ir_rc5: Convert to PD API version 3.

3 years agodsi: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 14:22:57 +0000 (16:22 +0200)]
dsi: Convert to PD API version 3.

3 years agodsi: Cosmetics, drop obsolete comments.
Uwe Hermann [Fri, 16 Jun 2017 14:17:50 +0000 (16:17 +0200)]
dsi: Cosmetics, drop obsolete comments.

3 years agorgb_led_ws281x: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 14:13:19 +0000 (16:13 +0200)]
rgb_led_ws281x: Convert to PD API version 3.

3 years agolpc: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 14:09:00 +0000 (16:09 +0200)]
lpc: Convert to PD API version 3.

3 years agowiegand: Convert to PD API version 3.
Gerhard Sittig [Sun, 15 Jan 2017 12:09:36 +0000 (13:09 +0100)]
wiegand: Convert to PD API version 3.

3 years agousb_signalling: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 11:37:21 +0000 (13:37 +0200)]
usb_signalling: Convert to PD API version 3.

3 years agodali: Change 'self.dali' to 'dali'.
Uwe Hermann [Fri, 16 Jun 2017 10:14:28 +0000 (12:14 +0200)]
dali: Change 'self.dali' to 'dali'.

3 years agodali: Convert to PD API version 3.
Uwe Hermann [Fri, 16 Jun 2017 10:08:50 +0000 (12:08 +0200)]
dali: Convert to PD API version 3.

3 years agodali: Random cleanups, remove obsolete comments.
Uwe Hermann [Fri, 16 Jun 2017 08:45:36 +0000 (10:45 +0200)]
dali: Random cleanups, remove obsolete comments.

3 years agoinstance.c: Drop a few unneeded checks.
Uwe Hermann [Thu, 15 Jun 2017 22:22:40 +0000 (00:22 +0200)]
instance.c: Drop a few unneeded checks.

3 years agospi: Simplify some code chunks.
Uwe Hermann [Thu, 15 Jun 2017 21:34:56 +0000 (23:34 +0200)]
spi: Simplify some code chunks.

3 years agodmx512: Use a nicer 'dmx' pin name variable.
Uwe Hermann [Thu, 15 Jun 2017 20:27:43 +0000 (22:27 +0200)]
dmx512: Use a nicer 'dmx' pin name variable.

3 years agodmx512: Convert to PD API version 3.
Uwe Hermann [Thu, 15 Jun 2017 20:02:52 +0000 (22:02 +0200)]
dmx512: Convert to PD API version 3.

3 years agopwm: Reduce indentation level.
Uwe Hermann [Thu, 15 Jun 2017 19:35:25 +0000 (21:35 +0200)]
pwm: Reduce indentation level.

3 years agopwm: Simplify waiting for initial edge.
Uwe Hermann [Thu, 15 Jun 2017 19:33:56 +0000 (21:33 +0200)]
pwm: Simplify waiting for initial edge.

3 years agosession.c: Fix a compiler warning on Mac OS X.
Uwe Hermann [Thu, 15 Jun 2017 15:52:55 +0000 (17:52 +0200)]
session.c: Fix a compiler warning on Mac OS X.

  session.c:203:22: warning: format specifies type 'unsigned long long'
  but the argument has type 'guint64' (aka 'unsigned long') [-Wformat]
                          sess->session_id, g_variant_get_uint64(data));
  libsigrokdecode-internal.h:103:44: note: expanded from macro 'srd_dbg'
  #define srd_dbg(...)    srd_log(SRD_LOG_DBG,  __VA_ARGS__)

3 years Bump package version to 0.6.0. libsigrokdecode-unreleased
Uwe Hermann [Thu, 15 Jun 2017 14:16:09 +0000 (16:16 +0200)] Bump package version to 0.6.0.

0.6.0 will be the next major release. Bump now, so that
there is no confusion of tarball 0.5.0 and 0.6.0-git snapshots.

3 years agoBump libtool version (not package version) to 4:0:0.
Uwe Hermann [Mon, 12 Jun 2017 01:16:21 +0000 (03:16 +0200)]
Bump libtool version (not package version) to 4:0:0.

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

This changes the library filename (e.g. on Linux) from to, the SONAME
(+symlink) becomes

3 years agoNEWS: Add list of user-visible changes so far.
Uwe Hermann [Fri, 9 Jun 2017 13:00:46 +0000 (15:00 +0200)]
NEWS: Add list of user-visible changes so far.

3 years agoVarious Doxygen fixes.
Uwe Hermann [Mon, 12 Jun 2017 00:50:23 +0000 (02:50 +0200)]
Various Doxygen fixes.

3 years agospi: Missing samplerate spec is not fatal
Gerhard Sittig [Tue, 6 Jun 2017 21:12:04 +0000 (23:12 +0200)]
spi: Missing samplerate spec is not fatal

Check for the availability of a samplerate specification, but don't
abort in its absence. Lack of the samplerate only prevents the emission
of throughput annotations, but still allows decoding the clocked signal.

3 years agospi: Improve use of PD API version 3
Gerhard Sittig [Tue, 6 Jun 2017 20:55:52 +0000 (22:55 +0200)]
spi: Improve use of PD API version 3

Eliminate more instances where Python code tracked signal changes instead
of having common library code do the work. Reduce the number of boundary
crossings between library and decoder code (Python vs C), by inspecting
fewer input edges. Yet cope with the CS signal's being optional. Emit
identical annotations to the previous implementation for CS changes,
including the initial change "from None to 0/1". Sort the checks for
optional input signals at the top of decode() into a more natural order.
Improve and update comments.

This commit results in a more appropriate use of the PD v3 API in the
SPI protocol decoder.

3 years agogpib: Convert to PD API version 3
Gerhard Sittig [Tue, 6 Jun 2017 19:13:27 +0000 (21:13 +0200)]
gpib: Convert to PD API version 3

Have edges detected in common library code. Cope with optional initial
low level (without an edge) at the start of sample data. Handle the
optionally enforced processing at a specified sample number.

3 years agopwm: Convert to PD API version 3
Gerhard Sittig [Tue, 6 Jun 2017 18:32:07 +0000 (20:32 +0200)]
pwm: Convert to PD API version 3

Have edges detected in common library code.

3 years agospi: Convert to PD API version 3
Gerhard Sittig [Tue, 6 Jun 2017 18:05:05 +0000 (20:05 +0200)]
spi: Convert to PD API version 3

Have edges detected in common library code. Move the test for presence
of optional channels out of the decode loop. Do inspect the very first
sample for pixel compatibility with the previous implementation.

3 years agoHACKING: Update URL to Linux kernel coding style.
Uwe Hermann [Tue, 6 Jun 2017 12:43:08 +0000 (14:43 +0200)]
HACKING: Update URL to Linux kernel coding style.

3 years agoRename eeprom93cxx decoder to eeprom93xx.
Uwe Hermann [Tue, 6 Jun 2017 10:46:08 +0000 (12:46 +0200)]
Rename eeprom93cxx decoder to eeprom93xx.

Not all EEPROMs of this series have (just) a 'c' in the name.