arm: zynq: fix a bug in Zynq linker script
Subsystems (2)
PATCH 1/1 arm: zynq: fix a bug in Zynq linker script
Commit 41623c91 moved exception handlers to ".vectores" section but it missed to adjust Zynq linker script.
Zynq boards hang up after relocation because "_start" symbol does not point to the correct address and gd->relocaddr gets insane.
--- arch/arm/cpu/armv7/zynq/u-boot.lds | 1 + 1 file changed, 1 insertion(+)
arch/arm/cpu/armv7/zynq/u-boot.lds +1 -0
@@ -18,6 +18,7 @@ SECTIONS .text : { *(.__image_copy_start) + *(.vectors) CPUDIR/start.o (.text*) *(.text*) }
Signature · 1 line
1.9.1
PATCH 1/1 arm: zynq: fix a bug in Zynq linker script
Hi Michal,
Quoted 19 lines · On Thu, 5 Jun 2014 19:47:45 +0900 Masahiro Yamada <[email protected]>…
Commit
41623c91moved exception handlers to ".vectores" section but it missed to adjust Zynq linker script.Zynq boards hang up after relocation because "
_start" symbol does not point to the correct address andgd->relocaddrgets insane.Signed-off-by Masahiro Yamada <[email protected]>Cc Albert ARIBAUD <[email protected]>Cc Michal Simek <[email protected]>Tested-by Michal Simek <[email protected]>--- arch/arm/cpu/armv7/zynq/u-boot.lds | 1 + 1 file changed, 1 insertion(+)arch/arm/cpu/armv7/zynq/u-boot.lds +0
-0
arch/arm/cpu/armv7/zynq/u-boot.lds SECTIONS · line 24
Quoted 7 lines
arch/arm/cpu/armv7/zynq/u-boot.lds +1
-0@@ -18,6 +18,7 @@ SECTIONS .text : { *(.__image_copy_start) + *(.vectors) CPUDIR/start.o (.text*) *(.text*) }
This bug should be fixed asap.
Because this patch is assigned to you, could you apply it and send a pull-request, please?
Best Regards
Masahiro Yamada
PATCH 1/1 arm: zynq: fix a bug in Zynq linker script
Quoted 35 lines · Masahiro Yamada
Hi Michal,
Quoted 19 lines · On Thu, 5 Jun 2014 19:47:45 +0900 Masahiro Yamada <[email protected]>…
Commit
41623c91moved exception handlers to ".vectores" section but it missed to adjust Zynq linker script.Zynq boards hang up after relocation because "
_start" symbol does not point to the correct address andgd->relocaddrgets insane.Signed-off-by Masahiro Yamada <[email protected]>Cc Albert ARIBAUD <[email protected]>Cc Michal Simek <[email protected]>Tested-by Michal Simek <[email protected]>--- arch/arm/cpu/armv7/zynq/u-boot.lds | 1 + 1 file changed, 1 insertion(+)arch/arm/cpu/armv7/zynq/u-boot.lds +0
-0arch/arm/cpu/armv7/zynq/u-boot.lds SECTIONS · line 24
Quoted 7 lines
arch/arm/cpu/armv7/zynq/u-boot.lds +1
-0@@ -18,6 +18,7 @@ SECTIONS .text : { *(.__image_copy_start) + *(.vectors) CPUDIR/start.o (.text*) *(.text*) }This bug should be fixed asap.
Quoted 2 lines
Because this patch is assigned to you, could you apply it and send a pull-request, please?
Applied and pull request sent.
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: 263 bytes Desc: OpenPGP digital signature URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140617/47309fd7/attachment.pgp>
PATCH 1/1 arm: zynq: fix a bug in Zynq linker script
Quoted 12 lines · Michal Simek
Quoted 6 lines · Masahiro Yamada
....
This bug should be fixed asap.
Because this patch is assigned to you, could you apply it and send a pull-request, please?
Applied and pull request sent.
Thanks,
Michal
This very same commit breaks the SPL on TT-01. Note, the SPL is not contributed yet (I'm about to).
Has anyone tested a current u-boot on one of the mx31 boards lately? Simply pulling in the vectors section in the linker script does not do the trick.
I'm a bit tight with my time currently, so I'd very much appreciate if someone could test and possibly offer a suggestion on what to change to get that working again, if necessary.
Helmut
--
Scanned by MailScanner.