From mboxrd@z Thu Jan  1 00:00:00 1970
From: Scott Wood <scottwood@freescale.com>
Date: Fri, 10 Aug 2012 11:05:48 -0500
Subject: [U-Boot] [PATCH 2/2] powerpc/mpc85xx/p1_p2_rdb: add all LAWs
 during SPL
In-Reply-To: <110EED8CC96DFC488B7E717A2027A27C11A4E4@039-SN1MPN1-002.039d.mgd.msft.net>
References: <20120809010333.GA25740@tyr.buserror.net>
	<20120809010618.GA25761@tyr.buserror.net>
	<CAEsOVNcZK=pv4=+uUm-g1MbiOg2bHSBKEHYGt1hw6VEbxuwHww@mail.gmail.com>
	<CAEsOVNdBhYGGoTdF1Wj2UpLsY_pVxrA+11D_Nq_5P1sMzs1reA@mail.gmail.com>
	<5023EAB2.7080800@freescale.com>
	<110EED8CC96DFC488B7E717A2027A27C11A4E4@039-SN1MPN1-002.039d.mgd.msft.net>
Message-ID: <5025315C.8040603@freescale.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

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@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@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