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

Extend ESP image load header #83611

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

marekmatej
Copy link

Extend the ESP program image header to cover new features and fix the debugging issues.

@zephyrbot
Copy link
Collaborator

zephyrbot commented Jan 6, 2025

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

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@5a10d43 zephyrproject-rtos/hal_espressif@51bf6d7 (zephyr) zephyrproject-rtos/hal_espressif@5a10d43a..51bf6d7b

All manifest checks OK

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

@zephyrbot zephyrbot added manifest manifest-hal_espressif DNM This PR should not be merged (Do Not Merge) labels Jan 6, 2025
@marekmatej marekmatej force-pushed the bugfix/esp_header_extend branch from 5a41b59 to d92e47a Compare January 6, 2025 22:27
@marekmatej marekmatej force-pushed the bugfix/esp_header_extend branch from d92e47a to 26f963b Compare January 7, 2025 13:38
@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Jan 7, 2025
@marekmatej marekmatej force-pushed the bugfix/esp_header_extend branch from 26f963b to e1c6ebe Compare January 8, 2025 13:03
@marekmatej marekmatej marked this pull request as ready for review January 8, 2025 13:04
@zephyrbot zephyrbot added the platform: ESP32 Espressif ESP32 label Jan 8, 2025
@zephyrbot zephyrbot requested a review from uLipe January 8, 2025 13:04
sylvioalves
sylvioalves previously approved these changes Jan 8, 2025
raffarost
raffarost previously approved these changes Jan 8, 2025
wmrsouza
wmrsouza previously approved these changes Jan 8, 2025
uLipe
uLipe previously approved these changes Jan 8, 2025
Use newest version to support latest features and bugfixes.

Signed-off-by: Marek Matej <marek.matej@espressif.com>
@marekmatej marekmatej dismissed stale reviews from uLipe, wmrsouza, raffarost, and sylvioalves via bd435f0 January 13, 2025 12:31
@marekmatej marekmatej force-pushed the bugfix/esp_header_extend branch from e1c6ebe to bd435f0 Compare January 13, 2025 12:31
Add new fields to the `esp_image_load_header_t`

* provide IROM and DROM fields to fix debugging features
* extend the header to up to 96 Bytes for future use

Signed-off-by: Marek Matej <marek.matej@espressif.com>
@marekmatej marekmatej force-pushed the bugfix/esp_header_extend branch from bd435f0 to 97ab279 Compare January 13, 2025 13:04
@kartben kartben merged commit d276cf7 into zephyrproject-rtos:main Jan 13, 2025
26 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.

8 participants