From 928621357ca283905dd973b0f97d17b41b5841ea Mon Sep 17 00:00:00 2001 From: John Van Ostrand Date: Tue, 5 Sep 2017 17:07:55 -0400 Subject: Changed backoff strategy for short reads. Back off was exponential starting at 10ms, which for high baud rate and no flow-control connections might cause buffer overrun. This was causing problems when reading Cochran DCs, the hearbeat byte was being missed. Signed-off-by: John Van Ostrand Signed-off-by: Dirk Hohndel --- core/serial_ftdi.c | 7 ------- 1 file changed, 7 deletions(-) (limited to 'core') diff --git a/core/serial_ftdi.c b/core/serial_ftdi.c index f8cba95d3..297d0fa20 100644 --- a/core/serial_ftdi.c +++ b/core/serial_ftdi.c @@ -395,19 +395,12 @@ static dc_status_t serial_ftdi_read (dc_custom_io_t *io, void *data, size_t size ERROR (device->context, "%s", ftdi_get_error_string(device->ftdi_ctx)); return DC_STATUS_IO; //Error during read call. } else if (n == 0) { - // Exponential backoff. if (slept >= timeout) { ERROR(device->context, "%s", "FTDI read timed out."); return DC_STATUS_TIMEOUT; } serial_ftdi_sleep (device, backoff); slept += backoff; - backoff *= 2; - if (backoff + slept > timeout) - backoff = timeout - slept; - } else { - // Reset backoff to 1 on success. - backoff = 1; } nbytes += n; -- cgit v1.2.3-70-g09d2