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.
This commit is contained in:
Uwe Hermann 2011-04-15 20:47:26 +02:00
parent 2f5c8c9696
commit 15f2d0c0f2
1 changed files with 0 additions and 4 deletions

View File

@ -605,7 +605,6 @@ static void hw_closedev(int device_index)
if (sdi->status == SR_ST_ACTIVE) { if (sdi->status == SR_ST_ACTIVE) {
sr_dbg("la8: %s: status ACTIVE, closing device", __func__); sr_dbg("la8: %s: status ACTIVE, closing device", __func__);
/* TODO: Handle or ignore errors here? */
(void) la8_close_usb_reset_sequencer(la8); /* Ignore errors. */ (void) la8_close_usb_reset_sequencer(la8); /* Ignore errors. */
} else { } else {
sr_dbg("la8: %s: status not ACTIVE, nothing to do", __func__); 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; 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. */ /* Send end packet to the session bus. */
sr_dbg("la8: %s: sending SR_DF_END", __func__); sr_dbg("la8: %s: sending SR_DF_END", __func__);
packet.type = SR_DF_END; packet.type = SR_DF_END;