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

mkexport: Allow boards to supply custom gnu-elf.ld. #9618

Merged
merged 1 commit into from Jun 28, 2023

Conversation

g2gps
Copy link
Contributor

@g2gps g2gps commented Jun 26, 2023

Summary

Allows boards to supply their own gnu-elf linker script, used when building Nuttx applications and CONFIG_BUILD_KERNEL.

This is useful when building fully-linked applications (see #9395).

Impact

Makes it easier to customize Nuttx userspace application link configuration.

Testing

Using the arty:knsh configuration:

$ make export -j 16
$ cd ../apps && ./tools/mkimport.sh -z -x ../nuttx/nuttx-export-*.tar.gz
$ diff -q import/scripts/gnu-elf.ld ../nuttx/binfmt/libelf/gnu-elf.ld

After placing a custom linker script at boards/risc-v/litex/arty_ay/scripts/gnu-elf.ld

$ make export -j 16
$ cd ../apps && ./tools/mkimport.sh -z -x ../nuttx/nuttx-export-*.tar.gz
$ diff -q import/scripts/gnu-elf.ld ../nuttx/binfmt/libelf/gnu-elf.ld
Files import/scripts/gnu-elf.ld and ../nuttx/binfmt/libelf/gnu-elf.ld differ

Allows boards to supply their own gnu-elf linker script, used when building Nuttx applications and CONFIG_BUILD_KERNEL.

This is useful when building fully-linked applications (see apache#9395).
@xiaoxiang781216 xiaoxiang781216 merged commit 160e1a9 into apache:master Jun 28, 2023
24 of 26 checks passed
@g2gps g2gps deleted the pr/use-custom-gnu-elf branch June 28, 2023 01:27
@jerpelea jerpelea added this to To-Add in Release Notes - 12.3.0 Sep 26, 2023
@jerpelea jerpelea moved this from To-Add to Board in Release Notes - 12.3.0 Sep 27, 2023
@jerpelea jerpelea moved this from Board to done in Release Notes - 12.3.0 Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants