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

Align coresight_stm to the latest nrfutil trace #2423

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nordic-segl
Copy link
Contributor

@nordic-segl nordic-segl commented Jan 13, 2025

Latest version of nrfutil trace doesn't support setting uart baudrate.
Remove --baudrate {baudrate_val} from command that executes nrfutil.

Also, combine pytest script for:

  • samples.boards.nordic.coresight_stm,
  • tests.boards.nrf.coresight_stm,

to minimize code duplication (fix compliance check R0801).

Upstream PR #: 83939

manifest-pr-skip

@nordic-segl nordic-segl marked this pull request as draft January 13, 2025 13:34
@nordic-segl nordic-segl force-pushed the NRFX-6940_Fix-coresight_stm branch 5 times, most recently from f0299bf to 7d707ce Compare January 14, 2025 07:45
@nordic-segl nordic-segl changed the title DNM testing Align coresight_stm to the latest nrfutil trace Jan 14, 2025
@nordic-segl nordic-segl marked this pull request as ready for review January 14, 2025 07:46
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 14, 2025
Include
Align coresight_stm to the latest nrfutil trace
nrfconnect/sdk-zephyr#2423
zephyrproject-rtos/zephyr#83939

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6940_Fix-coresight_stm branch from 7d707ce to 354c7c1 Compare January 14, 2025 07:49
@nordic-segl nordic-segl force-pushed the NRFX-6940_Fix-coresight_stm branch 3 times, most recently from c173a9b to 7650d53 Compare January 14, 2025 13:03
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 14, 2025
Include
Align coresight_stm to the latest nrfutil trace
nrfconnect/sdk-zephyr#2423
zephyrproject-rtos/zephyr#83939

Signed-off-by: Sebastian Głąb <[email protected]>
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 14, 2025
Include
Align coresight_stm to the latest nrfutil trace
nrfconnect/sdk-zephyr#2423
zephyrproject-rtos/zephyr#83939

Signed-off-by: Sebastian Głąb <[email protected]>
…ines

Simplify string building and reduce line length.

Signed-off-by: Pieter De Gendt <[email protected]>

Signed-off-by: Sebastian Głąb <[email protected]>
(cherry picked from commit 9ca1b0f)
…ace command

Latest version of nrfutil trace doesn't support setting
uart baudrate.
Remove --baudrate {baudrate_val} from command that executes nrfutil.

Also, combine pytest script for:
- samples.boards.nordic.coresight_stm,
- tests.boards.nrf.coresight_stm,
to minimize code duplication (fix compliance check R0801).

Upstream PR #: 83939

Signed-off-by: Sebastian Głąb <[email protected]>
@nordic-segl nordic-segl force-pushed the NRFX-6940_Fix-coresight_stm branch from 7650d53 to 418655a Compare January 16, 2025 07:54
nordic-segl added a commit to nordic-segl/sdk-nrf that referenced this pull request Jan 16, 2025
Include
Align coresight_stm to the latest nrfutil trace
nrfconnect/sdk-zephyr#2423
zephyrproject-rtos/zephyr#83939

Signed-off-by: Sebastian Głąb <[email protected]>
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.

2 participants