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

No flow-control on TARGET_LPC55S69 #10400

Closed
teetak01 opened this issue Apr 15, 2019 · 8 comments

Comments

Projects
None yet
5 participants
@teetak01
Copy link
Contributor

commented Apr 15, 2019

Description

It would seem that either the board does not support flow-control, or Mbed OS is not enabling it.

https://github.com/ARMmbed/mbed-os/blob/master/targets/TARGET_NXP/TARGET_MCUXpresso_MCUS/TARGET_LPC55S69/TARGET_LPCXpresso/PeripheralPins.c#L65-L85

This would be required to enable stable network connectivity with AT-based ESP8266 Wi-Fi.

Related with internal ticket: https://jira.arm.com/browse/IOTHAL-483

Issue request type

[ ] Question
[ ] Enhancement
[X] Bug
@teetak01

This comment has been minimized.

Copy link
Contributor Author

commented Apr 15, 2019

@ciarmcom

This comment has been minimized.

Copy link
Member

commented Apr 15, 2019

@maclobdell

This comment has been minimized.

Copy link
Contributor

commented Apr 18, 2019

@mmahadevan108 - this one is urgent for our device management team. it will enable reliable communication to a uart-based external module.

It looks like the serial_set_flow_control function is not implemented for this target.

The definition is here
https://github.com/ARMmbed/mbed-os/blob/master/hal/serial_api.h

Also PeripheralPins.c doesn't have pins for flow control.

const PinMap PinMap_UART_CTS[] = {
    {NC   , NC    , 0}
};

const PinMap PinMap_UART_RTS[] = {
    {NC   , NC    , 0}
};

Based on the reference manual and schematic, I believe the following pins are available on the header and support UART function with flow control using the Flexcomm 2 peripheral.

TX = P0_27
RX = P1_24
CTS = P1_26
RTS = P1_27

@mmahadevan108

This comment has been minimized.

Copy link
Contributor

commented Apr 22, 2019

P1_26 and P1_27 are not connected to the Arduino headers

@mmahadevan108

This comment has been minimized.

Copy link
Contributor

commented Apr 23, 2019

I can provide an implementation using Flexcomm 0 (UART0) that uses the below
TX = P1_6
RX = P1_5
RTS = P1_7
CTS = P1_8

@maclobdell

This comment has been minimized.

Copy link
Contributor

commented Apr 24, 2019

I have a patch from Mahesh that I'm testing. Will share results soon.

@maclobdell

This comment has been minimized.

Copy link
Contributor

commented Apr 29, 2019

@mmahadevan108 I tested your patch. It works. Please make a PR!

@maclobdell

This comment has been minimized.

Copy link
Contributor

commented May 7, 2019

The PR is merged now. Closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.