From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Tue, 29 Sep 2009 06:36:11 +0200
Subject: [U-Boot] [PATCH] ppc4xx: Merge PPC4xx DDR and DDR2 ECC handling
In-Reply-To: <4AC135E3.8040302@embedded-sol.com>
References: <1254156809-10678-1-git-send-email-sr@denx.de>
	<4AC135E3.8040302@embedded-sol.com>
Message-ID: <200909290636.11176.sr@denx.de>
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

Hi Felix,

On Tuesday 29 September 2009 00:17:07 Felix Radensky wrote:
> > Pieter, the p3p440 is currently the only PPC4xx DDR based board (non
> > DDR2) with ECC enabled. Could you please test this patch on the p3p440,
> > and let me know if you see any problems?
> >
> > And Felix & Grant, it would be great if you could test this patch on your
> > platforms as well.
> >
> > Thanks.
> 
> Patch does not apply. I've tried both u-boot.git and u-boot-ppc4xx.git.
> 
> Applying ppc4xx: Merge PPC4xx DDR and DDR2 ECC handling
> error: patch failed: cpu/ppc4xx/44x_spd_ddr2.c:198
> error: cpu/ppc4xx/44x_spd_ddr2.c: patch does not apply
> Patch failed at 0001.

Hmmm. It should apply upon u-boot-ppc4xx/master after your patch "ppc4xx: 
Reorganize DDR2 ECC handling" is applied. Could be that the whitespace fixup 
done by my "git am" of your patch is generating these problems.
 
> After fixing the problem manually I was able to successfully test
> your patch on 405EXr and 460EX boards with ECC.

OK, thanks.
 
Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de