]> sigrok.org Git - libsigrok.git/commit
ols: Do not randomly probe serial ports
authorAlexandru Gagniuc <redacted>
Sat, 10 Nov 2012 01:52:45 +0000 (19:52 -0600)
committerBert Vermeulen <redacted>
Sun, 11 Nov 2012 02:12:10 +0000 (03:12 +0100)
commita99e0d2a0c9d1bb4db5623ba50f83486238ee793
tree70a83f1a06a941924f3731b6344293bff18d51dc
parent2244356d1069c5b13fe3b728a421750900deaafe
ols: Do not randomly probe serial ports

ols driver used to probe a series of available serial ports obtained
by regexp matching of common serial port names.
There are a number of problems with this approach:
1. It will probe all serial devices, including devices that do not
like to be probed, potentially causing them to act up.
2. It will try to probe serial ports which may already be opened in
other applications for other purposes.
3. It assumes the naming of the serial ports is set in stone, and
creates an unnecessary OS-specific list.
4. It produces unnecessary debug output even when an OLS device is
not connected.
5. etc...

Do not implicitly probe serial ports. Only probe the port specified
by the frontend, if any; otherwise, just quit.
Also get rid of all functionality in serial.c which was designed
specifically for random probing.

Signed-off-by: Alexandru Gagniuc <redacted>
hardware/common/serial.c
hardware/openbench-logic-sniffer/ols.c
libsigrok-internal.h