Re: [U-Boot] [PATCH] arm: zynq: fix a bug in Zynq linker script
On 06/17/2014 12:38 PM, Michal Simek wrote: On 06/17/2014 10:45 AM, Masahiro Yamada wrote: 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. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] arm: zynq: fix a bug in Zynq linker script
On 06/17/2014 10:45 AM, Masahiro Yamada wrote:
> Hi Michal,
>
> On Thu, 5 Jun 2014 19:47:45 +0900
> Masahiro Yamada wrote:
>
>> 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.
>>
>> Signed-off-by: Masahiro Yamada
>> Cc: Albert ARIBAUD
>> Cc: Michal Simek
>> Tested-by: Michal Simek
>> ---
>> arch/arm/cpu/armv7/zynq/u-boot.lds | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/cpu/armv7/zynq/u-boot.lds
>> b/arch/arm/cpu/armv7/zynq/u-boot.lds
>> index 69500a6..4dc9bb0 100644
>> --- a/arch/arm/cpu/armv7/zynq/u-boot.lds
>> +++ b/arch/arm/cpu/armv7/zynq/u-boot.lds
>> @@ -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?
Applied and pull request sent.
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
Re: [U-Boot] [PATCH] arm: zynq: fix a bug in Zynq linker script
Hi Michal,
On Thu, 5 Jun 2014 19:47:45 +0900
Masahiro Yamada wrote:
> 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.
>
> Signed-off-by: Masahiro Yamada
> Cc: Albert ARIBAUD
> Cc: Michal Simek
> Tested-by: Michal Simek
> ---
> arch/arm/cpu/armv7/zynq/u-boot.lds | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/cpu/armv7/zynq/u-boot.lds
> b/arch/arm/cpu/armv7/zynq/u-boot.lds
> index 69500a6..4dc9bb0 100644
> --- a/arch/arm/cpu/armv7/zynq/u-boot.lds
> +++ b/arch/arm/cpu/armv7/zynq/u-boot.lds
> @@ -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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] 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.
Signed-off-by: Masahiro Yamada
Cc: Albert ARIBAUD
Cc: Michal Simek
Tested-by: Michal Simek
---
arch/arm/cpu/armv7/zynq/u-boot.lds | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/cpu/armv7/zynq/u-boot.lds
b/arch/arm/cpu/armv7/zynq/u-boot.lds
index 69500a6..4dc9bb0 100644
--- a/arch/arm/cpu/armv7/zynq/u-boot.lds
+++ b/arch/arm/cpu/armv7/zynq/u-boot.lds
@@ -18,6 +18,7 @@ SECTIONS
.text :
{
*(.__image_copy_start)
+ *(.vectors)
CPUDIR/start.o (.text*)
*(.text*)
}
--
1.9.1
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
