[U-Boot] [PATCH v7 15/19] nand: mxc: Switch NAND SPL to generic SPL

[Benoît Thébaudeau]

On Friday, February 22, 2013 10:31:13 PM, Benoît Thébaudeau wrote:
> On Friday, February 22, 2013 10:27:57 PM, Benoît Thébaudeau wrote:
> > Hi Troy,
> > 
> > On Friday, February 22, 2013 10:30:21 PM, Troy Kisky wrote:
> > > On 2/22/2013 2:13 PM, Benoît Thébaudeau wrote:
> > > > Hi Fabio,
> > > >
> > > > On Friday, February 22, 2013 10:06:42 PM, Fabio Estevam wrote:
> > > >> Hi Benoît,
> > > >>
> > > >> On Fri, Feb 22, 2013 at 5:09 PM, Benoît Thébaudeau
> > > >> <benoit.thebaudeau at advansee.com> wrote:
> > > >>
> > > >>> It looks very much like the issue that Marek had on i.MX53 (which
> > > >>> self-resolved
> > > >>> for an unknown reason).
> > > >>>
> > > >>> Try to enable the debug trace to see if anything else is printed
> > > >>> after
> > > >>> display_dram_config(). I would especially be interested in:
> > > >>>          debug("relocation Offset is: %08lx\n", gd->reloc_off);
> > > >>>          debug("monitor flash len: %08lX\n", monitor_flash_len);
> > > >> Here is the output:
> > > >>
> > > >> U-Boot 2013.01 (Feb 22 2013 - 18:00:50)
> > > >>
> > > >> U-Boot code: 87F00000 -> 87F2C184  BSS: -> 87F311B4
> > > >> CPU:   Freescale i.MX31 rev 2.0 at 532 MHz.
> > > >> Reset cause: POR
> > > >> Board: MX31PDK
> > > >> monitor len: 000311B4
> > > >> ramsize: 08000000
> > > >> TLB table from 87ff0000 to 87ff4000
> > > >> Top of RAM usable for U-Boot at: 87ff0000
> > > >> Reserving 196k for U-Boot at: 87fbe000
> > > >> Reserving 640k for malloc() at: 87f1e000
> > > >> Reserving 32 Bytes for Board Info at: 87f1dfe0
> > > >> Reserving 128 Bytes for Global Data at: 87f1df60
> > > >> New Stack Pointer is: 87f1df50
> > > >> RAM Configuration:
> > > >> Bank #0: 80000000 128 MiB
> > > >> relocation Offset is: 000be000
> > > > CONFIG_SYS_TEXT_BASE seems fine according to this trace. But anyway,
> > > > can
> > > > you
> > > > test with this change in mx31pdk.h?
> > > > #define CONFIG_SPL_TEXT_BASE	0x84000000
> > > > #define CONFIG_SYS_TEXT_BASE	0x83000000
> > > >
> > > > Can you try to revert 10/19, only for arm1136/start.S (1 line to
> > > > restore)?
> > > > There
> > > > is an issue in this patch that I have already fixed in my v8.
> > > >
> > > > Best regards,
> > > > Benoît
> > > > _______________________________________________
> > > >
> > > How about changing
> > > 
> > > #define CONFIG_SYS_NAND_U_BOOT_SIZE   0x30000
> > > #define CONFIG_SYS_NAND_U_BOOT_SIZE   0x32000
> > > 
> > > is see monitor len: 311B4 above is over 0x30000
> > > 
> > > 
> > > Please excuse me if they are unrelated.
> > 
> > That could be it. Thanks! Fabio, please try.
> 
> But Fabio, how have you been able to program u-boot-with-spl.bin if it
> overlaps
> the next MTD partition, unless you destroyed what follows or
> CONFIG_SYS_NAND_U_BOOT_SIZE does not match your NAND images layout?

u-boot.bin is also a little bit larger than 0x30000 with my compiler, so there
is definitely an issue with CONFIG_SYS_NAND_U_BOOT_SIZE. If it really has to fit
in 0x30000 in your NAND, then try -ffunction-section, -fdata-sections and
-Wl,--gc-sections in the board config.mk.

Best regards,
Benoît