Skip to content

Custom channel sequence in tx with sleep#28036

Merged
rlubos merged 2 commits into
nrfconnect:mainfrom
thomas-m-johansen:custom_channel_sequence_in_tx_with_sleep
Apr 13, 2026
Merged

Custom channel sequence in tx with sleep#28036
rlubos merged 2 commits into
nrfconnect:mainfrom
thomas-m-johansen:custom_channel_sequence_in_tx_with_sleep

Conversation

@thomas-m-johansen
Copy link
Copy Markdown
Contributor

No description provided.

@thomas-m-johansen thomas-m-johansen requested review from a team as code owners April 10, 2026 11:32
@NordicBuilder NordicBuilder added the doc-required PR must not be merged without tech writer approval. label Apr 10, 2026
@thomas-m-johansen thomas-m-johansen added this to the 3.3.1 milestone Apr 10, 2026
@Mirlenko Mirlenko self-requested a review April 10, 2026 11:38
@NordicBuilder
Copy link
Copy Markdown
Contributor

NordicBuilder commented Apr 10, 2026

CI Information

To view the history of this post, click the 'edited' button above
Build number: 6

Inputs:

Sources:

sdk-nrf: PR head: 5a539c0b984fef36b9b3a836e16531016071f49f

more details

sdk-nrf:

PR head: 5a539c0b984fef36b9b3a836e16531016071f49f
merge base: e267d1a1dfed944907d172fcf8d2e4a045a50c7f
target head (main): bb74c2c0e756c16d2d6f1606241769b927267a81
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (5)
doc
│  ├── nrf
│  │  ├── releases_and_maturity
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
samples
│  ├── peripheral
│  │  ├── radio_test
│  │  │  ├── README.rst
│  │  │  ├── src
│  │  │  │  ├── radio_cmd.c
│  │  │  │  ├── radio_test.c
│  │  │  │  │ radio_test.h

Outputs:

Toolchain

Version: 911f4c5c26
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:911f4c5c26_5ea73affbf

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 18
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-ble_samples
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@thomas-m-johansen thomas-m-johansen force-pushed the custom_channel_sequence_in_tx_with_sleep branch from 03af616 to f9e1701 Compare April 10, 2026 11:45
Comment thread doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst Outdated
Comment thread samples/peripheral/radio_test/src/radio_cmd.c Outdated
Comment thread samples/peripheral/radio_test/src/radio_cmd.c
Comment thread samples/peripheral/radio_test/src/radio_test.h Outdated
@github-actions
Copy link
Copy Markdown

@thomas-m-johansen thomas-m-johansen force-pushed the custom_channel_sequence_in_tx_with_sleep branch from f9e1701 to 347bd9f Compare April 10, 2026 13:04
@thomas-m-johansen thomas-m-johansen force-pushed the custom_channel_sequence_in_tx_with_sleep branch from 347bd9f to 9894173 Compare April 10, 2026 13:12
Comment on lines +344 to +349
* Added the new ``start_tx_sweep_with_sleep`` shell command.
It allows for running a TX sweep with silent periods between each frequency.
* Added the new ``set_channel_sequence`` shell command.
It allows for setting a custom channel sequence for the ``start_tx_sweep_with_sleep`` command.
* Added the new ``print_channel_sequence`` shell command.
It prints the channel sequence set with the ``set_channel_sequence`` command.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Added the new ``start_tx_sweep_with_sleep`` shell command.
It allows for running a TX sweep with silent periods between each frequency.
* Added the new ``set_channel_sequence`` shell command.
It allows for setting a custom channel sequence for the ``start_tx_sweep_with_sleep`` command.
* Added the new ``print_channel_sequence`` shell command.
It prints the channel sequence set with the ``set_channel_sequence`` command.
* Added:
* ``start_tx_sweep_with_sleep`` shell command.
It allows for running a TX sweep with silent periods between each frequency.
* ``set_channel_sequence`` shell command.
It allows for setting a custom channel sequence for the ``start_tx_sweep_with_sleep`` command.
* ``print_channel_sequence`` shell command.
It prints the channel sequence set with the ``set_channel_sequence`` command.

Comment thread samples/peripheral/radio_test/src/radio_cmd.c Outdated
Comment thread samples/peripheral/radio_test/src/radio_cmd.c Outdated
Comment thread samples/peripheral/radio_test/src/radio_test.h Outdated
Comment thread samples/peripheral/radio_test/README.rst
Comment thread samples/peripheral/radio_test/README.rst Outdated
Comment thread doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst Outdated
we see value in being able to run a custom sweep in the tx_with_sleep
mode where not all channels are used, or even where the sequence isn't a
strictly rising sweep.

added a new command that allows users to configure if they want a custom
sweep sequence with a custom amount of channels in the sweep.

also added a new command to print out this channel sequence so that
users can verify that it is configured correctly and check initial array
configuration.

Signed-off-by: Thomas Johansen <thomas.johansen@nordicsemi.no>
@thomas-m-johansen thomas-m-johansen force-pushed the custom_channel_sequence_in_tx_with_sleep branch from 9894173 to 321513f Compare April 10, 2026 13:34
Copy link
Copy Markdown
Contributor

@richabp richabp left a comment

Choose a reason for hiding this comment

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

Approving with a nit.

Comment thread doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst Outdated
added documentation for the new set_channel_sequence and
set_channel_sequence commands in radio_test as well as a changelog entry
for the new commands.

Signed-off-by: Thomas Johansen <thomas.johansen@nordicsemi.no>
@thomas-m-johansen thomas-m-johansen force-pushed the custom_channel_sequence_in_tx_with_sleep branch from 321513f to 5a539c0 Compare April 13, 2026 06:06
@rlubos rlubos merged commit 9e47dbc into nrfconnect:main Apr 13, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog doc-required PR must not be merged without tech writer approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants