Skip to content

tests: drivers: uart: Add test for async API using two instance #86992

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nordic-krch
Copy link
Collaborator

Add test which is using two independent UART devices. Validate behavior of asynchronous API.

@zephyrbot zephyrbot added the area: UART Universal Asynchronous Receiver-Transmitter label Mar 12, 2025
@zephyrbot zephyrbot requested a review from dcpleung March 12, 2025 13:18
@nordic-krch nordic-krch force-pushed the uart_async_stress branch 3 times, most recently from 5683777 to 68edcf5 Compare March 21, 2025 13:23
Add test which is using two independent UART devices. Validate behavior of
asynchronous API.

Signed-off-by: Krzysztof Chruściński <[email protected]>
@kartben kartben requested a review from Copilot April 18, 2025 19:12
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 a new YAML testcase to validate the asynchronous UART API using two independent UART devices with various configuration options.

  • Adds four test configurations under drivers.uart.async_dual targeting different runtime and simulation cases
  • Specifies platform allowances, dependencies, and extra configuration flags for each test variant
Files not reviewed (15)
  • tests/drivers/uart/uart_async_dual/CMakeLists.txt: Language not supported
  • tests/drivers/uart/uart_async_dual/Kconfig: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf52_bsim.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54h20dk_nrf54h20_common.dtsi: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54h20dk_nrf54h20_cpuapp.conf: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54h20dk_nrf54h20_cpuapp.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54h20dk_nrf54h20_cpuppr.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54h20dk_nrf54h20_cpurad.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54l15dk_nrf54l15_cpuapp.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf54l15pdk_nrf54l15_cpuapp.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf9160dk_nrf9160.conf: Language not supported
  • tests/drivers/uart/uart_async_dual/boards/nrf9160dk_nrf9160.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/prj.conf: Language not supported
  • tests/drivers/uart/uart_async_dual/sysbuild/vpr_launcher/boards/nrf54h20dk_nrf54h20_cpuapp.overlay: Language not supported
  • tests/drivers/uart/uart_async_dual/sysbuild/vpr_launcher/prj.conf: Language not supported
Comments suppressed due to low confidence (1)

tests/drivers/uart/uart_async_dual/testcase.yaml:62

  • [nitpick] Consider aligning the naming convention across test variants for clarity. The use of 'explicit' may benefit from a brief inline comment or renaming to better convey its intent relative to 'busy_sim'.
  drivers.uart.async_dual.pm_runtime.explicit:

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants