[U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-03-06 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 Tested-by: Fabio Estevam --- Changes in v9: None Changes in v8: - Update doc/README.arm-relocation. - Drop useless line feed at end of inline

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-03-31 Thread Albert ARIBAUD
Hi Benoît, I have managed to rebase your patch series and have tested it over the ARM targets. This particular patch was the only one to cause an issue, and an amusing one at that: On Wed, 6 Mar 2013 19:59:24 +0100, Benoît Thébaudeau wrote: > This also fixes support for mx31pdk and tx25, which

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-03-31 Thread Benoît Thébaudeau
Hi Albert, On Sunday, March 31, 2013 7:30:24 PM, Albert ARIBAUD wrote: > Hi Benoît, > > I have managed to rebase your patch series and have tested it over the > ARM targets. This particular patch was the only one to cause an issue, > and an amusing one at that: > > On Wed, 6 Mar 2013 19:59:24 +

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-01 Thread Albert ARIBAUD
Hi Benoît, On Mon, 1 Apr 2013 02:30:00 +0200 (CEST), Benoît Thébaudeau wrote: > Hi Albert, > > On Sunday, March 31, 2013 7:30:24 PM, Albert ARIBAUD wrote: > > Hi Benoît, > > > > I have managed to rebase your patch series and have tested it over the > > ARM targets. This particular patch was th

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-01 Thread Benoît Thébaudeau
Hi Albert, On Monday, April 1, 2013 10:26:21 AM, Albert ARIBAUD wrote: > Hi Benoît, > > On Mon, 1 Apr 2013 02:30:00 +0200 (CEST), Benoît Thébaudeau > wrote: > > > Hi Albert, > > > > On Sunday, March 31, 2013 7:30:24 PM, Albert ARIBAUD wrote: > > > Hi Benoît, > > > > > > I have managed to reba

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-01 Thread Albert ARIBAUD
Hi Benoît, On Mon, 1 Apr 2013 15:13:05 +0200 (CEST), Benoît Thébaudeau wrote: > OK, then it's probably better if I send v10 ASAP. I may not have access to my > repo before this date, and I have not kept my v9 bundle in my outbox, so can > you > just send me back this bundle please? I will try t

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-02 Thread Benoît Thébaudeau
Hi Albert, On Monday, April 1, 2013 5:53:55 PM, Albert ARIBAUD wrote: > Hi Benoît, > > On Mon, 1 Apr 2013 15:13:05 +0200 (CEST), Benoît Thébaudeau > wrote: > > > OK, then it's probably better if I send v10 ASAP. I may not have access to > > my > > repo before this date, and I have not kept my v

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-03 Thread Albert ARIBAUD
Hi Benoît, On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau wrote: > Hi Albert, > > Here is the v10 bundle for those who want to test: > http://dl.free.fr/vdXBGExyq Thanks, will build-test it ASAP. People with ARM (expecially tx25 and mx31pdk) and non-ARM boards please run-test it

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-03 Thread Albert ARIBAUD
On Wed, 3 Apr 2013 10:05:57 +0200, Albert ARIBAUD wrote: > > The resulting HEAD is identical to yours, except for > > board/freescale/mx31ads/u-boot.lds in which you had removed the following > > unrelated line for 30/30: > > __bss_end = .; > > > > Regarding this line, it is also present

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-03 Thread Fabio Estevam
Hi Benoît, On Wed, Apr 3, 2013 at 3:30 AM, Benoît Thébaudeau wrote: > Here is the v10 bundle for those who want to test: > http://dl.free.fr/vdXBGExyq I would like to test it on mx31. I downloaded the mxc-nand-v10.bundle file, but I was not able to open it. Any suggestion? Regards, Fabio Est

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-03 Thread Benoît Thébaudeau
Hi Fabio, On Wednesday, April 3, 2013 11:01:51 PM, Fabio Estevam wrote: > Hi Benoît, > > On Wed, Apr 3, 2013 at 3:30 AM, Benoît Thébaudeau > wrote: > > > Here is the v10 bundle for those who want to test: > > http://dl.free.fr/vdXBGExyq > > I would like to test it on mx31. I downloaded the mxc

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-04 Thread Albert ARIBAUD
Hi Benoît, On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau wrote: > The resulting HEAD is identical to yours, except for > board/freescale/mx31ads/u-boot.lds in which you had removed the following > unrelated line for 30/30: > __bss_end = .; > > Regarding this line, it is al

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-04 Thread Benoît Thébaudeau
Hi Albert, On Thursday, April 4, 2013 11:07:23 AM, Albert ARIBAUD wrote: > Hi Benoît, > > On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau > wrote: > > > The resulting HEAD is identical to yours, except for > > board/freescale/mx31ads/u-boot.lds in which you had removed the following

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-04 Thread Albert ARIBAUD
Hi Benoît, On Thu, 4 Apr 2013 16:05:27 +0200 (CEST), Benoît Thébaudeau wrote: > Hi Albert, > > On Thursday, April 4, 2013 11:07:23 AM, Albert ARIBAUD wrote: > > Hi Benoît, > > > > On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau > > wrote: > > > > > The resulting HEAD is identical

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-08 Thread Benoît Thébaudeau
Hi Fabio, On Wednesday, April 3, 2013 11:42:07 PM, Benoît Thébaudeau wrote: > Hi Fabio, > > On Wednesday, April 3, 2013 11:01:51 PM, Fabio Estevam wrote: > > Hi Benoît, > > > > On Wed, Apr 3, 2013 at 3:30 AM, Benoît Thébaudeau > > wrote: > > > > > Here is the v10 bundle for those who want to t

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-08 Thread Fabio Estevam
Hi Benoît, On Mon, Apr 8, 2013 at 3:47 PM, Benoît Thébaudeau wrote: > Did you have a chance to perform this test? I have built v10 bundle and will have access to my mx31pdk tomorrow morning. Will let you know the results. Regards, Fabio Estevam ___

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Fabio Estevam
Hi Benoît, On Mon, Apr 8, 2013 at 3:55 PM, Fabio Estevam wrote: > Hi Benoît, > > On Mon, Apr 8, 2013 at 3:47 PM, Benoît Thébaudeau > wrote: > >> Did you have a chance to perform this test? > > I have built v10 bundle and will have access to my mx31pdk tomorrow morning. > > Will let you know the

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Fabio Estevam
On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > Just tested v10 on mx31pdk. > > As reported in v9 I start to get reboots and never reach the prompt. > > Then I disabled watchdog options in mx31pdk.h and I get the following: > > U-Boot 2013.04-rc1-14135-gab21c72-dirty (Apr 09 2013 - 09:23:0

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
Hi Fabio, On Tuesday, April 9, 2013 2:46:21 PM, Fabio Estevam wrote: > On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > > > Just tested v10 on mx31pdk. > > > > As reported in v9 I start to get reboots and never reach the prompt. > > > > Then I disabled watchdog options in mx31pdk.h and I g

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
On Tuesday, April 9, 2013 4:32:21 PM, Benoît Thébaudeau wrote: > Hi Fabio, > > On Tuesday, April 9, 2013 2:46:21 PM, Fabio Estevam wrote: > > On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > > > > > Just tested v10 on mx31pdk. > > > > > > As reported in v9 I start to get reboots and never

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
Hi Fabio, On Tuesday, April 9, 2013 2:46:21 PM, Fabio Estevam wrote: > On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > > > Just tested v10 on mx31pdk. > > > > As reported in v9 I start to get reboots and never reach the prompt. > > > > Then I disabled watchdog options in mx31pdk.h and I g

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
On Tuesday, April 9, 2013 4:38:23 PM, Benoît Thébaudeau wrote: > Hi Fabio, > > On Tuesday, April 9, 2013 2:46:21 PM, Fabio Estevam wrote: > > On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > > > > > Just tested v10 on mx31pdk. > > > > > > As reported in v9 I start to get reboots and never

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
On Tuesday, April 9, 2013 5:04:45 PM, Benoît Thébaudeau wrote: > On Tuesday, April 9, 2013 4:38:23 PM, Benoît Thébaudeau wrote: > > Hi Fabio, > > > > On Tuesday, April 9, 2013 2:46:21 PM, Fabio Estevam wrote: > > > On Tue, Apr 9, 2013 at 9:30 AM, Fabio Estevam wrote: > > > > > > > Just tested v1

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Fabio Estevam
Hi Benoît, On Tue, Apr 9, 2013 at 11:38 AM, Benoît Thébaudeau wrote: > Can you try different values for the following configs in mx31pdk.h? > - CONFIG_SYS_NAND_U_BOOT_SIZE > - CONFIG_SPL_TEXT_BASE > - CONFIG_SYS_TEXT_BASE > - (CONFIG_ENV_OFFSET) > - (CONFIG_ENV_OFFSET_REDUND) > > I would tr

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
Hi Fabio, On Tuesday, April 9, 2013 6:40:45 PM, Fabio Estevam wrote: > Hi Benoît, > > On Tue, Apr 9, 2013 at 11:38 AM, Benoît Thébaudeau > wrote: > > > Can you try different values for the following configs in mx31pdk.h? > > - CONFIG_SYS_NAND_U_BOOT_SIZE > > - CONFIG_SPL_TEXT_BASE > > - CONF

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
Hi Albert, On Wednesday, April 3, 2013 10:05:57 AM, Albert ARIBAUD wrote: > Hi Benoît, > > On Wed, 3 Apr 2013 08:30:12 +0200 (CEST), Benoît Thébaudeau > wrote: > > > Hi Albert, > > > > Here is the v10 bundle for those who want to test: > > http://dl.free.fr/vdXBGExyq > > Thanks, will build-te

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
On Tuesday, April 9, 2013 6:56:18 PM, Benoît Thébaudeau wrote: > Hi Fabio, > > On Tuesday, April 9, 2013 6:40:45 PM, Fabio Estevam wrote: > > Hi Benoît, > > > > On Tue, Apr 9, 2013 at 11:38 AM, Benoît Thébaudeau > > wrote: > > > > > Can you try different values for the following configs in mx31

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Fabio Estevam
On Tue, Apr 9, 2013 at 2:37 PM, Benoît Thébaudeau wrote: > Especially, does it work with only: > -#define CONFIG_HW_WATCHDOG > -#define CONFIG_IMX_WATCHDOG > -#define CONFIG_SYS_NAND_U_BOOT_SIZE0x32000 > +#define CONFIG_SYS_NAND_U_BOOT_SIZE0x3f800 No, it hangs in "Net :". Also tried 0x40

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
Hi Fabio, On Tuesday, April 9, 2013 8:07:31 PM, Fabio Estevam wrote: > On Tue, Apr 9, 2013 at 2:37 PM, Benoît Thébaudeau > wrote: > > > Especially, does it work with only: > > -#define CONFIG_HW_WATCHDOG > > -#define CONFIG_IMX_WATCHDOG > > -#define CONFIG_SYS_NAND_U_BOOT_SIZE0x32000 > > +#d

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Fabio Estevam
On Tue, Apr 9, 2013 at 3:10 PM, Benoît Thébaudeau wrote: > These values were very wide, just for testing. It'd be better to adjust them. > So > can you just try the following, and I'll stop asking for more tests: > -#define CONFIG_SPL_TEXT_BASE 0x87ec > -#define CONFIG_SYS_TEXT_BASE 0x87

Re: [U-Boot] [PATCH v9 18/30] nand: mxc: Switch NAND SPL to generic SPL

2013-04-09 Thread Benoît Thébaudeau
On Tuesday, April 9, 2013 8:35:09 PM, Fabio Estevam wrote: > On Tue, Apr 9, 2013 at 3:10 PM, Benoît Thébaudeau > wrote: > > > These values were very wide, just for testing. It'd be better to adjust > > them. So > > can you just try the following, and I'll stop asking for more tests: > > -#define