atten-pps3xxx: Use serial_timeout().

serial: Add serial_timeout().

This calculates a proper timeout value for blocking writes on the
given serial port, for the given number of bytes. Timeout is based
on a fixed 10ms OS overhead, baud rate, data bits and stop bits.

atten-pps3xxx: Fix serial port timeout.

fluke-dmm: Fix blocking serial write timeout.

agilent-dmm: Fix blocking serial write timeout.

input/csv: Fix size_t printing.

unit tests: Fix build due to recent changes.

Add a configurable read timeout to blocking SCPI reads, default 1s.

Add some Agilent USBTMC PIDs.

Add a timeout parameter to blocking serial calls.

Set this new parameter to 0 (no timeout) at every call site. This is
consistent with previous behaviour, so cannot cause any regressions.

Waiting forever for a serial operation is clearly always wrong. Without
specific knowledge of each device and driver however, I can't choose
appropriate timeouts for each call. The maintainers of these drivers
will need to do so, and also add appropriate handling of timeouts.

When this commit is merged, a bug should be entered for each driver
that is touched by it.

Avoid serial_write_blocking() warnings.

Fix similar broken error handling on several serial calls.

kecheng-kc-330b: Fix missing time/frequency weighting.

hantek-dso: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

demo: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

fluke-dmm: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

chronovu-la8: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

asix-sigma: Trigger code cleanup.

This cleans up a warning generated by clang's static analyzer.

agilent-dmm: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

scpi/vxc: Avoid NULL dereference.

This cleans up a warning generated by clang's static analyzer.

input/csv: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

input/chronovu_la8: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

input/binary: Code cleanup.

This cleans up a warning generated by clang's static analyzer.

input: Avoid NULL dereference.

This cleans up a warning generated by clang's static analyzer.

sr_parse_sizestring: Deal with invalid strings better.

This cleans up a warning generated by clang's static analyzer.

Clean up sr_session_load().

This cleans up some warnings generated by clang's static analyzer.
The function now also returns SR_ERR to signify the specified filename
does not point to a valid session file.

Other SR_ERR_* returns indicate a session file was found, but loading
failed.

Return sensible Device::description() for session and input devices.

C++: Fix management of SessionDevice objects.

C++: Add spaces in Device::description().

C++: Expose device serial number and connection ID.

C++: Expose config key capabilities.

bindings: Update for input API changes.

Clean up internal input API docs.

input: Use SR_ERR_NA instead of SR_OK_CONTINUE.

input: sr_input_free() is now a void function.

Its backend, input_module.cleanup(), is now also a void function.

input: Add sr_input_end().

This signifies to the module instance no more input will come. This
will cause the module to process any data it may have buffered. The
SR_DF_END packet will also typically be sent at this time.

input: Free instance-private storage at instance free.

input: Add sdi_ready flag to struct sr_input.

When an input module instance has received enough input to fully
populate the struct sr_dev_inst, sdi_ready is set to TRUE and its
receive() method returns immediately. Any remaining received data
is buffered until the next time the function is called.

Removal of sdi->index, step 8: fix victor-dmm

Removal of sdi->index, step 7: fix testo

Removal of sdi->index, step 6: fix sysclk-lwla

Removal of sdi->index, step 5: fix ikalogic-scanalogic2

Removal of sdi->index, step 4: fix trivial sr_dev_inst_new() calls

Removal of sdi->index, step 3: sr_dev_inst_new() calls for input mods

Removal of sdi->index, step 2: remove it from sr_session_load()

Removal of sdi->index, step 1: remove it from headers and helper funcs

saleae-logic16: Use physical USB connection instead of sdi->index

Remove sdi->index from openbench-logic-sniffer and pipistrello-ols

hantek-dso: Use physical USB connection instead of sdi->index

zeroplus-logic-cube: Use physical USB connection instead of sdi->index

C++: handle uint32_t SR_CONF keys which contain key capabilities

This adapts the C++ bindings according to commits 584560f and 5827f61.

fx2lafw: Use physical USB connection instead of sdi->index

Previously, sdi->index was used to tell several identical fx2lafw-compatible
devices apart. This was a bit of a hack, so this patch makes it use physical
device connections instead. They're guaranteed to remain the same even if
the USB device reconnects.

Add sr_packet_copy/_free functions.

These will be used to make a copy of a packet on the session bus,
valid for the duration of the device instance that generated it.

serial: Eliminate serial_read(), serial_write() and SERIAL_NONBLOCK.

std_serial_dev_open: Remove SERIAL_NONBLOCK flag.

All calls in drivers are now explicitly (non)blocking.

scpi_serial: Mark read/write calls nonblocking, remove SERIAL_NONBLOCK.

These calls were already nonblocking. Having marked them as such, the flag can
be removed.

tondaj-sl-814: Remove SERIAL_NONBLOCK flag.

All calls in the driver are now explicitly (non)blocking.

tondaj-sl-814: Mark serial read/write calls as blocking.

These calls are executed from an event handler and were previously nonblocking,
but they have no partial read/write handling. The code is already marked TODO
for improvement.

tondaj-sl-814: Mark serial read call as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. Partial reads are handled.

teleinfo: Mark serial read as nonblocking, remove SERIAL_NONBLOCK flag.

This call was already nonblocking due to the flag.

openbench-logic-sniffer: Remove SERIAL_NONBLOCK flag.

All calls in this driver are already explicitly (non)blocking.

norma-dmm: Remove SERIAL_NONBLOCK flag.

All calls in the driver are now explicitly (non)blocking.

norma-dmm: Make serial write call block.

This call is executed from an event handler and was previously nonblocking,
but has no partial write handling. It sends a short packet so should be OK
to block, most likely the output buffer will be empty anyway.

norma-dmm: Make write call block.

This call was previously nonblocking, but has no partial write handling.
It is called in the scan where it is free to block.

norma-dmm: Mark serial read call as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. It only reads one byte, and a zero result is handled
appropriately.

motech-lps-30x: Remove SERIAL_NONBLOCK.

All calls in the driver are now explicitly (non)blocking.

motech-lps-30x: Make serial write call block.

This call was previously explicitly nonblocking, but has no partial write
handling. It sends a short packet so should be OK to block, most likely the
output buffer will be empty anyway.

motech-lps-30x: Mark serial read call as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. It only reads one byte, and a zero result is handled
appropriately.

mic-985xx: Remove SERIAL_NONBLOCK call.

All calls in the driver are now explicitly (non)blocking.

mic-985xx: Make serial write call block.

This call is executed from an event handler and was previously nonblocking,
but has no partial write handling. It sends a short packet so should be OK
to block, most likely the output buffer will be empty anyway.

mic-985xx: Mark serial read as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. Partial reads are handled.

gmc-mh-1x-2x: Remove SERIAL_NONBLOCK flag.

All calls in the driver are now explicitly (non)blocking.

gmc-mh-1x-2x: Make serial write calls block.

These calls are executed from an event handler and were previously nonblocking,
but have no partial write handling. They send short packets so should be OK to
block, most likely the output buffer will be empty anyway.

Fix error handling for some: serial_write can return any negative error code.

gmc-mh-1x-2x: Make serial write call block, and fix error handling.

This call was previously nonblocking, but there is no handling of partial
writes. It is called from config_set where it is free to block.

Also fix error handling: serial_write can return any negative error code,
not just -1.

gmc-mh-1x-2x: Mark serial read calls as nonblocking.

These calls were already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. They only read one byte. A return value of zero is not
handled, but should not occur in theory due to the G_IO_IN check. It might be
good to add handling of a zero return anyway, since I'm not sure if this is
always accurate.

gmc-mh-1x-2x: Mark serial read call as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. It only reads one byte, and a zero result is handled
appropriately.

fluke-dmm: Make serial write calls block, and fix error handling.

These calls are executed from an event handler and were previously nonblocking,
but have no partial write handling. They send short packets so should be OK to
block, most likely the output buffer will be empty anyway.

Also fix error handling for these calls, which seems to have been retained from
previous direct usage of write() to a serial port fd.

fluke-dmm: Make serial write calls block, and fix error handling.

These calls were previously nonblocking, but have no partial write handling.
They are made from scan and acquisition_start contexts where they are free
to block.

Remove the SERIAL_NONBLOCK at open, which only applied during scan, since all
calls in the scan are now explicitly blocking.

Also fix error handling for these calls, which appears to have been kept
from a previous direct usage of write() on a serial port fd.

fluke-dmm: Mark serial read call as nonblocking.

This call was already nonblocking since the driver opens the port with the
SERIAL_NONBLOCK flag. Only one byte is read. The case of 0 being returned
is not handled, but the call is only made if G_IO_IN occurred so in theory,
there should be a byte available. It might be wise to add handling for a
return of 0 nonetheless, as I'm not sure if this is always accurate.

conrad-digi-35-cpu: Make serial write call block.

This call was previously nonblocking, but there is no partial write handling.
It is only called from config_set so it is free to block.

conrad-digi-35-cpu: Remove SERIAL_NONBLOCK flag from open at scan time.

This open call is only made to check access, the port is closed immediately
afterwards. The flag therefore has no effect.

colead-slm: Mark serial read calls as nonblocking.

This is an odd one. These calls are made from a receive handler so should not
block, and appear to be setup correctly to handle partial reads or no data
available. However, the driver was not opening the port with SERIAL_NONBLOCK
so these calls would have been blocking. Make them nonblocking.

center-3xx: Remove SERIAL_NONBLOCK flag.

All serial calls in this driver are now explicitly (non)blocking.

center-3xx: Make serial write call block.

This call is executed from an event handler and was previously nonblocking,
but has no partial write handling. It sends a short packet so should be OK
to block, most likely the output buffer will be empty anyway.

center-3xx: Mark serial read call as nonblocking.

This call was already nonblocking because the driver opens the port with the
SERIAL_NONBLOCK flag. Partial reads are handled.

cem-dt-885x: Mark serial access as nonblocking, remove SERIAL_NONBLOCK flag.

These calls were already nonblocking since this driver opened the port with
the SERIAL_NONBLOCK flag. Having marked them as such, we can remove the flag.

Also remove an unnecessary reopen of the port to change its blocking status.

brymen-dmm: Remove SERIAL_NONBLOCK flag.

All serial read/write calls in the driver are now explicitly (non)blocking.

brymen-dmm: Make serial write call block.

This call is executed from an event handler and was previously nonblocking,
but has no partial write handling. It sends a short packet so should be OK
to block, most likely the output buffer will be empty anyway.

brymen-dmm: Mark serial read calls as nonblocking.

The driver opens the port with SERIAL_NONBLOCK so these were already
nonblocking, and are handled appropriately.

atten-pps3xxx: Remove SERIAL_NONBLOCK.

All serial read/write calls in the driver are now explicitly (non)blocking.

atten-pps3xxx: Make serial write call block.

This call is executed from an event handler and was previously nonblocking,
but has no partial write handling. It sends a short packet so should be OK
to block, most likely the output buffer will be empty anyway.

atten-pps3xxx: Make serial write call block in scan.

This is called at scan time so free to block. There is no partial write
handling and a response is expected afterwards. This should therefore be a
blocking call.

appa-55ii: Mark serial read nonblocking and remove SERIAL_NONBLOCK.

This is the only read/write call in the driver. It is already non-blocking
and is handled appropriately.

aglient-dmm: Remove SERIAL_NONBLOCK flag.

All serial calls in this driver are now explicitly (non)blocking.

agilent-dmm: Make serial write call block in send.

This call is executed from an event handler context was previously
nonblocking, however there is no handling for a partial write.

The output buffer is unlikely to be full and the commands to be sent
are short, so it should be OK to make this a blocking call.

agilent-dmm: Make serial write call block in scan.

This call is executed at scan time so is free to block. There is no
handling for a partial write and a response is expected immediately
afterwards. It should therefore be a blocking call.

agilent-dmm: Mark serial read call as nonblocking.

This driver opens the port with the SERIAL_NONBLOCK flag, so this call is
already a nonblocking one, and is handled appropriately.

serial-dmm: Remove SERIAL_NONBLOCK flag.

All serial read/write calls in this driver are now explicitly
nonblocking so there is no need for this flag.

serial-dmm: Mark serial read/write calls as nonblocking.

This driver opens the port with the SERIAL_NONBLOCK flag so these calls were
already non-blocking.

colead-slm: Mark serial write call as blocking.

This driver opens the port without the SERIAL_NONBLOCK flag,
so this call was already blocking.