Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-22 Thread Benoît Thébaudeau
On Friday, February 22, 2013 6:36:45 PM, Fabio Estevam wrote: > Hi Benoît, > > On Mon, Feb 18, 2013 at 4:30 PM, Benoît Thébaudeau > wrote: > > > So I will make a v8 with CONFIG_SPL_PAD_TO as an image offset, and use it > > to > > generate u-boot-with-spl.bin. But first, I will wait for more feed

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-22 Thread Fabio Estevam
Hi Benoît, On Mon, Feb 18, 2013 at 4:30 PM, Benoît Thébaudeau wrote: > So I will make a v8 with CONFIG_SPL_PAD_TO as an image offset, and use it to > generate u-boot-with-spl.bin. But first, I will wait for more feedback on v7 > (Fabio should give his test results this week), and for Stefano to

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 9:06:30 PM, Scott Wood wrote: > On 02/18/2013 02:01:59 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 8:47:28 PM, Scott Wood wrote: > > > On 02/18/2013 01:52:10 PM, Benoît Thébaudeau wrote: > > > > On Monday, February 18, 2013 8:37:29 PM, Scott Wood wrote

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 02:01:59 PM, Benoît Thébaudeau wrote: On Monday, February 18, 2013 8:47:28 PM, Scott Wood wrote: > On 02/18/2013 01:52:10 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 8:37:29 PM, Scott Wood wrote: > > > On 02/17/2013 10:16:49 AM, Benoît Thébaudeau wrote: > > > > all

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 8:47:28 PM, Scott Wood wrote: > On 02/18/2013 01:52:10 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 8:37:29 PM, Scott Wood wrote: > > > On 02/17/2013 10:16:49 AM, Benoît Thébaudeau wrote: > > > > Before this patch, PAD_TO was used, but there is no such

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 01:52:10 PM, Benoît Thébaudeau wrote: On Monday, February 18, 2013 8:37:29 PM, Scott Wood wrote: > On 02/17/2013 10:16:49 AM, Benoît Thébaudeau wrote: > > Before this patch, PAD_TO was used, but there is no such definition > > for this > > board for generic SPL, so this board see

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 8:37:29 PM, Scott Wood wrote: > On 02/17/2013 10:16:49 AM, Benoît Thébaudeau wrote: > > Hi Poonam, Andy, > > > > On Friday, February 15, 2013 9:54:19 PM, Benoît Thébaudeau wrote: > > > PAD_TO is not a generic SPL configuration option, so use > > CONFIG_SPL_MAX_SIZE > >

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/17/2013 10:16:49 AM, Benoît Thébaudeau wrote: Hi Poonam, Andy, On Friday, February 15, 2013 9:54:19 PM, Benoît Thébaudeau wrote: > PAD_TO is not a generic SPL configuration option, so use CONFIG_SPL_MAX_SIZE > instead. > > We want to use --pad-to with a size, but this option expects an

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
Hi Scott, On Monday, February 18, 2013 8:11:22 PM, Scott Wood wrote: > On 02/18/2013 12:52:51 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 7:24:51 PM, Scott Wood wrote: > > > On 02/18/2013 12:22:58 PM, Benoît Thébaudeau wrote: > > > > On Monday, February 18, 2013 7:02:49 PM, Scott

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 12:52:51 PM, Benoît Thébaudeau wrote: On Monday, February 18, 2013 7:24:51 PM, Scott Wood wrote: > On 02/18/2013 12:22:58 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 7:02:49 PM, Scott Wood wrote: > > > On 02/18/2013 12:00:52 PM, Benoît Thébaudeau wrote: > > > > The

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 7:24:51 PM, Scott Wood wrote: > On 02/18/2013 12:22:58 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 7:02:49 PM, Scott Wood wrote: > > > On 02/18/2013 12:00:52 PM, Benoît Thébaudeau wrote: > > > > The only question is if we may need to have an empty gap

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 12:22:58 PM, Benoît Thébaudeau wrote: On Monday, February 18, 2013 7:02:49 PM, Scott Wood wrote: > On 02/18/2013 12:00:52 PM, Benoît Thébaudeau wrote: > > The only question is if we may need to have an empty gap between the > > SPL and > > U-Boot within the resulting image. I don

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 7:02:49 PM, Scott Wood wrote: > On 02/18/2013 12:00:52 PM, Benoît Thébaudeau wrote: > > On Monday, February 18, 2013 6:27:50 PM, Tom Rini wrote: > > > > Commit 74752ba performs a '--pad-to=$(or $(CONFIG_SPL_PAD_TO),0)' > > > > on u-boot-spl. I could use this CONFIG_SPL_

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 12:00:52 PM, Benoît Thébaudeau wrote: On Monday, February 18, 2013 6:27:50 PM, Tom Rini wrote: > > Commit 74752ba performs a '--pad-to=$(or $(CONFIG_SPL_PAD_TO),0)' > > on u-boot-spl. I could use this CONFIG_SPL_PAD_TO for this series > > too, but is it really necessary to have both

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Scott Wood
On 02/18/2013 06:28:15 AM, Benoît Thébaudeau wrote: I'm also wondering why there is both generic SPL for NAND and legacy "NAND SPL" for p1_p2_rdb, all the more the "NAND SPL" version does not seem to be used in boards.cfg. "p1_p2_rdb_pc" and "P1_P2_RDB" are different targets (unfortunately)

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 6:27:50 PM, Tom Rini wrote: > > Commit 74752ba performs a '--pad-to=$(or $(CONFIG_SPL_PAD_TO),0)' > > on u-boot-spl. I could use this CONFIG_SPL_PAD_TO for this series > > too, but is it really necessary to have both CONFIG_SPL_PAD_TO and > > CONFIG_SPL_MAX_SIZE? In oth

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/18/2013 12:26 PM, Benoît Thébaudeau wrote: [snip] > But I've just seen that commit 74752ba did something for that in > u-boot/master, and this commit is not in u-boot-imx/master on which > I based this series. Why is u-boot-imx/master not sync'e

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Monday, February 18, 2013 5:50:59 PM, Tom Rini wrote: > On Sun, Feb 17, 2013 at 05:16:49PM +0100, Beno??t Th??baudeau wrote: > > > Hi Poonam, Andy, > > > > On Friday, February 15, 2013 9:54:19 PM, Beno??t Th??baudeau wrote: > > > PAD_TO is not a generic SPL configuration option, so use > > > C

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Tom Rini
On Sun, Feb 17, 2013 at 05:16:49PM +0100, Beno??t Th??baudeau wrote: > Hi Poonam, Andy, > > On Friday, February 15, 2013 9:54:19 PM, Beno??t Th??baudeau wrote: > > PAD_TO is not a generic SPL configuration option, so use CONFIG_SPL_MAX_SIZE > > instead. > > > > We want to use --pad-to with a siz

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-18 Thread Benoît Thébaudeau
On Sunday, February 17, 2013 5:16:49 PM, Benoît Thébaudeau wrote: > Hi Poonam, Andy, > > On Friday, February 15, 2013 9:54:19 PM, Benoît Thébaudeau wrote: > > PAD_TO is not a generic SPL configuration option, so use > > CONFIG_SPL_MAX_SIZE > > instead. > > > > We want to use --pad-to with a size,

Re: [U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-17 Thread Benoît Thébaudeau
Hi Poonam, Andy, On Friday, February 15, 2013 9:54:19 PM, Benoît Thébaudeau wrote: > PAD_TO is not a generic SPL configuration option, so use CONFIG_SPL_MAX_SIZE > instead. > > We want to use --pad-to with a size, but this option expects an address, so > use > u-boot-spl.bin instead of u-boot-spl

[U-Boot] [PATCH v7 13/19] Makefile: u-boot-with-spl.bin: Fix SPL padding

2013-02-15 Thread Benoît Thébaudeau
PAD_TO is not a generic SPL configuration option, so use CONFIG_SPL_MAX_SIZE instead. We want to use --pad-to with a size, but this option expects an address, so use u-boot-spl.bin instead of u-boot-spl as the input file in order to get addresses starting at 0. Signed-off-by: Benoît Thébaudeau -