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

2013-02-22 Thread Benoît Thébaudeau
Dear Marek Vasut, On Saturday, February 23, 2013 12:56:32 AM, Marek Vasut wrote: > Dear Benoît Thébaudeau, > > > Dear Marek Vasut, > > > > On Saturday, February 23, 2013 12:11:47 AM, Marek Vasut wrote: > > > Dear Fabio Estevam, > > > > > > > On Fri, Feb 22, 2013 at 7:42 PM, Benoît Thébaudeau >

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

2013-02-22 Thread Marek Vasut
Dear Benoît Thébaudeau, > Dear Marek Vasut, > > On Saturday, February 23, 2013 12:11:47 AM, Marek Vasut wrote: > > Dear Fabio Estevam, > > > > > On Fri, Feb 22, 2013 at 7:42 PM, Benoît Thébaudeau > > > > > > wrote: > > > > Great! So can you confirm that for my series, all I have to change > >

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

2013-02-22 Thread Benoît Thébaudeau
Dear Marek Vasut, On Saturday, February 23, 2013 12:11:47 AM, Marek Vasut wrote: > Dear Fabio Estevam, > > > On Fri, Feb 22, 2013 at 7:42 PM, Benoît Thébaudeau > > > > wrote: > > > Great! So can you confirm that for my series, all I have to change is: > > > #define CONFIG_SYS_NAND_U_BOOT_SIZE

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

2013-02-22 Thread Marek Vasut
Dear Fabio Estevam, > On Fri, Feb 22, 2013 at 7:42 PM, Benoît Thébaudeau > > wrote: > > Great! So can you confirm that for my series, all I have to change is: > > #define CONFIG_SYS_NAND_U_BOOT_SIZE 0x32000 > > Yes, correct. This is the only change you need to do in your series. > > > Did yo

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

2013-02-22 Thread Marek Vasut
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 wrote: > >>> Just tested your patch series on a mx31pdk, but unfortunately it does > >>> not f

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 7:42 PM, Benoît Thébaudeau wrote: > Great! So can you confirm that for my series, all I have to change is: > #define CONFIG_SYS_NAND_U_BOOT_SIZE 0x32000 Yes, correct. This is the only change you need to do in your series. > Did you also have a chance to test nand comma

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

2013-02-22 Thread Benoît Thébaudeau
Hi Fabio, On Friday, February 22, 2013 11:08:14 PM, Fabio Estevam wrote: > On Fri, Feb 22, 2013 at 7:03 PM, Fabio Estevam wrote: > > On Fri, Feb 22, 2013 at 7:02 PM, Fabio Estevam wrote: > > > >> U-Boot code: 87F0 -> 87F2C184 BSS: -> 87F311B4 > >> CPU: Freescale i.MX31 rev 2.0 at 532 MHz.

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 7:03 PM, Fabio Estevam wrote: > On Fri, Feb 22, 2013 at 7:02 PM, Fabio Estevam wrote: > >> U-Boot code: 87F0 -> 87F2C184 BSS: -> 87F311B4 >> CPU: Freescale i.MX31 rev 2.0 at 532 MHz. >> Reset cause: WDOG >> Board: MX31PDK >> monitor len: 000311B4 >> ramsize: 080

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 7:02 PM, Fabio Estevam wrote: > U-Boot code: 87F0 -> 87F2C184 BSS: -> 87F311B4 > CPU: Freescale i.MX31 rev 2.0 at 532 MHz. > Reset cause: WDOG > Board: MX31PDK > monitor len: 000311B4 > ramsize: 0800 > TLB table from 87ff to 87ff4000 > Top of RAM usable for

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 6:51 PM, Fabio Estevam wrote: > On Fri, Feb 22, 2013 at 6:30 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, 20

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 6:31 PM, Benoît Thébaudeau wrote: > 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? Yes, I program u-boo

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 6:30 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 >>> wrote: >>> It looks very

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

2013-02-22 Thread 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, > > > > >

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

2013-02-22 Thread Benoît Thébaudeau
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

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

2013-02-22 Thread Benoît Thébaudeau
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 > >> wrote: > >>

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

2013-02-22 Thread Troy Kisky
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 wrote: It looks very much like the issue that Marek had on i.MX53 (which self-resolved for an unknown reason)

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

2013-02-22 Thread Benoît Thébaudeau
Hi Fabio, On Friday, February 22, 2013 10:26:47 PM, Fabio Estevam wrote: > On Fri, Feb 22, 2013 at 6:13 PM, Benoît Thébaudeau > wrote: > > > 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

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 6:13 PM, Benoît Thébaudeau wrote: > 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_BASE0x8400 > #define CONFIG_SYS_TEXT_BASE0x8300 > > Can you try to revert 10/19,

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

2013-02-22 Thread Benoît Thébaudeau
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 > 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 d

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

2013-02-22 Thread Fabio Estevam
Hi Troy, On Fri, Feb 22, 2013 at 5:55 PM, Troy Kisky wrote: > 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 n

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

2013-02-22 Thread Fabio Estevam
Hi Benoît, On Fri, Feb 22, 2013 at 5:09 PM, Benoît Thébaudeau 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 especial

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

2013-02-22 Thread 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 wrote: Just tested your patch series on a mx31pdk, but unfortunately it does not fix mx31pdk boot. I will start looking at it

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

2013-02-22 Thread Benoît Thébaudeau
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 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.

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

2013-02-22 Thread Fabio Estevam
On Fri, Feb 22, 2013 at 4:14 PM, Fabio Estevam 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

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

2013-02-22 Thread Fabio Estevam
Hi Benoît, On Fri, Feb 15, 2013 at 6:54 PM, Benoît Thébaudeau wrote: > This also fixes support for mx31pdk and tx25, which had been broken by commit > e05e5de7fae5bec79617e113916dac6631251156. Just tested your patch series on a mx31pdk, but unfortunately it does not fix mx31pdk boot. I will sta

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

2013-02-15 Thread Benoît Thébaudeau
This also fixes support for mx31pdk and tx25, which had been broken by commit e05e5de7fae5bec79617e113916dac6631251156. Signed-off-by: Benoît Thébaudeau --- Changes in v7: None Changes in v6: - Automate 'u-boot.imx' and 'SPL' make targets for all imx processors. - Move board_init_f() to .c. -