uni-t-ut32x: drop redundant close and free at end of scan

The UT32x driver requires a user spec for the connection. The device
cannot get identified, that's why successful open/close for the port
will suffice. Lack of an input spec as well as failure in the early
scan phase will terminate the scan routine early.

When we reach the end of the scan which creates the device instance
and registers it with the list of found devices, the port already
is closed and the list of devices will never be empty. Remove the
redundant close call and the dead branch which frees the serial port.

diff --git a/src/hardware/uni-t-ut32x/api.c b/src/hardware/uni-t-ut32x/api.c
index 21e5be7d1a8989134bff5c9ea0b7b4c1ebdabfe0..ce81b3cc792e32b4da3b9a9d97b35565c4f1761f 100644 (file)
--- a/src/hardware/uni-t-ut32x/api.c
+++ b/src/hardware/uni-t-ut32x/api.c
@@ -114,10 +114,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        }
        devices = g_slist_append(devices, sdi);
 
-       serial_close(serial);
-       if (!devices)
-               sr_serial_dev_inst_free(serial);
-
        return std_scan_complete(di, devices);
 }