Skip to content

Update sdk-nrf to 3.4.0-rc1 and remove Partition Manger#111

Open
ArekBalysNordic wants to merge 5 commits into
nrfconnect:mainfrom
ArekBalysNordic:update_and_dts
Open

Update sdk-nrf to 3.4.0-rc1 and remove Partition Manger#111
ArekBalysNordic wants to merge 5 commits into
nrfconnect:mainfrom
ArekBalysNordic:update_and_dts

Conversation

@ArekBalysNordic
Copy link
Copy Markdown
Contributor

@ArekBalysNordic ArekBalysNordic commented May 29, 2026

  • Switched the sdk-nrf revision to 3.4.0-rc1
  • Removed obsolete pm_static.yaml files
  • Added common partitioning dtsi files
  • Set dts root in module.yaml file.
  • Aligned Light Bulb and Light Switch to the recent changes.

Previously only Partition Manager was supported in the zb_nrf_nvram implementation. Now using partition from device tree is supported as well.

@ArekBalysNordic ArekBalysNordic requested a review from a team as a code owner May 29, 2026 13:30
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Documentation Preview

The documentation has been built successfully. You can view the preview here: preview

Generated at: 2026-06-01 14:03:59 UTC with commit 8607d12.

Expand to view changed pages
File Preview
docs/configuring.rst View
docs/configuring/configuring_libraries.rst View
samples/light_switch/README.rst View
samples/ncp/README.rst View

Note: The preview will be available after GitHub Pages deployment completes (usually 1-2 minutes). Ensure that generated at timestamp is up to date. If the Pull Request is stale for more than 30 days, the preview may be deleted. In this case, please update the Pull Request, or re-run the Build Documentation workflow to generate a new preview.

Comment thread samples/light_bulb/sysbuild_matter_fota.conf
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe not needed now, but we might think about some better structuring/naming for these files, in case we want to support both options when comes to using external flash or not, for instance.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's right. In sdk-nrf we have something like _internal.dtsi for non-external flash partitions. However, here I don't have any strong opinion what's the best naming.

Switched the sdk-nrf revision to 3.4.0-rc1

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
Comment thread subsys/Kconfig
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Look at the last lines of this file. They should be guarded by if PARTITION_MANAGER_ENABLED.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's right. I will add it there. However, it's not visible while building even if it is not guarded.

Previously only Partition Manager was supported in the
zb_nrf_nvram implementation. Now using partition from device tree
is supported as well.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
the dfu_multi_target library now supports both PM and DTS
partitioning.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
- Removed obsolete pm_static.yaml files
- Added common partitioning dtsi files
- Set dts root in module.yaml file.
- Aligned Light Bulb and Light Switch to the recent changes.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
Updated README files and configuration sections.
Described the transition process, and pointed sdk-nrf
documentation.

Signed-off-by: Arkadiusz Balys <arkadiusz.balys@nordicsemi.no>
@ArekBalysNordic
Copy link
Copy Markdown
Contributor Author

FYI, Do not merge, I'm still checking partitioning.

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