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

Changed wording of setDTR(), clearDTR, setRTS(), and clearRTS() to re… #548

Merged
merged 7 commits into from
Mar 20, 2024

Conversation

jskubick
Copy link
Contributor

Updated javadocs for setDTR(), clearDTR(), setRTS(), and clearRTS() to reduce confusion when library is used to drive common USB-UART bridges like the SiLabs CP210x family (where DTR and RTS are active low, so asserting them to 1 drives the pin low, and clearing it to 0 drives it high).

Sources:
https://www.silabs.com/documents/public/data-sheets/cp2102n-datasheet.pdf page 28 (which states that DTR and RTS are active low)
the book "Serial Port Complete" (page 364), which describes RTS and DTR as

  • 0: de-assert (RS-232 negative voltage)
  • 1: assert (RS-232 positive voltage)

…duce confusion when used with common USB-UART bridges.
…d with USB serial adapters and Arduino/esp32 boards.
…ud rates > 230400 on Apple computers & elaborate upon the constraints imposed by USB-serial adapters upon baudrate selection.

USB-serial baudrate is one of those things that generally "just works" at or below 115200 baud, but can fail at faster rates for reasons that aren't intuitive or obvious until you discover that the data is "really" being transferred across the USB bus in 64-byte chunks once per millisecond.
@hedgecrw hedgecrw merged commit 7c9ce1c into Fazecast:master Mar 20, 2024
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