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

MCUboot as application for ESP32xx #54835

Merged
merged 6 commits into from May 25, 2023

Conversation

marekmatej
Copy link

@marekmatej marekmatej commented Feb 14, 2023

This work provides MCUboot as default 2nd stage bootloader for ESP32xx targets, replacing/complementing currently used IDF bootloader.
Currently supported SoCs are:

  • ESP32
  • ESP32S2
  • ESP32C3
  • ESP32S3

@marekmatej marekmatej removed the DNM This PR should not be merged (Do Not Merge) label May 15, 2023
@marekmatej marekmatej force-pushed the mcuboot_esp_sb branch 8 times, most recently from 5538230 to cfdb4d8 Compare May 24, 2023 17:42
Marek Matej added 6 commits May 24, 2023 20:00
This make MCUboot build as Zephyr application.
Providing optinal 2nd stage bootloader to the
IDF bootloader, which is used by default.
This provides more flexibility when building
and loading multiple images and aims to
brings better DX to users by using the sysbuild.
MCUboot and applications has now separate
linker scripts.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Make use of the IDF partition table only if asked to.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Allow builds which has CONFIG_MULTITHREADING disabled.
This is reduce code footprint which is handy for
constrained targets as bootloaders.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Allow builds which has CONFIG_MULTITHREADING disabled.
This is reduce code footprint which is handy for
constrained targets as bootloaders.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Cover the 2nd stage bootloader options and
the extend the application build instructions
to include sysbuild.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
Update to support latest features.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
@carlescufi carlescufi merged commit fd346e8 into zephyrproject-rtos:main May 25, 2023
54 checks passed
@npitre
Copy link
Collaborator

npitre commented May 25, 2023

I just noticed the merging of this PR added commit f0b458a to
the main branch. This commit is completely bogus and does in fact make
footprint worse. I ask that this commit be reverted.

@sylvioalves
Copy link
Collaborator

@npitre, in case this and 13f8dad gets reverted, build will break. So better to revert all PR if necessary.

@npitre
Copy link
Collaborator

npitre commented May 25, 2023

I'm not talking about "arch: xtensa: Enable builds without the
multithreading" here which, for all I know about ARC, might be necessary.

I'm talking about "arch: riscv: Enable builds without the multithreading"
which is useless.

@marekmatej
Copy link
Author

@npitre in the context of the whole PR and the ESP32 RISCV chips, it is actually useful and it allows us to disable the MULTITHREADING which makes the code footprint lower.

@npitre
Copy link
Collaborator

npitre commented May 26, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants