[U-Boot] Zynq bug and fix by generic board

[Masahiro Yamada]

Hi Michal and Albert,


On Wed, 4 Jun 2014 12:54:23 +0200
Michal Simek <michal.simek at xilinx.com> wrote:

> Hi Albert and Masahiro,
> 
> surprisingly I have tested zynq in Tom's repo and
> I am getting stuck in full u-boot.
> I have also tried SPL with MMC and it is hanging in the same location
> 
> U-Boot SPL 2014.07-rc2-00063-g3fe1a8545b55 (Jun 04 2014 - 11:57:12)
> mmc boot
> reading system.dtb
> spl_load_image_fat_os: error reading image system.dtb, err - -1
> reading u-boot-dtb.img
> reading u-boot-dtb.img
> 
> 
> U-Boot 2014.07-rc2-00063-g3fe1a8545b55 (Jun 04 2014 - 11:57:12)
> 
> I2C:   ready
> Memory: ECC disabled
> DRAM:  1 GiB
> 
> Problem is with relocation and it is not related to OF_CONTROL.


Yes.
I confirmed the same problem as you saw.


I did "git bisect" and figured out
the cause of the problem is the following commit:


commit 41623c91b09a0c865fab41acdaff30f060f29ad6
Author: Albert ARIBAUD <albert.u.boot at aribaud.net>
Date:   Tue Apr 15 16:13:51 2014 +0200

    arm: move exception handling out of start.S files
    
    Exception handling is basically identical for all ARM targets.
    Factorize it out of the various start.S files and into a
    single vectors.S file, and adjust linker scripts accordingly.
    
    Signed-off-by: Albert ARIBAUD <albert.u.boot at aribaud.net>




> I have added 3 pending patches on the top of current master next
> and "ARM: zynq: Enable generic board for Xilinx Zynq" fix the problem.
> That's why I think that is easier for just apply it to mainline repo
> because we need to apply this patch anyway.
> 
> Masahiro: Can you please check if the latest u-boot is working for you?
> Albert: If Masahiro confirms the same issue, can I send you one more pull
> request with these 3 fixes? (I have also run buildman on them too)
> (Enabling generic board, Fix message which is caused by enabling generic board,
> + one fix from Masahiro regarding compilation out of tree files for SPL).


Just try  "git  revert  41623c91b0" on the u-boot/master branch.
Everything will work fine.






Best Regards
Masahiro Yamada