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

Marek Vasut
2013-02-22T23:09:55Z

Dear Troy Kisky,

> On 2/22/2013 1:09 PM, Beno?t Th?baudeau wrote:
> > Hi Fabio,
> >
> > On Friday, February 22, 2013 8:30:36 PM, Fabio Estevam wrote:
> >> On Fri, Feb 22, 2013 at 4:14 PM, Fabio Estevam <festevam@gmail.com> wrote:
> >>> Just tested your patch series on a mx31pdk, but unfortunately it does
> >>> not fix mx31pdk boot.
> >>>
> >>> I will start looking at it, but any suggestion is welcome.
> >>
> >> Ok, just changed to a proper toolchain and the result is a bit better
> >> now:
> >>
> >> U-Boot 2013.01 (Feb 22 2013 - 16:25:48)
> >>
> >> CPU: Freescale i.MX31 rev 2.0 at 532 MHz.
> >> Reset cause: POR
> >> Board: MX31PDK
> >> DRAM: 128 MiB
> >>
> >> (hangs here).
> >
> > 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);
> >
> > We should also check if CONFIG_SPL_TEXT_BASE and CONFIG_SYS_TEXT_BASE
> > don't overlap with something depending on code size. That's especially
> > true for CONFIG_SYS_TEXT_BASE.
> >
> > After that, JTAG if no clue left... ;(
> >
> > Best regards,
> > Beno?t
>
> It does look familiar, can you try changing imximage.c
>
>
> *header_size_ptr = ROUND(sbuf->st_size + imxhdr->flash_offset, 512);
>
> to
>
> *header_size_ptr = ROUND(sbuf->st_size + imxhdr->flash_offset, 4096);
>
> (or whatever your nand sector size is)
> and see if it makes a difference?

This patch is needed for NAND boot on mx53. I still dont have the patches quite
ready, but we can omit SPL on mx53 ;-)

Best regards,
Marek Vasut