Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed latency issue with serial port #116

Closed

Conversation

astumpf
Copy link

@astumpf astumpf commented Sep 11, 2017

This patch fixes the communication timeouts due to increased latency settings in newer Ubuntu kernels. The patch is based on http://projectgus.com/2011/10/notes-on-ftdi-latency-with-arduino/.

@astumpf
Copy link
Author

astumpf commented Sep 12, 2017

For some reason the programmatic solution does not work on every system. Although the latency timer was set to 1 which can be verified with

cat /sys/bus/usb-serial/devices/ttyUSB0/latency_timer

it does not influence the system behvoir at all.

However,

echo 1 | sudo tee /sys/bus/usb-serial/devices/ttyUSB0/latency_timer

seems to work always so far. Does anyone know why?

@LeonJung
Copy link
Contributor

@astumpf

Hallo, Alex,
wie gehts es dir

since I'll be in IROS next week,
My handling for this issue will be started from Octover 20th, I guess

entschuldige and I hope that you have patience on it

@astumpf
Copy link
Author

astumpf commented Sep 15, 2017

Adding udev rule
ACTION=="add", SUBSYSTEM=="usb-serial", DRIVER=="ftdi_sio", ATTR{latency_timer}="1"
works perfectly for now. However, programatic solution is preferred.

LeonJung added a commit that referenced this pull request Nov 24, 2017
…the default latency timer from 16 msec to 2 msec (now the latency timer should be adjusted by yourself... see port_handler_linux source code to see details) #116
@LeonJung
Copy link
Contributor

@astumpf

Hi,
what you said was perfect. I put some description for latency timer settings in the code comments.

@astumpf
Copy link
Author

astumpf commented Nov 27, 2017

Awesome und vielen Dank!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants