Re: [U-Boot] [PATCH] zynq: spl: Add vectors section to linker script
On 08/07/2014 02:26 PM, Peter Crosthwaite wrote: 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 0x. Tested-by: Michal Simek michal.si...@xilinx.com Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- 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(+) diff --git a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds index 0c4501e..0f2f756 100644 --- a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds +++ b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds @@ -22,6 +22,7 @@ SECTIONS .text : { __image_copy_start = .; + *(.vectors) CPUDIR/start.o (.text*) *(.text*) } .sram Applied. Thanks, Michal -- 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 signature.asc Description: OpenPGP digital signature ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] 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 0x. Tested-by: Michal Simek michal.si...@xilinx.com Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- 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(+) diff --git a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds index 0c4501e..0f2f756 100644 --- a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds +++ b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds @@ -22,6 +22,7 @@ SECTIONS .text : { __image_copy_start = .; + *(.vectors) CPUDIR/start.o (.text*) *(.text*) } .sram -- 1.9.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] zynq: spl: Add vectors section to linker script
On Tue, Aug 5, 2014 at 5:57 PM, Michal Simek michal.si...@xilinx.com wrote: On 08/04/2014 12:51 PM, Peter Crosthwaite wrote: The vectors section contains the _start symbol which is used as the program entry point. Add it to the linker script in same fasion as done fashion Fixed. for regular u-boot. This allows for correct generation of an spl elf with a non-zero entry point. Similar change was applied to sunxi platform in 9e5f80d823e3fd2a685b10ecf02009e34b86cff9. sunxi: Fix u-boot-spl.lds to refer to .vectors (sha1: 9e5f80d823e3fd2a685b10ecf02009e34b86cff9) Fixed. Regards, Peter ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] zynq: spl: Add vectors section to linker script
On 08/04/2014 12:51 PM, Peter Crosthwaite wrote: The vectors section contains the _start symbol which is used as the program entry point. Add it to the linker script in same fasion as done fashion for regular u-boot. This allows for correct generation of an spl elf with a non-zero entry point. Similar change was applied to sunxi platform in 9e5f80d823e3fd2a685b10ecf02009e34b86cff9. 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 0x. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- arch/arm/cpu/armv7/zynq/u-boot-spl.lds | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds index 0c4501e..0f2f756 100644 --- a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds +++ b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds @@ -22,6 +22,7 @@ SECTIONS .text : { __image_copy_start = .; + *(.vectors) CPUDIR/start.o (.text*) *(.text*) } .sram Just two issues in commit message but Tested-by: Michal Simek michal.si...@xilinx.com If there is no issue from others I will apply it to zynq branch. Thanks, Michal ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] 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 fasion as done for regular u-boot. This allows for correct generation of an spl elf with a non-zero entry point. Similar change was applied to sunxi platform in 9e5f80d823e3fd2a685b10ecf02009e34b86cff9. This also allows for placement of the vector table at the hivecs location by setting the TEXT_BASE to 0x. Signed-off-by: Peter Crosthwaite crosthwaite.pe...@gmail.com --- arch/arm/cpu/armv7/zynq/u-boot-spl.lds | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds index 0c4501e..0f2f756 100644 --- a/arch/arm/cpu/armv7/zynq/u-boot-spl.lds +++ b/arch/arm/cpu/armv7/zynq/u-boot-spl.lds @@ -22,6 +22,7 @@ SECTIONS .text : { __image_copy_start = .; + *(.vectors) CPUDIR/start.o (.text*) *(.text*) } .sram -- 1.9.1 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot