[U-Boot] [PATCH 2/2] powerpc/mpc85xx/p1_p2_rdb: add all LAWs during SPL

Scott Wood scottwood at freescale.com
Fri Aug 10 18:05:48 CEST 2012


On 08/10/2012 01:54 AM, Huang Changming-R66093 wrote:
> This maybe cause the overlap error:
>
> powerpc-linux-gnu-ld: section .resetvec loaded at
> [00000000ff800ffc,00000000ff800fff] overlaps section .data loaded at
> [00000000ff800ec8,00000000ff80102f]

What tree and toolchain are you using?  Did you apply patch 1/2 as well?
 Did it build OK for you before these two patches?

> maybe we can think about to remove or modify Kumar's patch:
> 
> commit 7639675131673e8f1582d760203a9af34fba9e79
> Author: Kumar Gala <galak at kernel.crashing.org>
> Date:   Thu Feb 3 09:02:13 2011 -0600
> 
>     powerpc/8xxx: Fix LAW init to respect pre-initialized entries
> 
>     If some pre-boot or earlier stage bootloader (NAND SPL) has setup LAW
>     entries consider them good and mark them used.
> 
>     In the NAND SPL case we skip re-initializing based on the law_table
>     since the SPL phase already did that.
> 
>     Signed-off-by: Kumar Gala <galak at kernel.crashing.org>

Kumar, was this patch to fix a bug (e.g. was one of the LAWs actually in
use at the time?) or just seemed nice?

-Scott




More information about the U-Boot mailing list