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

Add uart api call to use xon/xoff flow control #890

Closed

Conversation

Ealdwulf
Copy link
Contributor

@Ealdwulf Ealdwulf commented Aug 8, 2017

The uart driver has support for hardware flow control, but we (Kano Computing) are using a flow control pin to control device reset, which I guess is a common use case. We'd therefore like to use software flow control. Since the hardware has this build in, I've made this PR to switch it on.

We are not especially attached to this API and would be fine with anything which does the same job.

@CLAassistant
Copy link

CLAassistant commented Aug 8, 2017

CLA assistant check
All committers have signed the CLA.

@projectgus
Copy link
Contributor

Hi @Ealdwulf ,

Thanks for sending this. Are you able to click-through and sign our CLA as prompted, please? Once CLA is signed we can look at merging the PR.

Angus

@Ealdwulf
Copy link
Contributor Author

Thanks @projectgus, my manager confirmed this was okay, so I have signed it.

@projectgus projectgus added the Status: Pending blocked by some other factor label Sep 5, 2017
@projectgus
Copy link
Contributor

Thanks @Ealdwulf . Have cherry-picked this into our internal queue for review & merging.

igrr pushed a commit that referenced this pull request Sep 18, 2017
igrr added a commit that referenced this pull request Sep 18, 2017
uart: Add api call to switch on hardware support for 'software' flow control. (github #890)

See merge request !1215
@projectgus
Copy link
Contributor

Cherry-picked as 23f10e1.

@projectgus projectgus closed this Sep 20, 2017
@Ealdwulf
Copy link
Contributor Author

Cheers!

@Ealdwulf Ealdwulf deleted the Upstreaming/uart_flow_control branch September 20, 2017 09:58
chmorgan pushed a commit to chmorgan/esp-idf that referenced this pull request Sep 20, 2017
@igrr igrr removed the Status: Pending blocked by some other factor label Nov 14, 2017
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.

4 participants