Skip to content

Conversation

@pillo79
Copy link
Contributor

@pillo79 pillo79 commented Nov 14, 2024

The linker script for this SoC was not including the LLEXT section definitions when CONFIG_LLEXT was enabled. This patch adds the necessary include directive to the linker script and fixes the build issue identified by CI.

The linker script for this SoC was not including the LLEXT section
definitions when CONFIG_LLEXT was enabled. This patch adds the
necessary include directive to the linker script and fixes the build
issue identified by CI.

Signed-off-by: Luca Burelli <[email protected]>
@pillo79 pillo79 added bug The issue is a bug, or the PR is fixing a bug Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. labels Nov 14, 2024
@pillo79 pillo79 marked this pull request as ready for review November 14, 2024 08:52
@pillo79 pillo79 added area: Linker Scripts area: llext Linkable Loadable Extensions labels Nov 14, 2024
Copy link
Contributor

@teburd teburd left a comment

Choose a reason for hiding this comment

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

I expect this will be a recurring issue.

@mmahadevan108 mmahadevan108 merged commit 07df2c5 into zephyrproject-rtos:main Nov 14, 2024
34 checks passed
@pillo79 pillo79 deleted the pr-fix-llext-sensry branch November 19, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Linker Scripts area: llext Linkable Loadable Extensions bug The issue is a bug, or the PR is fixing a bug Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants