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

libc/picolibc: Picolibc fixes #58462

Merged
merged 3 commits into from
Jun 17, 2023

Conversation

keith-packard
Copy link
Collaborator

These are a handful of fixes required to use picolibc as the default C library

@zephyrbot zephyrbot added area: Userspace Userspace area: TF-M ARM Trusted Firmware-M (TF-M) labels May 30, 2023
@zephyrbot zephyrbot requested review from ceolin and joerchan May 30, 2023 23:25
@zephyrbot zephyrbot added manifest manifest-sof DNM This PR should not be merged (Do Not Merge) labels May 30, 2023
@zephyrbot
Copy link
Collaborator

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

Name Old Revision New Revision Diff
sof zephyrproject-rtos/sof@ffbf9c2 (zephyr) zephyrproject-rtos/sof#28 zephyrproject-rtos/sof#28/files

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

Copy link
Contributor

@joerchan joerchan left a comment

Choose a reason for hiding this comment

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

Approving TF-M sample change

joerchan
joerchan previously approved these changes May 31, 2023
microbuilder
microbuilder previously approved these changes May 31, 2023
This conflicts with what the compiler expects for the definition of 'main'
when -ffreestanding is not used.

Signed-off-by: Keith Packard <[email protected]>
The default malloc arena when using picolibc is too large for the
hifive1_revb target.

Signed-off-by: Keith Packard <[email protected]>
… zero

This test fails when using picolibc with the default malloc arena
value. Fix this by explicitly setting the malloc arena size to zero, as
when using the minimal C library.

Signed-off-by: Keith Packard <[email protected]>
@keith-packard keith-packard dismissed stale reviews from microbuilder and joerchan via 522439a June 16, 2023 03:23
@zephyrbot zephyrbot added area: Portability Standard compliant code, toolchain abstraction area: CMSIS API Layer CMSIS-RTOS API Layer labels Jun 16, 2023
@keith-packard keith-packard removed the DNM This PR should not be merged (Do Not Merge) label Jun 16, 2023
@keith-packard
Copy link
Collaborator Author

I moved the modules/audio/sof fix out of this PR as that didn't fit the other changes here which are all minor fixes to samples.

@nashif nashif merged commit a476e87 into zephyrproject-rtos:main Jun 17, 2023
@keith-packard keith-packard deleted the picolibc-fixes branch June 18, 2023 02:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: CMSIS API Layer CMSIS-RTOS API Layer area: Portability Standard compliant code, toolchain abstraction area: TF-M ARM Trusted Firmware-M (TF-M) area: Userspace Userspace
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants