asix-sigma: update comments in firmware download code paths

The current implementation of the ASIX Sigma firmware download contains
comments which express uncertainty. Rephrase them, no magic is involved.

Discuss the polarity of the CCLK hardware signal. Which shall eliminate
potential concerns in future reviews or maintenance.

This commit only updates comments, and does not change behaviour.

Signed-off-by: Gerhard Sittig <redacted>

Doxyfile*: Set GENERATE_TODOLIST to NO.

We don't really need or use this. Bugs/issues should go into Bugzilla.

demo: File naming consistency changes.

hantek-dso: File naming consistency changes.

fluke-dmm: File naming consistency changes.

agilent-dmm: File naming consistency changes.

resource: Improve logging output.

Don't show duplicate lines (per default) such as

  sr: resource: Failed to locate 'saleae-logic16-fx2.fw'.
  sr: resource: Failed to open resource 'saleae-logic16-fx2.fw'.

The first one is now an sr_dbg() instead of sr_err().

Also, mention that a higher loglevel will give more information as to
where the backend is looking for resources / firmware files.

This fixes bug #806.

output/csv: Add an option to output units for column labels.

This change tweaks the CSV output module to change the label
setting from on/off to units/channels/off, where channels is the old
on behavior, and units uses the meaning field to generate the column
label - except for the generated Time column, which uses the label from
the X axis when it's generating gnuplot output.

rigol-ds: Add DS1074Z Plus and DS1104Z Plus.

This was successfully tested on a DS1074Z Plus.

contrib: Drop all example gnuplot scripts.

These files (or at least similar ones) can now be generated by the
CSV output module.

output/gnuplot: Remove, obsoleted by improved CSV module.

We no longer need a gnuplot output module, the CSV module can output
gnuplot-compatible data now (and it can also generate .gpi files).

csv: Fix a segfault due to a g_malloc() allocating too few bytes.

csv: Fix a segfault when using non-hardware input.

New all-singing, almost all dancing, csv output module.

 - It now handles more than one analog value correctly - at least from the
   demo driver.
 - Add column headers from channel names.
 - Add a row dedup capability.
 - Add a sample time column.
 - Add a frame end formatting (for gnuplot).
 - Made almost all formatting controllable or at least optional.
 - Fix it so we can mix analog and digital values.
 - Add outputting a gnuplot script for the data.
 - Count actual channels, not just mine, to find end of sample.
 - Add trigger option (untested).

fluke-dmm: Fix two compiler warnings.

  src/hardware/fluke-dmm/fluke.c:312:24: warning: implicit conversion from \
  enumeration type 'enum sr_unit' to different enumeration type 'enum sr_mq' \
   [-Wenum-conversion]
          devc->mq = devc->unit = devc->mqflags = 0;
                   ~ ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~

  src/hardware/fluke-dmm/fluke.c:312:40: warning: implicit conversion from \
  enumeration type 'enum sr_mqflag' to different enumeration type 'enum \
  sr_unit' [-Wenum-conversion]
          devc->mq = devc->unit = devc->mqflags = 0;
                                ~ ~~~~~~~~~~~~~~^~~

es51919: Fix a compiler warning.

  src/lcr/es51919.c:538:73: warning: comparison of unsigned enum expression \
< 0 is always false [-Wtautological-compare]
    if ((analog->meaning->mq = parse_mq(pkt, is_secondary, pkt[2] & 0x80)) < 0)

strutil.c: Fix a Doxygen warning.

Fix various sample/time limit regressions.

These were accidentally introduced when the sw_limits helpers were added.

gmc-mh-1x-2x: Shorten some code by using the ternary operator.

Remove some unneeded double-spaces.

(also perform some other minor whitespace fixes while at it)

device.c: Whitespace/cosmetics and typo fixes.

analog.c: Whitespace/cosmetics and typo fixes.

HACKING: Prefer git pull requests over mailing list patches.

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

Have remaining drivers default to digits=2 for analog values.

The default so far was 0, which meant there would be no significant
digits at all, yielding results that looked strange/wrong to the user.

