Skip to content

soc: andestech: linker.ld: Handle symtab/strtab/shstrtab to fix warnings #88530

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

Merged
merged 1 commit into from
Apr 22, 2025

Conversation

thughes
Copy link
Contributor

@thughes thughes commented Apr 11, 2025

lld will produce warnings for the symtab, strtab, and shstrtab sections
if --orphan-handling=warn is specified and there are no matching rules
in the linker script for these sections. Handle these sections when
building with lld to prevent the warnings.

This is exactly the same as commit
c420733, but for the AE350 linker
script.

lld will produce warnings for the symtab, strtab, and shstrtab sections
if --orphan-handling=warn is specified and there are no matching rules
in the linker script for these sections. Handle these sections when
building with lld to prevent the warnings.

This is exactly the same as commit
c420733, but for the AE350 linker
script.

Signed-off-by: Tom Hughes <[email protected]>
@thughes thughes marked this pull request as ready for review April 11, 2025 22:17
@thughes
Copy link
Contributor Author

thughes commented Apr 17, 2025

@carlocaione @cfriedt @ycsin @jimmyzhe Would you be able to review or help find an appropriate reviewer since the Pull Request Assigner didn't add anyone? (I don't have permission to add reviewers or assignees.)

@cfriedt
Copy link
Member

cfriedt commented Apr 17, 2025

It would appear that there are no maintainers for Andes in the MAINTAINERS.yml or CODEOWNERS files, otherwise, this would have been auto-assigned.

Can someone from Andes please step up to maintain their platform in Zephyr?

@ycsin ycsin requested a review from jimmyzhe April 18, 2025 05:09
@jimmyzhe
Copy link
Collaborator

jimmyzhe commented Apr 18, 2025

Sorry for not noticing this PR, thank you for your contribution.
I will try to add Andes platform to MAINTAINERS.yml in another PR later.

@thughes
Copy link
Contributor Author

thughes commented Apr 18, 2025

Thanks @jimmyzhe.

I will need one more approval from a reviewer in order for this to be merged.

Based on #76462, should that be @kevinwang821020 or @wtlee1995?

@kartben kartben merged commit 7f0cd66 into zephyrproject-rtos:main Apr 22, 2025
26 checks passed
@thughes thughes deleted the push-vnslxlnlromx branch April 22, 2025 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants