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

Added dtr/sdr management #70

Open
wants to merge 1 commit into
base: v4.2.x
Choose a base branch
from
Open

Conversation

FalsinSoft
Copy link

Hi
Since was not implemented I added the dtr/sdr management for both Windows and Linux
Thank you for your library

@FalsinSoft
Copy link
Author

Sorry, just noted I made a mistake in wrote commit message. It's rts instead of sdr

@FalsinSoft
Copy link
Author

Hi, are you interested in this patch? If not just tell me and I'll close it...

@itas109
Copy link
Owner

itas109 commented Mar 20, 2023

Thank you very much for your patch.
I don't have a hardware device to test it.
And There is a corresponding relationship between flowControl and dtr/rts.
So I need some time to study it.

@FalsinSoft
Copy link
Author

Hi
I tested this patch on both Windows and Linux and seems to work as expected. In particular this patch is needed if you want to use your library to communicate with Arduino that require dtr to be set to work.

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.

None yet

2 participants