Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add nodes required by MCUboot #1493

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

michalek-no
Copy link
Contributor

No description provided.

Copy link
Contributor

@sigvartmh sigvartmh left a comment

Choose a reason for hiding this comment

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

LGTM You just need to fix git tags. Commits needs to reference upstream PRs.

@gchwier
Copy link
Contributor

gchwier commented Feb 22, 2024

Please update commit messages with the link to upstream PRs and open PR on sdk-nrf with manifest update

Kconfig.zephyr Outdated
@@ -129,6 +129,7 @@ endif # HAS_FLASH_LOAD_OFFSET
config ROM_START_OFFSET
hex
prompt "ROM start offset" if !BOOTLOADER_MCUBOOT
default 0x800 if BOOTLOADER_MCUBOOT && SOC_PLATFORM_NRF54L
Copy link
Contributor

Choose a reason for hiding this comment

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

@gchwier
Copy link
Contributor

gchwier commented Feb 26, 2024

@michalek-no can you also add third commit: zephyrproject-rtos/zephyr@580fb84
this is also added in upstream PR: zephyrproject-rtos/zephyr#68393

@michalek-no michalek-no force-pushed the l15-regions-zephyr branch 2 times, most recently from 48caa6b to 06719df Compare February 26, 2024 13:17
@michalek-no
Copy link
Contributor Author

rebase

michalek-no and others added 3 commits February 29, 2024 11:46
Upstream PR: zephyrproject-rtos/zephyr#68393

The commit adds partitons on rram0.

Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
Upstream PR: zephyrproject-rtos/zephyr#68393

setting uart20 as default uart-mcumgr

Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
Upstream PR: zephyrproject-rtos/zephyr#68393

Due to number of IRQs the ROM offset, that contains MCUboot,
needs to be moved to address that is aligned with 2048 byte
long vector table.
This comes from Cortex-M requirement where vector table needs
to be placed at offset equal to lowest power of two number
that will fit vector table.

Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>
Signed-off-by: Mateusz Michalek <mateusz.michalek@nordicsemi.no>
Signed-off-by: Andrzej Puzdrowski <andrzej.puzdrowski@nordicsemi.no>
@jfischer-no jfischer-no merged commit 3912101 into nrfconnect:main Mar 1, 2024
12 checks passed
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.

9 participants