tests: subsys: Add test that verifies west debug command #26119
Conversation
NordicBuilder
added
the
changelog-entry-required
CI InformationTo view the history of this post, click the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
| - nrf54l15dk/nrf54l15/cpuapp/ns | ||
| - nrf54l15dk/nrf54l15/cpuflpr | ||
| - nrf54l15dk/nrf54l15/cpuflpr/xip | ||
| - nrf54lm20dk/nrf54lm20a/cpuapp |
There was a problem hiding this comment.
Not sure if this is related to zephyrproject-rtos/zephyr#100386 or not but FYI a user reported issues with doing west flash with the JLink runner on flpr, I assume because in segger's eyes the flpr has no NVM and for that it would need to be programmed through the app core then debugged through the flpr core?
There was a problem hiding this comment.
I think this PR is a result of west debug and west attach not working on some platforms.
Here is fix for one of such failures.
I wasn't aware that it is possible to switch west flash from nrfutil device to JLink.
There was a problem hiding this comment.
west flash -r jlink, I didn't try with the M33 but with the RISCV on nrf54l15dk it flat out fails (though debugging without loading works fine on RISCV)
There was a problem hiding this comment.
-r jlink is a good candidate for another test
There was a problem hiding this comment.
nordic-segl
force-pushed
the
NRFX-8719_Add-test-for-west-debug
branch
from
b821e25 to
59558c7
Compare
nordic-segl
requested review from
a team,
PerMac,
katgiadla and
nordic-piks
as code owners
NordicBuilder
removed
the
changelog-entry-required
nordic-segl
force-pushed
the
NRFX-8719_Add-test-for-west-debug
branch
10 times, most recently
from
1d3bfe9 to
3f4aebf
Compare
| - nrf5340dk/nrf5340/cpuapp/ns | ||
| - nrf5340dk/nrf5340/cpunet | ||
| - nrf54h20dk/nrf54h20/cpuapp | ||
| - nrf54h20dk/nrf54h20/cpuflpr |
nordic-segl
force-pushed
the
NRFX-8719_Add-test-for-west-debug
branch
6 times, most recently
from
4627d47 to
cdc7c9e
Compare
nordic-segl
force-pushed
the
NRFX-8719_Add-test-for-west-debug
branch
2 times, most recently
from
cd0928a to
edf7c26
Compare
Include boards: nordic: nrf54h20dk: Add workaround for RISC-V debugging nrfconnect/sdk-zephyr#3621 Upstream PR #: 100744 Signed-off-by: Sebastian Głąb <[email protected]>
Add test that checks if platform can be debugged with - west attach - west debug. Add test variant with debug optimizations and thread info enabled. Signed-off-by: Sebastian Głąb <[email protected]>
nordic-segl
force-pushed
the
NRFX-8719_Add-test-for-west-debug
branch
from
edf7c26 to
b97e5ad
Compare
|
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
|
You can find the documentation preview for this PR here. |
4 participants
Add test that checks if platform can be debugged with
Add test variant with debug optimizations and thread info enabled.