Skip to content

Conversation

@rghaddab
Copy link
Contributor

MULTITHREADING is needed by the mram driver for the nrf54h20_cpuapp target

@rghaddab rghaddab marked this pull request as draft December 10, 2025 14:12
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 10, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#582

Signed-off-by: Nordic Builder <[email protected]>
rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 10, 2025
rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 10, 2025
@rghaddab rghaddab marked this pull request as ready for review December 10, 2025 16:34
nrf-squash! [nrf noup] boot/zephyr/socs: nrf54h20 prj.conf for S2RAM

MULTITHREADING is needed by the mram driver for the nrf54h20_cpuapp
target

Signed-off-by: Riadh Ghaddab <[email protected]>
@rghaddab rghaddab force-pushed the rghaddab-sdk-mcuboot/select-multithread branch from 153ed1c to 9b6e31e Compare December 11, 2025 07:53
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 11, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#582

Signed-off-by: Nordic Builder <[email protected]>
@sonarqubecloud
Copy link

rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 11, 2025
rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 11, 2025
@bjarki-andreasen
Copy link
Contributor

With CONFIG_MULTITHREADING enabled, you may want to explicitly disable POWER_DOMAIN_NRFS_GDPWR or POWER_DOMAIN. The power domain drivers where previously excluded based on https://github.com/nrfconnect/sdk-zephyr/blob/e37df13e0e9ba6d539d84dba7fe14c3311d70500/drivers/power_domain/Kconfig.nrfs_gdpwr#L7 which will now be enabled, and enabling NRFS services

@rghaddab
Copy link
Contributor Author

Ok, let me investigate first all the unnecessary CONFIGS added when enabling MULTITHREADING before merging

@rghaddab rghaddab added DNM and removed DNM labels Dec 12, 2025
@rghaddab
Copy link
Contributor Author

@bjarki-andreasen the benefits from disabling those extra configs is very little.
I will do it in a separate PR

rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 12, 2025
@carlescufi carlescufi merged commit 897cf2c into nrfconnect:main Dec 16, 2025
6 checks passed
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Dec 16, 2025
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-mcuboot#582

Signed-off-by: Nordic Builder <[email protected]>
rghaddab pushed a commit to rghaddab/sdk-nrf that referenced this pull request Dec 19, 2025
carlescufi pushed a commit to nrfconnect/sdk-nrf that referenced this pull request Dec 22, 2025
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.

5 participants