pwm: Fix and rework

Reworked in the algorithm:
 - Fixed the polarity setting
 - Taken in consideration the first transition
 - Using the 'None' state instead of -1 and 0 value
 - Simplify the algorithm and remove useless branches and variables
 - Avoid re-calculating the same thing more than once
 - Renamed a few variables for a better understanding
 - Duty cycle precision changed to floating value

Otherwise:
 - Added a meta OUTPUT for the duty cycle average
 - Renamed the polarity option:
'polarity', 'active low/high' are well-understood terms.
 - Added comments

Signed-off-by: Sebastien Bourdelin <redacted>

spi: Use 'None' to signal the missing CS# pin.

nrf24l01: Don't decode incomplete commands.

Wait for the CS# line to be inactive before decoding the commands.

nrf24l01: Check for CS# pin and only throw exceptions once.

spi: Tell stacked decoders about missing CS# signal.

This could previously be detected by the absence of a CS-CHANGE packet
before the first data packet, but it makes the stacked decoder simpler
if it is told directly.

Add a PWM decoder.

uart: Fix code comment.

uart: Emit databyte and bits list at the same time.

This will allow for much simpler code in stacked PDs.

Adapt stacked PDs to new API.

instance.c: Fix whitespace.

midi: Fix two incorrect sample numbers.

All PDs: More consistent names for ss/es variables.

Use self.ss/self.es, or if there's a need to differentiate
them a bit more, use self.ss_<suffix>/self.es_<suffix> consistently.

Also, drop some unused variables.

All PDs: Only import the 'Decoder' object.

Anything else in the pd.py files doesn't have to be imported/exposed.

am230x: Factor out handle_byte() to simplify state machine.

swd: Drop unnecessary debug code.

swd: Add SWD protocol decoder for ARM Serial Wire Debug format.

Supports annotated output for analysing debug sessions, Python output
for potential stacked decoders looking at higher level debug
operations.

am230x: Minor documentation fix.

am230x: Use slightly more liberal timing values.

This seems to be required at least for some AM2301 and DHT11 sensors.

am230x: Change option to 'device' to allow for more devices.

am230x: Fix self.reset() method name.

am230x: Minor cosmetics.

Add decoder for AM230x digital humidity and temperature sensors.

Replace ann_format with ann_class.

The naming has been changed to "annotation class" a while ago.

sdcard_spi/usb_packet: Drop unused variables.

uart: Implement signal inversion

Fix out-of-tree build of the unit tests.

Various PDs: Whitespace, cosmetics.

rfm12: Cosmetics.

Also, update author email address.

rfm12: Frequencies: Fix typo (915MHz), add 315MHz entry.

rfm12: Avoid using the Python 'range' keyword as variable.

rfm12: Drop debug printing.

Any errors during decoding should be handled by the PD state machine.

RFM12 decoder

PD tests moved into the sigrok-test repo.

Drop them from the libsigrokdecode repository.

spdif: Simplify get_pulse_type() a bit.

spdif: Use strings for decoder states.

spdif: Add another annotation helper.

spdif: Refactor and reduce nesting level.

Add initial S/PDIF decoder.

ds1307: Warn about (and ignore) non-DS1307 traffic.

ds1307: Correctly handle address wrap-around.

ds1307: Refactoring and cleanups.

ds1307: Handle SRAM register accesses.

ds1307: Handle register 0x07 (control register).

ds1307: Emit per-bit annotations for registers.

Also, add more annotation classes (and annotation rows).

configure.ac: Use AM_CFLAGS instead of CFLAGS.

CFLAGS, CPPFLAGS, LDFLAGS and so on are so-called "user variables" and
are meant for users of the package to use/override. The package itself
should not set or modify them in any way. Instead, the "shadow variables"
such as AM_CFLAGS, AM_CPPFLAGS, AM_LDFLAGS and so on, can be modified by
the package.

tca6408a: Cosmetics.

Add decoder for TCA6408A I2C I/O expanders.

tests: Factor out srdtest_setup() / srdtest_teardown().

Whitespace and consistency fixes.

pdtests: Check for SamplerateError in all PDs that emit it.

parallel: Enforce that at least one pin must be provided.

spi/nrf24l01/uart: Use ChannelError exception.

Rename the old MissingDataError to the clearer ChannelError. Also, add
ChannelError in the UART decoder.

nrf24l01: prettify annotation output

tests/pdtest: Small fix to make it work with Python 3.2.

Python 3.2 doesn't yet have the copy() method for lists:

  AttributeError: 'list' object has no attribute 'copy'

tests/check_session.c: Fix compiler warning.

This fixes 'warning: "_POSIX_C_SOURCE" redefined' with some Python
versions.

Drop references to obsolete sigrok-commits mailing list.

runtc: Update to the new libsigrok session API.

(also fixes the build for libsigrokdecode)

Fix incorrect doxygen comment for srd_decoder_list().

This fixes bug #378.

All PDs: Minor whitespace and consistency fixes.

 - No newlines at the end of files.
 - No trailing ';' characters.
 - Comparison with None: Use 'is None' or 'is not None'.
 - Comparison with True/False: Use 'if cond:' or 'if not cond:'.
 - Various minor whitespace fixes.

