Skip to contentSkip to conversation

zynq: spl: Add vectors section to linker script

PATCH New Peter Crosthwaite 2 participants 2 messages Aug 7, 2014 – Aug 8, 2014

Subsystems (2)
Arm (32-bit) Arm/Zynq architecture
lore
Share & tools
lore

Peter Crosthwaite

author
#0 lore raw

PATCH 1/1 zynq: spl: Add vectors section to linker script

The vectors section contains the _start symbol which is used as the program entry point. Add it to the linker script in same fashion as done for regular u-boot. This allows for correct generation of an spl elf with a non-zero entry point.

A similar change was applied to sunxi platform in
"sunxi: Fix u-boot-spl.lds to refer to .vectors"
(sha1: 9e5f80d823e3fd2a685b10ecf02009e34b86cff9)

This also allows for placement of the vector table at the hivecs location by setting the TEXT_BASE to 0xffff0000.

Tested-by Michal Simek <[email protected]>
Signed-off-by Peter Crosthwaite <[email protected]>

---
Changed since v1 (Michal Review):
s/fasion/fashion
elaborate commit SHA1 reference

---
 arch/arm/cpu/armv7/zynq/u-boot-spl.lds | 1 +
 1 file changed, 1 insertion(+)
arch/arm/cpu/armv7/zynq/u-boot-spl.lds +1 -0
index 0c4501e..0f2f756 100644
@@ -22,6 +22,7 @@ SECTIONS
 	.text :
 	{
 		__image_copy_start = .;
+		*(.vectors)
 		CPUDIR/start.o (.text*)
 		*(.text*)
 	} > .sram
Signature · 1 line
1.9.1
Applied
#1 lore raw

PATCH 1/1 zynq: spl: Add vectors section to linker script

Quoted 29 lines · Peter Crosthwaite

The vectors section contains the _start symbol which is used as the program entry point. Add it to the linker script in same fashion as done for regular u-boot. This allows for correct generation of an spl elf with a non-zero entry point.

A similar change was applied to sunxi platform in
"sunxi: Fix u-boot-spl.lds to refer to .vectors"
(sha1: 9e5f80d823e3fd2a685b10ecf02009e34b86cff9)

This also allows for placement of the vector table at the hivecs location by setting the TEXT_BASE to 0xffff0000.

Tested-by Michal Simek <[email protected]>
Signed-off-by Peter Crosthwaite <[email protected]>

---
Changed since v1 (Michal Review):
s/fasion/fashion
elaborate commit SHA1 reference

---
 arch/arm/cpu/armv7/zynq/u-boot-spl.lds | 1 +
 1 file changed, 1 insertion(+)
arch/arm/cpu/armv7/zynq/u-boot-spl.lds +0 -0
index 0c4501e..0f2f756 100644

arch/arm/cpu/armv7/zynq/u-boot-spl.lds SECTIONS · line 28

Quoted 7 lines
arch/arm/cpu/armv7/zynq/u-boot-spl.lds +1 -0
@@ -22,6 +22,7 @@ SECTIONS
 	.text :
 	{
 		__image_copy_start = .;
+		*(.vectors)
 		CPUDIR/start.o (.text*)
 		*(.text*)
 	} > .sram

Applied.

Thanks,
Michal

Signature · 14 lines
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140808/b99448cb/attachment.pgp>