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

fix(BLE): Re-implement UART with DMA on MAX32690 #1280

Merged
merged 16 commits into from
Dec 9, 2024
Merged

Conversation

crsz20
Copy link
Contributor

@crsz20 crsz20 commented Nov 26, 2024

Description

Initializing a UART peripheral with MXC_UART_IBRO_CLK would cause the MAX32690 to occasionally fail to assert a half-empty interrupt signal. This is a hardware issue.

In practice, this causes the Tx buffer to get stuck and eventually overflows. This is evident in the BLE examples with a large amount of serial output.

Solution

This device's pal_uart.c in Cordio has been re-implemented with DMA to rely on other interrupt signals to handle buffer management more reliably.

Remarks

Eric added a PAL_UART example to help test echoing in the serial output.

@crsz20 crsz20 added WIP work in progress MAX32690 Related to the MAX32690 (ME18) BLE Related to Bluetooth labels Nov 26, 2024
@crsz20
Copy link
Contributor Author

crsz20 commented Nov 26, 2024

/clang-format-run

@crsz20 crsz20 requested review from yc-adi and EricB-ADI November 27, 2024 15:34
@EricB-ADI
Copy link
Contributor

Remove the PAL_UART Example

@github-actions github-actions bot removed the MAX32690 Related to the MAX32690 (ME18) label Nov 27, 2024
@crsz20 crsz20 added the MAX32690 Related to the MAX32690 (ME18) label Nov 27, 2024
@crsz20 crsz20 merged commit ae0b07b into main Dec 9, 2024
8 checks passed
sihyung-maxim pushed a commit to analogdevicesinc/hal_adi that referenced this pull request Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLE Related to Bluetooth MAX32690 Related to the MAX32690 (ME18) WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants