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

[nrf fromlist] tests: drivers: watchdog: disable DCACHE for nRF54H20dk #2279

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

mstasiaknordic
Copy link
Contributor

Disabled data caching for nRF54H20dk in wdt_basic_api test. It caused variables stored in noinit section to be cached, resulting in reset loop.

Upstream PR #: 81438

@nordic-segl
Copy link
Contributor

nordic-segl commented Nov 19, 2024

That doesn't work as expected.
You can check locally:

segl@segl01:~/workspace/ncs$ python3 ./zephyr/scripts/twister -T zephyr/tests/drivers/watchdog/wdt_basic_api/ --platform nrf54h20dk/nrf54h20/cpuapp --ninja --device-testing --verbose --verbose --jobs 1 --device-serial /dev/serial/by-id/usb-SEGGER_J-Link_001051169581-if00 --west-flash="--erase"

Two test variants are executed:
DEBUG - adding nrf54h20dk/nrf54h20/cpuapp/tests/drivers/watchdog/wdt_basic_api/drivers.watchdog
DEBUG - adding nrf54h20dk/nrf54h20/cpuapp/tests/drivers/watchdog/wdt_basic_api/drivers.watchdog.nrf54h20dk

One of them is passing, second one (with DCACHE enabled) is failing.

What about instead defining a separate test suite drivers.watchdog.nrf54h20dk
just add file nrf54h20dk_nrf54h20_cpuapp.conf in ./boards/ directory.
Put CONFIG_DCACHE=n in that file
???

Copy link
Contributor

@nordic-segl nordic-segl left a comment

Choose a reason for hiding this comment

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

Don't extend sample.yaml as test drivers.watchdog is keep failing for nrf54h20.
Add ./boards/nrf54h20dk_nrf54h20_cpuapp.conf with
CONFIG_DCAHE=n
in it.

Disabled data caching for nRF54H20dk in wdt_basic_api test.
It caused variables stored in noinit section to be cached,
resulting in reset loop.

Upstream PR #: 81438

Signed-off-by: Michał Stasiak <[email protected]>
@rlubos rlubos merged commit 3d0bbe4 into nrfconnect:main Nov 20, 2024
15 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.

5 participants