Long-term all remaining drivers should be fixed to use the actual,
correct digits and spec_digits values according to the device's
capabilities and/or datasheet/manual. Until that is done, a default
of digits=2 is used as a temporary workaround.

This fixes the remaining parts of bug #815.

Document if or why sometimes digits/spec_digits is 0.

appa-55ii: properly set encoding digits also for log memory reception

analog: improve output readability by using SI prefix

gwinstek-gds-800: properly set encoding digits

hantek-dso: properly set encoding digits

hantek-6xxx: properly set encoding digits

hung-chang-dso-2100: properly set encoding digits

baylibre-acme: properly set encoding digits

tondaj-sl-814: properly set encoding digits

kecheng-kc-330b: properly set encoding digits

colead-slm: properly set encoding digits

cem-dt-885x: properly set encoding digits

gmc-mh-1x-2x: properly set encoding digits

README.devices: Update firmware related information.

document encoding.digits and spec.spec_digits

mic-985xx: fix encoding digits

uni-t-ut32x: properly set encoding digits

lascar-el-usb: properly set encoding digits

center-3xx: properly set encoding digits

kern-scale: properly set encoding digits

demo: properly set encoding digits

motech-lps-30x: properly set encoding digits

manson-hcs-3xxx: properly set encoding digits

korad-kaxxxxp: properly set encoding digits

atten-pps3xxx: properly set encoding digits

deree-de5000: properly set encoding digits

rigol-ds: properly set encoding digits

scpi-pps: properly set encoding digits

maynuo-m97: properly set encoding digits

vc870: properly set encoding digits

ut71x: properly set encoding digits

ut372: properly set encoding digits

rs9lcd: properly set encoding digits

metex14: properly set encoding digits

This closes bug #815.

m2110: properly set encoding digits

fs9922: properly set encoding digits

fs9721: properly set encoding digits

es519xx: properly set encoding digits

dtm0660: properly set encoding digits

brymen-bm25x: properly set encoding digits

brymen-bm86x: properly set encoding digits

brymen-bm86x: remove redundant initilialization

appa-55ii: properly set encoding digits

hantek-6xxx: Ignore requests to set Hantek 6022BE coupling.

Here's a patch to "Ignore requests to set coupling for the Hantek 6022BE",
this clears the LIBUSB errors for me.

Also in the patch:

 - There is a crash because config_list() can be called with sdi == NULL.

   This can be reproduced by doing:
     "sigrok-cli.exe --driver hantek-6xxx --show"

 - There seems to be a very unsafe loop in config_set() when setting COUPLING;
   the coupling vector is assumed to be zero terminated, but is not declared
   as such.

   Note: The same issue is present also for other hardware, at least for
   hantek-dso/api.c. The patch is only for hantek-6xxx though.

hantek-6xxx: sdi might be NULL - don't assign devc until !sdi is checked.

hantek-6xxx: Fix the device scan to not scan the NULL entry.

drivers: don't try to access the sr_driver_list section with no driver compiled.

This closes bug #820.

hantek-6xxx: only show DC coupling for the Hantek device

hantek-6xxx: Fix some issues by using power-of-two data sizes.

There were issues when using non-power-of-two data sizes with e.g.
the Hantek 6022BE device. For example, on Windows the acquisition would
simply hang and never complete:

  hantek-6xxx: receive_transfer(): status LIBUSB_TRANSFER_ERROR received 0 bytes

The issue was reported by Erik Montnemery on the mailing list, the
original patch was posted by "mmark" here (thanks!):

  http://www.eevblog.com/forum/testgear/sainsmart-dds120-usb-oscilloscope-(buudai-bm102)/msg911729/#msg911729

The issue has been verified by me on Windows and Linux, and also that
this change does indeed fix it (tested Hantek 6022BE and Sainsmart DDS120).

Neither PulseView nor sigrok-cli hang anymore on Windows, and on Linux
the log messages suggest improvements as well:

  -hantek-6xxx: data_amount 712
  +hantek-6xxx: data_amount: 200 (rounded to power of 2: 512)

  -hantek-6xxx: receive_transfer(): calculated samplerate == 2327ks/s
  -hantek-6xxx: receive_transfer(): status LIBUSB_TRANSFER_OVERFLOW received 512 bytes.
  +hantek-6xxx: receive_transfer(): calculated samplerate == 1969ks/s
  +hantek-6xxx: receive_transfer(): status LIBUSB_SUCCESS / LIBUSB_TRANSFER_COMPLETED received 512 bytes.

