Skip to content

Conversation

ArekBalysNordic
Copy link
Contributor

No description provided.

@ArekBalysNordic ArekBalysNordic requested a review from a team September 30, 2025 12:37
@ArekBalysNordic ArekBalysNordic requested review from a team as code owners September 30, 2025 12:37
@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. manifest labels Sep 30, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 30, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
matter nrfconnect/sdk-connectedhomeip@4031bf3 (master) nrfconnect/sdk-connectedhomeip#644 nrfconnect/sdk-connectedhomeip#644/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 30, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 2c78992339dbb690ca4a0fe3f94d17bb1fd8d7d8
matter: PR head: e64e2eb8b5363090565495821550890b71ad84c9

more details

sdk-nrf:

PR head: 2c78992339dbb690ca4a0fe3f94d17bb1fd8d7d8
merge base: 0c840a9db6f90e4c590460d4814bd3cfb434cff4
target head (main): d5ec504d111728d08f8c6ab67378a4d645cb0ead
Diff

matter:

PR head: e64e2eb8b5363090565495821550890b71ad84c9
merge base: 4031bf321f9eea476552639dd39d23c90b00d62a
target head (master): 4031bf321f9eea476552639dd39d23c90b00d62a
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 (17)
applications
│  ├── matter_bridge
│  │  │ Kconfig
│  ├── matter_weather_station
│  │  │ Kconfig
doc
│  ├── nrf
│  │  ├── includes
│  │  │  ├── advanced_conf_matter.txt
│  │  │  ├── matter_building_nrf5340dk_70ek
│  │  │  │ matter_custom_board.txt
│  │  ├── releases_and_maturity
│  │  │  ├── migration
│  │  │  │  │ migration_guide_3.2.rst
│  │  │  ├── releases
│  │  │  │  │ release-notes-changelog.rst
modules
│  ├── lib
│  │  ├── matter
│  │  │  ├── config
│  │  │  │  ├── nrfconnect
│  │  │  │  │  ├── chip-module
│  │  │  │  │  │  ├── Kconfig
│  │  │  │  │  │  ├── Kconfig.defaults
│  │  │  │  │  │  │ Kconfig.features
samples
│  ├── matter
│  │  ├── light_bulb
│  │  │  │ Kconfig
│  │  ├── light_switch
│  │  │  │ Kconfig
│  │  ├── lock
│  │  │  │ Kconfig
│  │  ├── manufacturer_specific
│  │  │  │ Kconfig
│  │  ├── template
│  │  │  │ Kconfig
│  │  ├── thermostat
│  │  │  │ Kconfig
west.yml

Outputs:

Toolchain

Version: f66cf421f3
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:f66cf421f3_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 104
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
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-ble_samples
    • 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_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-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-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

Copy link

@ArekBalysNordic ArekBalysNordic force-pushed the remove_dk_dependency branch 2 times, most recently from 15e6d3b to 94f1314 Compare October 1, 2025 07:24
+++++++++++

|no_changes_yet_note|
* Removed dependencies to Nordic DK-specific configurations in Matter configurations.
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
* Removed dependencies to Nordic DK-specific configurations in Matter configurations.
* Removed dependencies on Nordic DK-specific configurations in Matter configurations.

@ArekBalysNordic ArekBalysNordic requested a review from a team as a code owner October 3, 2025 13:56
@ArekBalysNordic ArekBalysNordic force-pushed the remove_dk_dependency branch 4 times, most recently from 81ee51b to 6aa386e Compare October 3, 2025 14:06
CHIP_WIFI cannot depend on DK, it must rely on WIFI_NRF70.

Signed-off-by: Arkadiusz Balys <[email protected]>
@ArekBalysNordic ArekBalysNordic added the CI-limit-jobs Limit number of twister jobs run on each CI node label Oct 6, 2025
.. code-block:: console

cmake -GNinja -Bbuild -DBOARD=nrf5340dk/nrf5340/cpuapp -DSHIELD=nrf7002ek -DFILE_SUFFIX=nrf70ek -DSB_CONFIG_WIFI_NRF70=y -DAPP_DIR=*app_path* *path_to_zephyr*/share/sysbuild
cmake -GNinja -Bbuild -DBOARD=nrf5340dk/nrf5340/cpuapp -DSHIELD=nrf7002ek -DFILE_SUFFIX=nrf70ek -DSB_CONFIG_WIFI_NRF70=y-DAPP_DIR=*app_path* *path_to_zephyr*/share/sysbuild
Copy link
Contributor

Choose a reason for hiding this comment

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

this adds a mistake

#. Write Kconfig files to enable all required Kconfig options for your board.
#. If you want to build your custom board with nRF70 Wi-Fi support, set the :kconfig:option:`CONFIG_CHIP_WIFI` and :kconfig:option:`SB_CONFIG_WIFI_NRF70` Kconfig options to ``y``.
#. If your device uses external flash, add its devicetree definition under the :file:`board/<board_name>_<soc_name>.overlay` file, and set ``nordic,pm-ext-flash`` in the devicetree's ``chosen`` configuration.
#. Refer to the :ref:`ug_matter_device_advanced_kconfigs` user guide, create your list of advanced configurations for your board, and apply the selected Kconfig options in the :file:`board/<board_name>_<soc_name>.conf` file.
Copy link
Contributor

Choose a reason for hiding this comment

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

variant? Link to the zephyr docs where this is explained rather than regurgitate it wrongly

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I couldn't find an appropriate guide in Zephyr, but I've changed it.

A new guide is added to each sample and contains
instruction on how to create a custom board and prepare it
to work with Matter.

Signed-off-by: Arkadiusz Balys <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-limit-jobs Limit number of twister jobs run on each CI node DNM doc-required PR must not be merged without tech writer approval. manifest manifest-matter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants