Skip to content

Fix hal_nordic manifest history#4024

Merged
nordicjm merged 5 commits into
nrfconnect:mainfrom
SebastianBoe:fix_commit_message
Apr 27, 2026
Merged

Fix hal_nordic manifest history#4024
nordicjm merged 5 commits into
nrfconnect:mainfrom
SebastianBoe:fix_commit_message

Conversation

@SebastianBoe
Copy link
Copy Markdown
Contributor

The git history has become a mess.

I have described my analysis of what has happened in 3375423.

This PR attempts to fix the history and also update to the latest hal_nordic with some fromtree commits.

This reverts commit 328e2f2.

I'm not sure how it was created, but it does not seem to have followed
the process correctly.

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
…ate nrfx 4.2.1"

This reverts commit 241d699.

In 241d699, we updated
hal_nordic in sdk-zephyr with a fromlist from
44fd3d44b15cb75f80a25b4679f91d2787e28664 to
7142bc9c3c94f06e3b520890dde93d363c0e67bf

But the fromlist must not have been merged because I find no such
equivalent change in upstream Zephyr.

What I do find in upstream Zephyr is this sequence:

f9799c5 (Sebastian Bøe           2026-04-21 206)       revision: c8f0c2b6ecf66b0a4af24b5272d879c305868feb
191f572 (Krzysztof Chruściński   2026-04-16 206)       revision: 0424db2d8320cfe417af5675b5bfd2b9a8b43d23
f63ae95 (Sebastian Bøe           2026-04-13 206)       revision: af3848538da37360913bc68eeef40f32430b7c52
02b7d68 (Håkon Amundsen          2026-03-27 206)       revision: 44fd3d44b15cb75f80a25b4679f91d2787e28664

So from 44f to af3, not from 44f to 714.

Since 714 is not in upstream, I can't update hal_nordic without
causing a conflict. So I will revert the fromlist, and then fromtree
the 4 correct commits.

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

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

Name Old Revision New Revision Diff
hal_nordic zephyrproject-rtos/hal_nordic@af38485 zephyrproject-rtos/hal_nordic@c8f0c2b (master) zephyrproject-rtos/hal_nordic@af384853..c8f0c2b6

All manifest checks OK

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

NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Apr 27, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#4024

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
@SebastianBoe SebastianBoe changed the title Fix commit message Fix hal_nordic manifest history Apr 27, 2026
SebastianBoe and others added 3 commits April 27, 2026 10:54
update hal_nordic revision.

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
(cherry picked from commit f63ae95)
Update hal_nordic with fix in nrfx_gppi helper module.

Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
(cherry picked from commit 191f572)
Update hal_nordic revision.

Requires zephyrproject-rtos/hal_nordic#357

Signed-off-by: Sebastian Bøe <sebastian.boe@nordicsemi.no>
(cherry picked from commit f9799c5)
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Apr 27, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#4024

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
Copy link
Copy Markdown

@shanthanordic shanthanordic left a comment

Choose a reason for hiding this comment

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

approved for scope

@nordicjm nordicjm merged commit d947269 into nrfconnect:main Apr 27, 2026
17 checks passed
NordicBuilder added a commit to NordicBuilder/sdk-nrf that referenced this pull request Apr 27, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#4024

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
nordicjm pushed a commit to nrfconnect/sdk-nrf that referenced this pull request Apr 27, 2026
Automatically created by action-manifest-pr GH action from PR:
nrfconnect/sdk-zephyr#4024

Signed-off-by: Nordic Builder <pylon@nordicsemi.no>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants