osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/41177?usp=email ) Change subject: ccid_host: fix read_timeout defined but not used ...................................................................... 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(-) Approvals: laforge: Looks good to me, but someone else must approve fixeria: Looks good to me, approved Jenkins Builder: Verified 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 https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/41177?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email 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>