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

Documentation of the NRF uart flow control seems missing #42

Open
gautierhattenberger opened this issue Feb 3, 2020 · 3 comments
Open

Comments

@gautierhattenberger
Copy link

When developing a custom code for the F4 side of the Crazyflie 2.1 (support for Paparazzi UAV system), I struggled a bit with the communication at 1M baud between the NRF and the F4. I later realized that uart hardware flow control is enabled on the NRF side, but on the F4 side, this flow control should be done by hand (no hardware flow control for uart6).
Also, not doing it properly leads to several transmission errors making the communication with the Crazyradio not reliable.
Unless I missed this part of the documentation, I suggest to add this important information to the developer wiki/doc so that people don't face the sameo) issue in the future.

@krichardsson
Copy link
Contributor

Hi!

Thanks for the feedback!

Also, not doing it properly leads to several transmission errors making the communication with the Crazyradio not reliable.

Do you mean that you have seen transmission errors between the STM and NRF in the official Crazyflie firmware?

@gautierhattenberger
Copy link
Author

No, only when using my own firmware on F4 without the (software) flow control.
Funny thing was that it was almost working with debugging options but not with optimized code. It turns out that with debug flags, the code is slower and produce less errors (but only by chance). Using properly the RTS line makes it work without problems anymore.

@krichardsson
Copy link
Contributor

Great, thanks!

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

No branches or pull requests

3 participants