Skip to content

Commit 3045d61

Browse files
shenkitrini
authored andcommitted
aspeed/spl: Remove OVERLAY from linker script
The generic arm linker script contains this section: .bss __rel_dyn_start (OVERLAY) : { ... } The (OVERLAY) syntax in the description causes the .bss section to be included in the NOR area of the image: $ objdump -t -j .bss spl/u-boot-spl SYMBOL TABLE: 0000c61c l d .bss 00000000 .bss 0000c640 l O .bss 00000040 __value.0 0000c68c g O .bss 00000000 __bss_end 0000c61c g O .bss 00000000 __bss_start 0000c680 g O .bss 0000000c stdio_devices This is what the custom linker script tries to avoid, as the NOR area is read-only. Remove the OVERLAY syntax to fix the BSS location: $ objdump -t -j .bss spl/u-boot-spl SYMBOL TABLE: 83000000 l d .bss 00000000 .bss 83000000 l O .bss 00000040 __value.0 0000c61c g O .bss 00000000 __image_copy_end 8300004c g O .bss 00000000 __bss_end 83000000 g O .bss 00000000 __bss_start 83000040 g O .bss 0000000c stdio_devices This restores the state of the linker script before the patch that fixed the linker lists issue. Fixes: f6810b7 ("aspeed/ast2600: Fix SPL linker script") Signed-off-by: Joel Stanley <[email protected]>
1 parent 154cffa commit 3045d61

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Diff for: arch/arm/mach-aspeed/ast2600/u-boot-spl.lds

+1-1
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ SECTIONS
6868

6969
_image_binary_end = .;
7070

71-
.bss __rel_dyn_start (OVERLAY) : {
71+
.bss : {
7272
__bss_start = .;
7373
*(.bss*)
7474
. = ALIGN(4);

0 commit comments

Comments
 (0)