uart/i2cfilter: Don't check multiple-choice options.

For options which only have a limited set of valid values, we don't need
to check (in the PD) whether a valid value was supplied, since the backend
can do that for us.

Various PDs: Minor consistency fixes.

(Consistently use single-quotes for all Python strings everywhere)

Various PDs: Throw SamplerateError instead of Exception.

Also, 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.

All PDs: Drop unneeded exceptions.

In all current PDs it is not necessary to raise an exception upon
invalid states (of the PD's state machine), since we can guarantee that
no such invalid state can ever be reached in these PDs.

Add nRF24L01(+) protocol decoder.

spi: Fix order of MISO/MOSI data items.

pdtest: Use new exception output.

Make exception output more predictable.

Remove dependency on Python.h from public API header

This prevents Python.h from being included into client code, where
it can mess things up by e.g. redefining _POSIX_C_SOURCE.

runtc: Fix incorrect include directive

runtc.c used to try to include "../config.h".  This only works when
building in the source tree, otherwise there is no config.h in the
parent directory of runtc.c.  However, the explicit path is
unnecessary since the build directory (whether the same as the source
directory or not) will always be added to the include path.

srd_decoder_load_all_path: Support zipimport paths

srd_decoder_load() works perfectly with decoders stored inside zip files.
However, srd_decoder_load_all{_path,}() did not, because it assumed that
normal directory listings can be used to enumerate decoders, which is not
the case for zipped decoders.  Fix this by providing a fallback based on
the zipimporter Python class.

spi: Add coverage tests for all exceptions.

spi: Add coverage tests for all mosi/miso and lsb/msb combinations.

spi: Refactor for more coverage, and remove dead code.

runtc: PD option value is a GVariant, not string.

This is still broken: it needs to properly check the expected type of
the option, not just blindly assume G_VARIANT_TYPE_STRING.

pdtest: Generate report regardless of verbose mode.

edid: Add another test, covering more code in the PD.

pdtest: Generate a report listing missed lines across the whole PD.

i2c: Generate proper exception for missing samplerate, and test for it.

Also remove some dead code.

pdtest/runtc: Support for output type 'exception'.

The match value should be the name of the exception thrown, e.g.
AttributeError, IndexError or even a custom exception defined in
the protocol decoder, such as SamplerateError.

do not add check to common cflags/libs

Only the unittests use these flags, so don't go linking them in for
the main library too.

fix test operator

The == operator is not in POSIX.

runtc.c: Fix compiler warning

c was of type char, while getopt returns an int. This
caused a compiler warning during build.

libsigrokdecode: Fix make -j issue with build_runtc

The build of runtc caused issues with make -j as runtc
was attempted to be linked with libsigrokdecode before
libsigrokdecode was linked. Using LDADD to insert
libsigrokdecode ensures that it links before runtc,
ensuring the build completes correctly.

Add python as a pkg-config dependency

The current libsigrokdecode build passes the python
CPPFLAGS and LDFLAGS directly into libsigrokdecode.pc.in.
This leads to issues while cross-compiling as while using
PKG_CONFIG_SYSROOT_DIR, there is a chance of the prefix
being duplicated. We now, while detecting the installation
of python (cross or normal), add a dependency for the
module name in the pkgconfig file (from configure.ac).

configure.ac: Bump package version to 0.3.0.

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

The libtool current:revision:age numbers change from 1:0:0 to 2:0: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.1.0.0 to libsigrokdecode.so.2.0.0, the SONAME (+symlink)
becomes libsigrokdecode.so.2.

Bump libsigrok (optional) dependency to 0.3.0.

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

Makefile.am: Add HACKING to the tarball.

Only load and use PDs of API version 2.

Decoders that implement any other PD API version are per definition not
compatible and cannot work with this library version.

README: Document python3-coverage requirement for tests.

Fix 'make install' on OSes where 'python3' doesn't exist.

Sometimes the Python 3 executable is called just 'python3', sometimes
it is called 'python3.4' and so on. Handle all cases transparently for
the user.

Before:

  $ make install
  /usr/bin/env: python3: No such file or directory

After:

  $ make install
  python3.4 ./tools/install-decoders -i ./decoders -o [...]
  Installing 36 protocol decoders:
  [...]

HACKING: Update to current conventions.

(mostly copied from libsigrok, which normally has the same conventions)

Mark some variables more consistently as SRD_PRIV.

log.c: Avoid using the same name for a variable and enum.

Update a few Doxygen @since tags.

libsigrokdecode.h: Give type names to all enumerations.

This matches the libsigrok conventions, and will be required (or
at least nice to have) for libsigrokdecode language bindings later.

tools/install-decoders: Minor consistency fix.

Move srd_check_init() to decoder.c and make it static.

The function srd_check_init() is only used in decoder.c.

Make 'pd_list' static, it's only used in one file.

Make the srd_Decoder type private.

It's not public API and is not used (nor should it be used) by any frontends.