On Tue, 20 Jul 2010 23:43:14 +0200
Wolfgang Denk <w...@denx.de> wrote:
> Dear Kim,
>
> In message <20100720160808.b7ecf34c.kim.phill...@freescale.com> you wrote:
> >
> > > However, I cannot see any gentenv() use before relocation for your
> > > board, so the problem there is still unclear. I think I need your
> > > help to debug this...
> >
> > looks like arch/powerpc/cpu/mpc8xxx/ddr/options.c gets memory
> > interleaving configuration from the environment, using getenv().
>
> Ah! Can you please try the following patch - it's not intended to be a
> fix, just to verify my hypothesis. Thanks in advance.
indeed, your patch plus the following makes the 8572 boot properly
again.
diff --git a/arch/powerpc/cpu/mpc8xxx/ddr/options.c
b/arch/powerpc/cpu/mpc8xxx/ddr/options.c
index 0b9cb1d..692669f 100644
--- a/arch/powerpc/cpu/mpc8xxx/ddr/options.c
+++ b/arch/powerpc/cpu/mpc8xxx/ddr/options.c
@@ -252,8 +252,7 @@ unsigned int populate_memctl_options(int
all_DIMMs_registered,
#endif
i = getenv_r("ba_intlv_ctl", tmp, sizeof (tmp));
- if (i > 0) && (CONFIG_CHIP_SELECTS_PER_CTRL > 1)) {
- (CONFIG_CHIP_SELECTS_PER_CTRL > 1)) {
+ if ((i > 0) && (CONFIG_CHIP_SELECTS_PER_CTRL > 1)) {
if (strcmp(tmp, "cs0_cs1") == 0)
popts->ba_intlv_ctl = FSL_DDR_CS0_CS1;
else if (strcmp(tmp, "cs2_cs3") == 0)
Kim
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
