Skip to content

Added flow control for K64F #2140

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

Merged
merged 2 commits into from
Jul 12, 2016
Merged

Added flow control for K64F #2140

merged 2 commits into from
Jul 12, 2016

Conversation

bogdanm
Copy link
Contributor

@bogdanm bogdanm commented Jul 11, 2016

This commit adds hardware flow control capabilities for the K64F family
of MCUs. This is a backport of these commits:

ARMmbed/mbed-hal-ksdk-mcu@9bfcfd0
ARMmbed/mbed-hal-frdm-k64f@77042cc

with a few changes:

  • since the current version of KSDK doesn't seem to have APIs for
    manipulating the flow control settings, we change the peripheral
    registers directly.
  • updated pin maps for RTS/CTS in accordance to the K64F datasheet.

This commit adds hardware flow control capabilities for the K64F family
of MCUs. This is a backport of these commits:

ARMmbed/mbed-hal-ksdk-mcu@9bfcfd0
ARMmbed/mbed-hal-frdm-k64f@77042cc

with a few changes:

- since the current version of KSDK doesn't seem to have APIs for
  manipulating the flow control settings, we change the peripheral
  registers directly.
- updated pin maps for RTS/CTS in accordance to the K64F datasheet.
@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 11, 2016

@0xc0170

@hasnainvirk
Copy link
Contributor

I have a fix for your 'fix'. But I can't make push to k64f_flow_control branch as I don't have permissions.

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 11, 2016

If your fix is not large, please post it here and I'll apply it manually. Otherwise I'll create a fork and we'll work there.

@hasnainvirk
Copy link
Contributor

hasnainvirk commented Jul 12, 2016

In serial_api.c line no. 304, change the register from UART_MODEM_TXRTSE to UART_MODEM_TXCTSE
Tested and verified with this change. Good to go after this change.

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 12, 2016

Thanks! Fix pushed, please check if it works.

@bogdanm
Copy link
Contributor Author

bogdanm commented Jul 12, 2016

@0xc0170, looks good ?

@0xc0170
Copy link
Contributor

0xc0170 commented Jul 12, 2016

LGTM

@bogdanm bogdanm merged commit 6f86064 into master Jul 12, 2016
@bogdanm bogdanm deleted the k64f_flow_control branch July 12, 2016 10:07
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.

3 participants