platform: nordic_nrf: align to nrfx 4.0 #220
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
masz-nordic
force-pushed
the
align_nrfx4
branch
from
d2739f1 to
8f93ff1
Compare
jaz1-nordic
force-pushed
the
align_nrfx4
branch
from
8f93ff1 to
9de84c3
Compare
masz-nordic
force-pushed
the
align_nrfx4
branch
2 times, most recently
from
55851f1 to
bf6117c
Compare
tomi-font
reviewed
Nov 13, 2025
Collaborator
tomi-font
left a comment
tomi-font
left a comment
There was a problem hiding this comment.
Looks good, just minor comments, I will submit this commit to upstream TF-M once we have a tag for nrfx 4.0 in https://github.com/zephyrproject-rtos/hal_nordic.
masz-nordic
force-pushed
the
align_nrfx4
branch
from
bf6117c to
401fd77
Compare
Collaborator
Contributor
|
I don't see anything problematic in the code, but we need a nightly TF-M CI run for this to be sure. Please apply commits tags, since Tomi already opened the upstream PRs add the commit tag: Ideally you can just cherry pick his commits and just apply the tag that I said, this will make it identifiable which upstream PRs the relate to. |
Collaborator
|
now fromtrees |
This was referenced Nov 21, 2025
masz-nordic
force-pushed
the
align_nrfx4
branch
from
8ce1ac8 to
a9a4995
Compare
Collaborator
|
I just updated zephyrproject-rtos/trusted-firmware-m#156 so please do the same here. |
Align common files. Change-Id: I73509382d31476f88c20dee42fde4de6126f565d Signed-off-by: Marcin Szymczyk <marcin.szymczyk@nordicsemi.no> (cherry picked from commit 8ff947dd53f91cc850ee05164d9df5e377a48fbd)
…factor Align with changes needed for nrfx 4.0. Refactor to use nrfx layer instead of HAL. Change-Id: Ibbffe2c2e27ea98ac08e0e23f4c33603c04fe5ea Signed-off-by: Marcin Szymczyk <marcin.szymczyk@nordicsemi.no> (cherry picked from commit f07f10b9c028ee1de036037264e6b1929432af1d)
nrfx_config from nrfx's templates can be used. Change-Id: I5f532376ef0c9c73ec19b3933685a644a35a9c19 Signed-off-by: Marcin Szymczyk <marcin.szymczyk@nordicsemi.no> Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
Update error codes. Change-Id: I492f82cfb777a5acfac536ae5e647ed8b7a61df3 Signed-off-by: Marcin Szymczyk <marcin.szymczyk@nordicsemi.no> Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
… last Since `compiler_ext_defs.h` should define compiler symbols only if they have not been previously defined, it should be included as the last header to avoid redefine warnigns. Fix in all files. Change-Id: I010b8885b4e1b150ec527f0b246b95a4dab4902a Signed-off-by: Marcin Szymczyk <marcin.szymczyk@nordicsemi.no> Signed-off-by: Tomi Fontanilles <tomi.fontanilles@nordicsemi.no>
Use nrfx 4.0 release. Change-Id: Ic5e2faf118f2b2c673587570e32dfc49e172cb9b Signed-off-by: Nikodem Kastelik <nikodem.kastelik@nordicsemi.no>
masz-nordic
force-pushed
the
align_nrfx4
branch
from
a9a4995 to
2069677
Compare
Author
|
@tomi-font done |
tomi-font
approved these changes
Nov 27, 2025
Vge0rge
approved these changes
Nov 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Align common files.