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.

Sign up for GitHub

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

Jump to bottom

doc: wifi: Corrections to Regulatory Testing doc #21125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

doc: wifi: Corrections to Regulatory Testing doc #21125

wants to merge 1 commit into from

Conversation

prsi98
Copy link
Contributor

@prsi98 prsi98 commented Mar 25, 2025 •
edited by jira bot
Loading

SHEL-3047: Update Regulatory testing with latest changes and remove errors.

@prsi98 prsi98 added the DNM label Mar 25, 2025
@prsi98 prsi98 requested a review from a team as a code owner March 25, 2025 07:39
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Mar 25, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 25, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 3931fc2bf5389555f137a2a3da3c68dd64e01087

more details

sdk-nrf:

PR head: 3931fc2bf5389555f137a2a3da3c68dd64e01087
merge base: 8b53800e77f37e4d0c985024d1f020a3d7a5f7eb
target head (main): 8b53800e77f37e4d0c985024d1f020a3d7a5f7eb
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 (9) 
doc
│  ├── nrf
│  │  ├── app_dev
│  │  │  ├── device_guides
│  │  │  │  ├── nrf70
│  │  │  │  │  │ power_profiling.rst
│  │  ├── protocols
│  │  │  ├── wifi
│  │  │  │  ├── regulatory_certification
│  │  │  │  │  ├── adaptivity_test_procedure.rst
│  │  │  │  │  ├── antenna_gain_compensation.rst
│  │  │  │  │  ├── band_edge_compensation.rst
│  │  │  │  │  ├── test_setup.rst
│  │  │  │  │  ├── using_wifi_shell_sample.rst
│  │  │  │  │  ├── wifi_radio_test_sample
│  │  │  │  │  │  ├── wifi_build_instructions.rst
│  │  │  │  │  │  │ wifi_radio_test_for_per_measurements.rst
samples
│  ├── wifi
│  │  ├── throughput
│  │  │  │ README.rst

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@prsi98 prsi98 force-pushed the certification_doc_corrections branch from 0e3efb2 to df3a062 Compare March 25, 2025 08:27
richabp
richabp requested changes Mar 25, 2025
View reviewed changes
doc/nrf/protocols/wifi/regulatory_certification/test_setup.rst Outdated

For hardware information, see the user guides `nRF7002 DK Hardware`_ and `nRF7002 EK Hardware`_.

For hardware information, see the user guides `nRF7002 DK Hardware`_ , `nRF7002 EK Hardware`_ and `nRF7002 EB Hardware`_.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
For hardware information, see the user guides `nRF7002 DK Hardware`_ , `nRF7002 EK Hardware`_ and `nRF7002 EB Hardware`_.
For hardware information, see the user guides `nRF7002 DK Hardware`_ , `nRF7002 EK Hardware`_ , and `nRF7002 EB Hardware`_.

...f/protocols/wifi/regulatory_certification/wifi_radio_test_sample/wifi_build_instructions.rst Outdated

.. note::

Combined build for Radio test and Wi-Fi Radio test is not supported with nrf7002ek due to ...
Copy link
Contributor

Choose a reason for hiding this comment

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

The sentence is incomplete.

Suggested change
Combined build for Radio test and Wi-Fi Radio test is not supported with nrf7002ek due to ...
Combined build for Radio test (short-range) and Wi-Fi Radio test is not supported with nRF7002 EK due to ...

richabp
richabp requested changes Mar 26, 2025
View reviewed changes
doc/nrf/protocols/wifi/regulatory_certification/test_setup.rst
Copy link
Contributor

Choose a reason for hiding this comment

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

This doc mentions about nrfjprog which needs to be replaced by nrfutil commands.
@krga2022

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@bama-nordic

@prsi98 prsi98 force-pushed the certification_doc_corrections branch from df3a062 to 731a857 Compare March 26, 2025 09:44
prsi98
prsi98 commented Mar 26, 2025
View reviewed changes
...f/protocols/wifi/regulatory_certification/wifi_radio_test_sample/wifi_build_instructions.rst Outdated

.. note::

Combined build for Radio test (short-range) and Wi-Fi Radio test is not supported with nRF7002 EK due to ...
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @bama-nordic need suggestion here.

@prsi98 prsi98 force-pushed the certification_doc_corrections branch from 731a857 to 3297159 Compare March 26, 2025 13:08
@prsi98 prsi98 requested a review from a team as a code owner March 26, 2025 13:08
@prsi98 prsi98 force-pushed the certification_doc_corrections branch from 3297159 to f33d4ab Compare March 26, 2025 13:13
@github-actions GitHub Actions
Copy link

You can find the documentation preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/app_dev/device_guides/nrf70/power_profiling.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/adaptivity_test_procedure.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/antenna_gain_compensation.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/band_edge_compensation.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/test_setup.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/using_wifi_shell_sample.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/wifi_radio_test_sample/wifi_build_instructions.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/protocols/wifi/regulatory_certification/wifi_radio_test_sample/wifi_radio_test_for_per_measurements.html
https://ncsdoc.z6.web.core.windows.net/PR-21125/nrf/samples/wifi/throughput/README.html

@richabp richabp mentioned this pull request Mar 27, 2025
Merged
@prsi98
doc: wifi: Corrections to Regulatory Testing doc
3931fc2 
[SHEL-3047]: Update Regulatory testing with latest changes and remove
errors.

Signed-off-by: Prasun Sinha <Prasun.Sinha@nordicsemi.no>
@prsi98 prsi98 force-pushed the certification_doc_corrections branch from f33d4ab to 3931fc2 Compare March 27, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richabp richabp richabp requested changes

@bama-nordic bama-nordic Awaiting requested review from bama-nordic

@srkanordic srkanordic Awaiting requested review from srkanordic

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM doc-required PR must not be merged without tech writer approval.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@prsi98 @NordicBuilder @richabp