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

Zephyr/NCS mcuboot partition table and overlays #4452

Closed

Conversation

hiviah
Copy link
Contributor

@hiviah hiviah commented Dec 17, 2024

Build with mcuboot needs overlay to get the mcuboot there and partition table to tell it where to put it.

There are 2 options for overlays, one is without update, one with update - it's for testing purposes, e.g. you may not want to enter the recovery update until you have it all finished since then boot won't continue.

There is a pin defined in DTS which is read to check whether to enter the update or not. Check the source because I think for some weird reason it may have name you would not expect like led0.

Based on @TychoVrahe 's branch, not main.

Build must be given the overlay via absolute path e.g.:

west -v build -b t3w1_nrf52833 testapp3 -- -Dmcuboot_OVERLAY_CONFIG=/home/ondro/work/satoshilabs/repos/ncs/testapp3/mcuboot.conf

There are two overlays because one is without recovery/update and one with.

[no changelog]
@hiviah
Copy link
Contributor Author

hiviah commented Dec 17, 2024

Reference to older NCS build where bootloader and update was known working, along with BLE debug - https://github.com/trezor/trezor-firmware/tree/omikle/T3W1/devkit1_with_zephyr_with_bootloader - see core/embed/ble_zephyr_fw directory. Look at BUILD.md and weep.

Copy link

core UI changes device test click test persistence test
T2T1 Model T test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3B1 Safe 3 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
T3T1 Safe 5 test(screens) main(screens) test(screens) main(screens) test(screens) main(screens)
All main(screens)

@TychoVrahe
Copy link
Contributor

Cherry-picked the necessary changes into #4449 , so i am closing this.

Note that i also switched from using child image to newer sysbuild there, so the build command is eventually different (added that to readme in that branch).

@TychoVrahe TychoVrahe closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🔎 Needs review
Development

Successfully merging this pull request may close these issues.

2 participants