Always return status from ftdi_open when called

Using dc_serial_open as a fallback to ftdi_open is just wrong, and will
never work, just mask the real error and introduce read herrings.

Signed-off-by: Anton Lundin <glance@acc.umu.se>
This commit is contained in:
Anton Lundin 2018-09-16 13:55:37 +02:00 committed by Dirk Hohndel
parent 4b42dcc6af
commit 1da9235cdf

View file

@ -1328,11 +1328,8 @@ dc_status_t divecomputer_device_open(device_data_t *data)
if (transports & DC_TRANSPORT_SERIAL) {
#ifdef SERIAL_FTDI
if (!strcmp(data->devname, "ftdi")) {
rc = ftdi_open(&data->iostream, context);
if (rc == DC_STATUS_SUCCESS)
return rc;
}
if (!strcmp(data->devname, "ftdi"))
return ftdi_open(&data->iostream, context);
#endif
rc = dc_serial_open(&data->iostream, context, data->devname);
if (rc == DC_STATUS_SUCCESS)