Skip to content

Fix typo on CTS/RTS bit mask. #12

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

Closed
wants to merge 1 commit into from
Closed

Fix typo on CTS/RTS bit mask. #12

wants to merge 1 commit into from

Conversation

nseidle
Copy link
Member

@nseidle nseidle commented Jul 14, 2020

UART_CR_CTSEN_Msk is not defined. Looks like a copy/paste error from Apollo(0) files. Flow control is configured based on UART0_CR_CTSEN_Msk + instance offset.

This PR is in response to request by @oclyke on PR 230: sparkfun/Arduino_Apollo3#230

UART_CR_CTSEN_Msk is not defined. Looks like a copy/paste error from Apollo(0) files. Flow control is configured based on UART0_CR_CTSEN_Msk + instance offset.
@oclyke oclyke changed the base branch from master to 2.4.2-patch-flow-control-defines July 14, 2020 21:03
@oclyke oclyke changed the base branch from 2.4.2-patch-flow-control-defines to master July 14, 2020 21:03
@oclyke
Copy link
Contributor

oclyke commented Jul 14, 2020

Thanks Nate - closing in favor of #13 because your branch was based on this repo's master branch which already contains other fixes.

Our strategy is to keep individual patches in aptly named branches (2.4.2-patch-flow-control-defines) based on the original (2.4.2-archive) branch. This makes it really clear what changes to the original SDK version were necessary.

Then we make sure all the patches get merged into the master branch so that default usage will get our fixes.

@oclyke oclyke closed this Jul 14, 2020
@oclyke oclyke deleted the fixFlowControl branch July 14, 2020 21:45
@nseidle
Copy link
Member Author

nseidle commented Jul 15, 2020

Roger. No problem and makes sense.

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