Skip to content

drivers: uart: add dma support for ambiq uart driver #90268

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 1 commit into from
May 28, 2025

Conversation

AlessandroLuo
Copy link
Collaborator

This commit adds dma support for ambiq uart driver

This commit adds dma support for ambiq uart driver

Signed-off-by: Hao Luo <[email protected]>
Copy link

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_ambiq zephyrproject-rtos/hal_ambiq@080be37 zephyrproject-rtos/hal_ambiq@f46941f (main) zephyrproject-rtos/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds DMA support to the Ambiq UART driver by updating the HAL reference and extending driver configuration options.

  • Updated hal_ambiq revision in west.yml to pull in DMA-capable HAL.
  • Enabled asynchronous UART support and introduced a cache-handling toggle in Kconfig.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
west.yml Bumped hal_ambiq revision to include DMA support
drivers/serial/Kconfig.ambiq Selected async support and added cache-handling flag
Comments suppressed due to low confidence (1)

drivers/serial/Kconfig.ambiq:20

  • [nitpick] The name UART_AMBIQ_HANDLE_CACHE can be ambiguous; consider renaming to UART_AMBIQ_CACHE_HANDLING to clarify that it controls cache management behavior.
config UART_AMBIQ_HANDLE_CACHE

Copy link

@AlessandroLuo AlessandroLuo marked this pull request as ready for review May 21, 2025 09:57
@github-actions github-actions bot added platform: Ambiq Ambiq area: UART Universal Asynchronous Receiver-Transmitter labels May 21, 2025
@AlessandroLuo
Copy link
Collaborator Author

@dcpleung please help to review, thank you!

@kartben kartben merged commit ba21058 into zephyrproject-rtos:main May 28, 2025
30 checks passed
@AlessandroLuo AlessandroLuo deleted the ambiq-uart-dma branch June 18, 2025 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: UART Universal Asynchronous Receiver-Transmitter manifest manifest-hal_ambiq platform: Ambiq Ambiq
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants