Introduce nRF7120 SoC and board definitions to sdk-nrf #20070

rob-robinson-14 · 2025-01-24T12:44:05Z

PR containing out of tree definitions for nrf7120.

CLAassistant · 2025-01-24T12:44:10Z

All committers have signed the CLA.

NordicBuilder · 2025-01-24T12:48:30Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 20

Inputs:

Sources:

sdk-nrf: PR head: 91b3e865fa40b85dbf0af6943e9772b5bb4e8e3b

more details

sdk-nrf:

PR head: 91b3e865fa40b85dbf0af6943e9772b5bb4e8e3b
merge base: 085e5a678ca84a3b5c8cd2e0c3b41de872f4b9cf
target head (main): e61c2baaac25b5ac2af4ade25209d775ca2a1138
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 (38)

CODEOWNERS
boards
│  ├── nordic
│  │  ├── nrf7120pdk
│  │  │  ├── Kconfig.defconfig
│  │  │  ├── Kconfig.nrf7120pdk
│  │  │  ├── board.cmake
│  │  │  ├── board.yml
│  │  │  ├── nrf7120_cpuapp_common.dtsi
│  │  │  ├── nrf7120pdk_nrf7120-common.dtsi
│  │  │  ├── nrf7120pdk_nrf7120-pinctrl.dtsi
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp.dts
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp.yaml
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_defconfig
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_ns.dts
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_ns.yaml
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_ns_defconfig
│  │  │  ├── nrf7120pdk_nrf7120_cpuflpr.dts
│  │  │  ├── nrf7120pdk_nrf7120_cpuflpr.yaml
│  │  │  ├── nrf7120pdk_nrf7120_cpuflpr_defconfig
│  │  │  ├── nrf7120pdk_nrf7120_cpuflpr_xip.dts
│  │  │  ├── nrf7120pdk_nrf7120_cpuflpr_xip.yaml
│  │  │  │ nrf7120pdk_nrf7120_cpuflpr_xip_defconfig
dts
│  ├── arm
│  │  ├── nordic
│  │  │  ├── nrf7120_enga.dtsi
│  │  │  │ nrf7120_enga_cpuapp.dtsi
│  ├── riscv
│  │  ├── nordic
│  │  │  │ nrf7120_enga_cpuflpr.dtsi
soc
│  ├── nordic
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  ├── Kconfig.soc
│  │  ├── nrf71
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── Kconfig.defconfig
│  │  │  ├── Kconfig.defconfig.nrf7120_enga_cpuapp
│  │  │  ├── Kconfig.defconfig.nrf7120_enga_cpuflpr
│  │  │  ├── Kconfig.soc
│  │  │  ├── align.ld
│  │  │  ├── kmu_push_area_section.ld
│  │  │  ├── soc.c
│  │  │  │ soc.h
│  │  │ soc.yml
zephyr
│  │ module.yml

Outputs:

Toolchain

Version: 342151af73
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:342151af73_912848a074

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

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister
- sdk-nrf test count: 251
✅ 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_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-ps
- test-fw-nrfconnect-rpc
- test-fw-nrfconnect-rs
- test-fw-nrfconnect-tfm
- test-fw-nrfconnect-thread
- test-fw-nrfconnect-zigbee
- test-low-level
- test-sdk-audio
- test-sdk-dfu
- test-sdk-find-my
- test-sdk-mcuboot
- test-sdk-pmic-samples
- test-sdk-sidewalk
- test-sdk-wifi
- test-secdom-samples-public

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

NordicBuilder · 2025-01-24T13:01:24Z

You can find the documentation preview for this PR at this link.

Note: This comment is automatically posted by the Documentation Publish GitHub Action.

soc/nordic/Kconfig

soc/nordic/Kconfig.defconfig

soc/nordic/Kconfig.soc

soc/nordic/nrf71/CMakeLists.txt

soc/nordic/nrf71/Kconfig.soc

soc/nordic/nrf71/soc.c

soc/nordic/soc.yml

boards/nordic/nrf7120pdk/Kconfig.defconfig

boards/nordic/nrf7120pdk/Kconfig.nrf7120pdk

nordicjm · 2025-01-24T15:44:44Z

also shouldn't be e.g. noup in sdk-nrf commit titles, that is for sdk-* OSS repos only

boards/nordic/nrf7120pdk/nrf7120pdk_nrf7120_enga_cpuapp.dts

dts/arm/nordic/nrf7120_enga.dtsi

dts/riscv/nordic/nrf7120_enga_cpuflpr.dtsi

boards/nordic/nrf7120pdk/nrf7120pdk_nrf7120_enga-common.dtsi

nordicjm · 2025-01-27T11:05:39Z

boards/nordic/nrf7120pdk/Kconfig

+config NRF_MPC_REGION_SIZE
+	hex
+	default 0x1000
+	help
+	  Region size for the Memory Protection Controller (MPC) in bytes.


this should be in dts

boards/nordic/nrf7120pdk/Kconfig

boards/nordic/nrf7120pdk/nrf7120pdk_nrf7120_enga_cpuapp_ns.dts

krish2718 · 2025-01-29T20:06:08Z

dts/arm/nordic/nrf7120_enga.dtsi

+			ranges = <0x0 0x20080000 0x40000>;
+		};
+
+		ram02_sram: memory@200c0000 {


wifi_sram as this is pretty much dedicated to Wi-Fi buffers?

I'd favor keeping this naming more generic and skip referring to any protocol here (esp. if this part of the RAM can indeed serve multiple purposes).

or may be ?

wifi_netbuf_sram: ram02_sram

soc/nordic/Kconfig

krish2718 · 2025-01-29T20:10:07Z

boards/nordic/nrf7120pdk/board.yml

+    - name: xip
+      cpucluster: cpuflpr


XIP and FLPR?

boards/nordic/nrf7120pdk/Kconfig

krish2718 · 2025-01-30T07:24:19Z

boards/nordic/nrf7120pdk/nrf7120pdk_nrf7120_enga_cpuapp_ns.yaml

+  - gnuarmemb
+  - xtools
+  - zephyr
+ram: 1024


is the RAM doubled compared for default PDK?

dts/arm/nordic/nrf7120_enga.dtsi

rob-robinson-14 · 2025-01-30T16:58:38Z

@nrfconnect/ncs-code-owners
Unsure who the code owners should be - please advise if these needs revising.

Add dts files for nRF7120 Signed-off-by: David Jewsbury <[email protected]> Co-authored-by: Robert Robinson <[email protected]>

Add nRF7120 to soc directory Signed-off-by: Robert Robinson <[email protected]> Co-authored-by: David Jewsbury <[email protected]>

Add board files for nRF7120pdk Signed-off-by: David Jewsbury <[email protected]> Co-authored-by: Robert Robinson <[email protected]>

This adds the nrf7120pdk/cpuapp/ns board variant to sdk-zephyr. It allows to build nrf7120pdk with TF-M in nRF Connect SDK. Once upstreamed to zephyr, commit will become a noup within sdk-zephyr. This is done as a noup as the out-of-tree board extension feature is not available. When zephyrproject-rtos/zephyr#69548 is completed this commit can be reverted and the variant be placed in sdk-nrf. Signed-off-by: Robert Robinson <[email protected]>

Once nrf7120 is upstreamed this commit will be noup in sdk-zephyr. The SoftDevice Controller is a different controller than the open source link layer with a different set of quirks. It should therefore have its own device tree binding. This commit converts the SoftDevice Controller driver to use this new DTS binding instead of reusing the existing one. This commit updates or adds additional overlays for existing samples, applications and tests that were using the open source link layer. Signed-off-by: Robert Robinson <[email protected]>

nRF7120 has been added to soc folder in NCS Therefore, soc_root needs to be added to module.yml Signed-off-by: David Jewsbury <[email protected]>

The license agreement for files within sdk-nrf differs from what will eventually be required from upstream zephyr. All SPDX-License-Identifier: Apache-2.0 have been updated to SPDX-License-Identifier: LicenseRef-Nordic-5-Clause Signed-off-by: Robert Robinson <[email protected]>

Assigned codeowners for new nrf7120 board folder and soc/nordic within nrf. Signed-off-by: Robert Robinson <[email protected]>

rob-robinson-14 requested review from a team as code owners January 24, 2025 12:44

github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jan 24, 2025

nordicjm reviewed Jan 24, 2025

View reviewed changes

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch 2 times, most recently from b6ae797 to 338f9ac Compare January 24, 2025 16:03

ioannisg reviewed Jan 27, 2025

View reviewed changes

boards/nordic/nrf7120pdk/nrf7120pdk_nrf7120_enga_cpuapp.dts Outdated Show resolved Hide resolved

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from 338f9ac to 58c71c8 Compare January 27, 2025 10:27

nordicjm requested changes Jan 27, 2025

View reviewed changes

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from 58c71c8 to d1e423f Compare January 29, 2025 13:11

rob-robinson-14 requested review from anangl, gmarull and bjarki-andreasen January 29, 2025 14:28

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch 2 times, most recently from 501616d to f73fc0b Compare January 29, 2025 17:05

krish2718 self-requested a review January 29, 2025 20:05

krish2718 reviewed Jan 30, 2025

View reviewed changes

carlescufi requested review from kl-cruz, nika-nordic and masz-nordic January 30, 2025 09:59

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from f73fc0b to 40b3dc2 Compare January 30, 2025 10:57

github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jan 30, 2025

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from 2de9ef1 to 8f5f539 Compare January 30, 2025 17:34

Hi-Im-David force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from 8f5f539 to 0c54122 Compare January 31, 2025 11:43

Hi-Im-David force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch 3 times, most recently from f8059d9 to 265b144 Compare January 31, 2025 15:49

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch 4 times, most recently from 348ada2 to cac233a Compare January 31, 2025 18:21

Hi-Im-David and others added 8 commits January 31, 2025 18:23

dts: arm: nordic: add support for Nordic nrf7120

eeabe8a

Add dts files for nRF7120 Signed-off-by: David Jewsbury <[email protected]> Co-authored-by: Robert Robinson <[email protected]>

soc: nordic: Add support for nRF7120 SoC

587712c

Add nRF7120 to soc directory Signed-off-by: Robert Robinson <[email protected]> Co-authored-by: David Jewsbury <[email protected]>

boards: nordic: Add nRF7120 PDK board

78000c6

Add board files for nRF7120pdk Signed-off-by: David Jewsbury <[email protected]> Co-authored-by: Robert Robinson <[email protected]>

modules: Add support for soc directory in NCS

313c8f1

nRF7120 has been added to soc folder in NCS Therefore, soc_root needs to be added to module.yml Signed-off-by: David Jewsbury <[email protected]>

CODEOWNERS: Assign code owners to new board and soc files

91b3e86

Assigned codeowners for new nrf7120 board folder and soc/nordic within nrf. Signed-off-by: Robert Robinson <[email protected]>

rob-robinson-14 force-pushed the NRFX-6596-add-nrf7120-soc-board-definitions branch from cac233a to 91b3e86 Compare January 31, 2025 18:25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Introduce nRF7120 SoC and board definitions to sdk-nrf #20070

Introduce nRF7120 SoC and board definitions to sdk-nrf #20070

rob-robinson-14 commented Jan 24, 2025

CLAassistant commented Jan 24, 2025 •

edited

Loading

NordicBuilder commented Jan 24, 2025 •

edited

Loading

NordicBuilder commented Jan 24, 2025

nordicjm commented Jan 24, 2025

nordicjm Jan 27, 2025

krish2718 Jan 29, 2025

ioannisg Jan 30, 2025

krish2718 Jan 30, 2025

krish2718 Jan 29, 2025

krish2718 Jan 30, 2025

rob-robinson-14 commented Jan 30, 2025

Introduce nRF7120 SoC and board definitions to sdk-nrf #20070

Are you sure you want to change the base?

Introduce nRF7120 SoC and board definitions to sdk-nrf #20070

Conversation

rob-robinson-14 commented Jan 24, 2025

CLAassistant commented Jan 24, 2025 • edited Loading

NordicBuilder commented Jan 24, 2025 • edited Loading

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

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

NordicBuilder commented Jan 24, 2025

nordicjm commented Jan 24, 2025

nordicjm Jan 27, 2025

Choose a reason for hiding this comment

krish2718 Jan 29, 2025

Choose a reason for hiding this comment

ioannisg Jan 30, 2025

Choose a reason for hiding this comment

krish2718 Jan 30, 2025

Choose a reason for hiding this comment

krish2718 Jan 29, 2025

Choose a reason for hiding this comment

krish2718 Jan 30, 2025

Choose a reason for hiding this comment

rob-robinson-14 commented Jan 30, 2025

CLAassistant commented Jan 24, 2025 •

edited

Loading

NordicBuilder commented Jan 24, 2025 •

edited

Loading