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

tests: drivers: uart: Add stress test for async API using 2 UART devices #73638

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

Conversation

nordic-krch
Copy link
Contributor

@nordic-krch nordic-krch commented Jun 3, 2024

Test is using flow control and tries various ways of asynchronous API usage. One of the tests is using similar way of UART handling as HCI UART.
Test revealed few issues in nrfx_uarte driver used by the UART shim and PR includes hal_nordic update with the fixes.

PR contains few improvements in busy_sim (#73637).

@zephyrbot zephyrbot added platform: nRF Nordic nRFx area: Testsuite Testsuite area: UART Universal Asynchronous Receiver-Transmitter labels Jun 3, 2024
@zephyrbot
Copy link
Collaborator

zephyrbot commented Jun 3, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff

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

@zephyrbot zephyrbot added manifest manifest-hal_nordic DNM This PR should not be merged (Do Not Merge) labels Jun 3, 2024
drivers.uart.async_dual:
harness: ztest
harness_config:
fixture: gpio_loopback
Copy link
Member

Choose a reason for hiding this comment

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

Might be a good idea to add another fixture as requirements since the test needs two UARTS connecting to each other.

Copy link

github-actions bot commented Aug 3, 2024

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Aug 3, 2024
@nordic-piks nordic-piks removed the Stale label Aug 5, 2024
@zephyrbot zephyrbot removed manifest manifest-hal_nordic DNM This PR should not be merged (Do Not Merge) labels Aug 13, 2024
Add test which is using two independent UART devices. Validate behavior of
asynchronous API.

Signed-off-by: Krzysztof Chruściński <[email protected]>
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Testsuite Testsuite area: UART Universal Asynchronous Receiver-Transmitter platform: nRF Nordic nRFx Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants