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

soc: espressif: update linkers and hal_espressif #67218

Merged

Conversation

sylvioalves
Copy link
Collaborator

@sylvioalves sylvioalves commented Jan 4, 2024

This PR also fixes #66351.

@zephyrbot zephyrbot added the platform: ESP32 Espressif ESP32 label Jan 4, 2024
@sylvioalves sylvioalves requested a review from marekmatej January 4, 2024 17:22
@zephyrbot zephyrbot added the area: RISCV RISCV Architecture (32-bit & 64-bit) label Jan 4, 2024
@sylvioalves sylvioalves requested a review from nashif January 4, 2024 17:22
@zephyrbot
Copy link
Collaborator

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
hal_espressif zephyrproject-rtos/hal_espressif@e98ce93 zephyrproject-rtos/hal_espressif@a248460 (zephyr) zephyrproject-rtos/[email protected]

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

Adds common thread-local-storage.ld provided
by Zephyr. This also fixes a wrong xtensa_core entry
that should be riscv_core.

Signed-off-by: Sylvio Alves <[email protected]>
Make sure some ROM libc calls are weak to allow
Zephyr's libc implementation instead.

Fixes zephyrproject-rtos#66351

Signed-off-by: Sylvio Alves <[email protected]>
@sylvioalves sylvioalves force-pushed the bugfix/esp32c3_riscv_linker branch from 4340cfd to b9d3ef6 Compare January 4, 2024 18:36
@barnas-michal
Copy link
Collaborator

I've checked this PR and it fixes all my issues with the (xiao) ESP32-C3. My programs are working correctly now and the wifi example is not doing any crashes when the shell is disabled.

Copy link

@marekmatej marekmatej left a comment

Choose a reason for hiding this comment

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

LGTM

@carlescufi carlescufi merged commit 617b45a into zephyrproject-rtos:main Jan 8, 2024
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: RISCV RISCV Architecture (32-bit & 64-bit) manifest manifest-hal_espressif platform: ESP32 Espressif ESP32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problem calling POSIX libc functions on espressif boards
6 participants