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: boards: nrf: hwinfo: Add test for reset cause on nrf54* #84029

Conversation

nordic-segl
Copy link
Contributor

Add test that checks reset cause reported by HWINFO.

Check that hwinfo_get_supported_reset_cause() returns expected value.

Check that hwinfo_get_reset_cause() correctly detects:

  • RESET_PIN,
  • RESET_SOFTWARE,
  • RESET_WATCHDOG.

Check that reset cause can be cleared
with hwinfo_clear_reset_cause().

@nordic-krch
Copy link
Contributor

new nrfx will be needed to make the nrf54h test pass.

@nordic-segl
Copy link
Contributor Author

nordic-segl commented Jan 16, 2025

This test is passing on nRF54L15 and nRF54L20. On nRF54H20, it will be added to a quarantine.

@nordic-segl nordic-segl force-pushed the NRFX-6895_Add-test-for-hwinfo-reset-cause-on-nrf54 branch from 9f82546 to 6e7042d Compare January 16, 2025 08:35
kl-cruz
kl-cruz previously approved these changes Jan 17, 2025
nordic-krch
nordic-krch previously approved these changes Jan 17, 2025
Add test that checks reset cause reported by HWINFO.

Check that hwinfo_get_supported_reset_cause() returns
expected value.

Check that hwinfo_get_reset_cause() correctly detects:
- RESET_PIN,
- RESET_SOFTWARE,
- RESET_WATCHDOG.

Check that reset cause can be cleared
with hwinfo_clear_reset_cause().

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl dismissed stale reviews from nordic-krch and kl-cruz via 2d51f2a January 17, 2025 11:30
@nordic-segl nordic-segl force-pushed the NRFX-6895_Add-test-for-hwinfo-reset-cause-on-nrf54 branch from 6e7042d to 2d51f2a Compare January 17, 2025 11:30
@nordic-segl
Copy link
Contributor Author

I got request to call sys_cache_data_flush_range() before reset rather than disable DCACHE on nrf54h20dk.

@kartben kartben merged commit 8f83267 into zephyrproject-rtos:main Jan 17, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants