]> sigrok.org Git - libsigrok.git/commitdiff
serial_hid: make --list-serial output compatible with conn=, print "VID.PID"
authorGerhard Sittig <redacted>
Thu, 20 Jun 2019 09:06:22 +0000 (11:06 +0200)
committerUwe Hermann <redacted>
Thu, 20 Jun 2019 15:21:47 +0000 (17:21 +0200)
When the list of all connections gets created which are supported by the
HID serial transport, items contain a "hid/ch9325/raw=/dev/hidraw3" path
and a "1a86:e008" pair of vendor and product IDs.

Separate the VID/PID pair by a period not a colon, so that --list-serial
output immediately becomes usable with "--driver <name>:conn=<spec>"
invocations. Eliminate the necessity to adjust clipboard context by the
user. This improves usability in cases where not a single connection
gets addressed, but a group of connections gets specified by ambiguous
conn= specs.

$ sigrok-cli -d uni-t-ut32x:conn=1a86.e008 --scan

src/serial_hid.c

index 0c5b9ccc27d5ee94a7e53b94df45f2ab57c32f67..0ad0e5130e8580acb1921b8c58fb4e21bdde8c1d 100644 (file)
@@ -206,7 +206,7 @@ static GSList *ser_hid_hidapi_list(GSList *list, sr_ser_list_append_t append)
                if (serno)
                        g_string_append_printf(desc, " %ls", serno);
                if (vid && pid)
-                       g_string_append_printf(desc, " %04hx:%04hx", vid, pid);
+                       g_string_append_printf(desc, " %04hx.%04hx", vid, pid);
                list = append(list, name, desc->str);
                g_string_free(desc, TRUE);
                g_free(name);