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: uart_pm: Fix fail on nrf54h20 #79268

Conversation

nordic-segl
Copy link
Contributor

Power states were recently added to the board definition of nrf54h20:
5b607ed
This has negative impact on drivers.uart.pm.int_driven test.

Delete property 'cpu-power-states' in the test overlay for nrf54h20.

Power states were recently added to the board definition of nrf54h20:
zephyrproject-rtos@5b607ed
This has negative impact on drivers.uart.pm.int_driven test.

Delete property 'cpu-power-states' in the test overlay for nrf54h20.

Signed-off-by: Sebastian Głąb <[email protected]>
@zephyrbot zephyrbot added the area: UART Universal Asynchronous Receiver-Transmitter label Oct 1, 2024
@nordic-segl
Copy link
Contributor Author

FYI @nordic-krch @gmarull @nordic-piks

nordic-segl added a commit to nordic-segl/sdk-zephyr that referenced this pull request Oct 1, 2024
Power states were recently added to the board definition of nrf54h20:
zephyrproject-rtos/zephyr@5b607ed
This has negative impact on drivers.uart.pm.int_driven test.

Delete property 'cpu-power-states' in the test overlay for nrf54h20.

Upstream PR: zephyrproject-rtos/zephyr#79268

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl
Copy link
Contributor Author

Fix was discussed in other PR
nrfconnect/sdk-nrf#17380 (comment)

Copy link
Member

@gmarull gmarull left a comment

Choose a reason for hiding this comment

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

Sounds like this is papering over a real bug, the PR doesn't really fix anything. I think test should be disabled/quarantined for 54h20 until fixed.

@nordic-krch
Copy link
Contributor

I'm trying to investigate the root cause.

@nordic-segl
Copy link
Contributor Author

Closing in favour of #79292

@nordic-segl nordic-segl closed this Oct 2, 2024
@nordic-segl nordic-segl deleted the NRFX-NONE_Fix-regression-in-uart_pm branch October 2, 2024 06:35
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.

5 participants