osmith submitted this change.

View Change

Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified
ccid_host: fix read_timeout defined but not used

Comment out the read_timeout fuction, as it is only used in already
commented out code. Fix for:

cuart_driver_tty.c:90:12: warning: 'read_timeout' defined but not used [-Wunused-function]
90 | static int read_timeout(int fd, uint8_t *out, size_t len, unsigned long timeout_ms)

Change-Id: I49fc59ebaa4abc3e9b8838e1b427f4455acdb390
---
M ccid_host/cuart_driver_tty.c
1 file changed, 2 insertions(+), 0 deletions(-)

diff --git a/ccid_host/cuart_driver_tty.c b/ccid_host/cuart_driver_tty.c
index f8a1438..a9032fa 100644
--- a/ccid_host/cuart_driver_tty.c
+++ b/ccid_host/cuart_driver_tty.c
@@ -87,6 +87,7 @@
OSMO_ASSERT(rc == 0);
}

+#if 0
static int read_timeout(int fd, uint8_t *out, size_t len, unsigned long timeout_ms)
{
struct timeval tv = { .tv_sec = timeout_ms / 1000, .tv_usec = (timeout_ms % 1000) * 1000 };
@@ -103,6 +104,7 @@

return read(fd, out, len);
}
+#endif

static int _flush(int fd)
{

To view, visit change 41177. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I49fc59ebaa4abc3e9b8838e1b427f4455acdb390
Gerrit-Change-Number: 41177
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Hoernchen <ewild@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>