Don't close/reset the FTDI device too often.

Only call la8_close_usb_reset_sequencer() in hw_closedev(), it's not
needed in hw_stop_acquisition().

Thanks Ken Mobley of ChronoVu for the report.

diff --git a/hardware/chronovu-la8/chronovu-la8.c b/hardware/chronovu-la8/chronovu-la8.c
index 98a732846812a99665c0265b784109f46996601c..13f07b063a0fced7b42d814f8447f853c87365d5 100644 (file)
--- a/hardware/chronovu-la8/chronovu-la8.c
+++ b/hardware/chronovu-la8/chronovu-la8.c
@@ -605,7 +605,6 @@ static void hw_closedev(int device_index)
 
        if (sdi->status == SR_ST_ACTIVE) {
                sr_dbg("la8: %s: status ACTIVE, closing device", __func__);
-               /* TODO: Handle or ignore errors here? */
                (void) la8_close_usb_reset_sequencer(la8); /* Ignore errors. */
        } else {
                sr_dbg("la8: %s: status not ACTIVE, nothing to do", __func__);
@@ -967,9 +966,6 @@ static void hw_stop_acquisition(int device_index, gpointer session_device_id)
                return;
        }
 
-       /* Reset the LA8 sequencer and close the device. */
-       (void) la8_close_usb_reset_sequencer(la8); /* Ignore errors. */
-
        /* Send end packet to the session bus. */
        sr_dbg("la8: %s: sending SR_DF_END", __func__);
        packet.type = SR_DF_END;