This fixes bug #821.

Fix a segfault with input/output modules.

Some functions in std.c were using

  const char *prefix = sdi->driver->name;

but were called from input/output modules as well (which don't have
a "driver" field).

As a temporary workaround, use "unknown" as prefix in such cases until
a more permanent solution is implemented.

This fixes bug #813.

ensure the only entry point of libdrivers doesn't have visibility=hidden

This fixes the following build issue on OSX:
Undefined symbols for architecture x86_64:
  "_sr_drivers_init", referenced from:
      _sr_init in backend.o

This closes bug #802.

Add VID/PID for the CWAV USBee ZX.

The hardware is identical to the SX.

This fixes bug #764.

fx2lafw: Set sr_analog_encoding.digits to 2 for now.

fx2lafw: Fix analog scaling.

Thanks to Patrick Remy <redacted> for the fix.

session.c: Drop unneeded check.

The dev_acquisition_stop() callback is not optional and always present.

uni-t-dmm: Drop unused UNI_T_UT_D04_NEW #define.

agilent-dmm: add support for U128x

agilent-dmm: add support for Keysight branded meters

agilent-dmm: add support for AC/DC flags in current mode

agilent-dmm: fix handling of AC/DC flags in volts mode

agilent-dmm: output correct number of digits in analog packets

agilent-dmm: split DIOD recv_conf into model specific functions

This reduce overall number of lines and facilitate computation
of significant digits.

agilent-dmm: replace divider by exponent to ease computing of significant digits

analog: add support for negative number of digits

When a meter display 105.2 kΩ, libsigrok will return 105200 Ω
but it is really valuable to know that the last 2 digits are not
significant, so encoding.digits should be set to -2.
This would allow a sigrok client to display 105200 as 105.2 k
instead of 105.200 k.

Logging: Filter out unwanted newlines

055804e89ea0f6b4145040a0eeb2f4e6951692fe changed the outgoing
SCPI message termination by always adding a newline. This results
in the following log output:

sr: [00:00.003102] scpi: Opening VXI device vxi/192.168.178.43.
sr: [00:00.005648] scpi_vxi: Successfully sent SCPI command: '*IDN?
'.
sr: [00:00.005931] scpi: Got response: 'YOKOGAWA,710130,91HC30402,F3.73', length 31.

This patch restores the previous unterminated SCPI message logging:

sr: [00:00.005462] scpi: Opening VXI device vxi/192.168.178.43.
sr: [00:00.007515] scpi_vxi: Successfully sent SCPI command: '*IDN?'.
sr: [00:00.007860] scpi: Got response: 'YOKOGAWA,710130,91HC30402,F3.73', length 31.

As it's located in the general logging mechanism, we deal with any
additional (and unwanted) newlines this way.

rigol-ds: properly report which channel is enabled.

rigol-ds: add support for getting/setting trigger level.

rigol-ds: add PROBE_FACTOR support.

rigol-ds: split out vertical config reading in its own function for re-use.

group all drivers into a single object

This single object also contains the sr_drivers_init function, that will
always be referenced. That ensures that the drivers object files won't
be optimized out during static linking due to the fact that they are
not referenced directly.

This addresses (parts of) bug #802.

output/analog: Drop unused fancyprint() and si_printf().

These functions were only used in the SR_DF_ANALOG_OLD case,
whereas the SR_DF_ANALOG case already used functions and lists
from src/analog.c.

This closes bug #636.

session: Drop SR_DF_ANALOG_OLD support completely.

This closes bug #728.

transform/scale: Drop support for SR_DF_ANALOG_OLD.

transform/invert: Drop support for SR_DF_ANALOG_OLD.

output/analog: Drop support for SR_DF_ANALOG_OLD.

output/csv: Drop support for SR_DF_ANALOG_OLD.