Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-08-13 Thread Tom Rini
On Sat, Jul 20, 2013 at 07:36:12PM -0400, Tom Rini wrote: > On Sat, Jul 20, 2013 at 05:29:19PM -0600, Simon Glass wrote: > > +Stephen > > > > Hi Tom, > > > > On Sat, Jul 20, 2013 at 4:38 PM, Tom Rini wrote: > > > > > On Sat, Jul 20, 2013 at 04:06:27PM -0600, Simon Glass wrote: > > > > Hi, > > >

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-23 Thread Tom Rini
On Sat, Jul 20, 2013 at 04:06:27PM -0600, Simon Glass wrote: > Hi, > > On Sat, Jul 13, 2013 at 8:55 PM, Tom Rini wrote: [snip] > > No, because what we have today is insufficient for the kernel, you > > still have to specify the load/entry point, in FIT at least, even on > > NOLOAD. I'd have swor

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-23 Thread Simon Glass
+Stephen Hi Tom, On Sat, Jul 20, 2013 at 4:38 PM, Tom Rini wrote: > On Sat, Jul 20, 2013 at 04:06:27PM -0600, Simon Glass wrote: > > Hi, > > > > On Sat, Jul 13, 2013 at 8:55 PM, Tom Rini wrote: > [snip] > > > No, because what we have today is insufficient for the kernel, you > > > still have t

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-20 Thread Tom Rini
On Sat, Jul 20, 2013 at 05:29:19PM -0600, Simon Glass wrote: > +Stephen > > Hi Tom, > > On Sat, Jul 20, 2013 at 4:38 PM, Tom Rini wrote: > > > On Sat, Jul 20, 2013 at 04:06:27PM -0600, Simon Glass wrote: > > > Hi, > > > > > > On Sat, Jul 13, 2013 at 8:55 PM, Tom Rini wrote: > > [snip] > > > >

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-20 Thread Simon Glass
Hi, On Sat, Jul 13, 2013 at 8:55 PM, Tom Rini wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 07/13/2013 10:26 PM, Stephen Warren wrote: > > On 07/13/2013 05:21 AM, Wolfgang Denk wrote: > >> Dear Tom Rini, > >> > >> In message <20130712212416.GV13531@bill-the-cat> you wrote: > >>

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/13/2013 10:26 PM, Stephen Warren wrote: > On 07/13/2013 05:21 AM, Wolfgang Denk wrote: >> Dear Tom Rini, >> >> In message <20130712212416.GV13531@bill-the-cat> you wrote: >>> >>> A FIT image with a ramdisk that sets the entry or load points >>>

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Stephen Warren
On 07/13/2013 05:21 AM, Wolfgang Denk wrote: > Dear Tom Rini, > > In message <20130712212416.GV13531@bill-the-cat> you wrote: >> >> A FIT image with a ramdisk that sets the entry or load points to 0x0 >> must be treated as meaning "leave in place" and NOT "relocate to 0x0". > > Why is this the ca

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/13/2013 07:21 AM, Wolfgang Denk wrote: > Dear Tom Rini, > > In message <20130712212416.GV13531@bill-the-cat> you wrote: >> >> A FIT image with a ramdisk that sets the entry or load points to >> 0x0 must be treated as meaning "leave in place" an

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Wolfgang Denk
Dear Tom Rini, In message <20130712212416.GV13531@bill-the-cat> you wrote: > > A FIT image with a ramdisk that sets the entry or load points to 0x0 > must be treated as meaning "leave in place" and NOT "relocate to 0x0". Why is this the case? 0x0 could be a valid address on some systems. If we

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Stefano Babic
Am 12/07/2013 23:24, schrieb Tom Rini: > On Fri, Jul 12, 2013 at 03:09:23PM +0200, Stefano Babic wrote: > >> Booting a FIT image containing a ramdisk, >> the ramdisk is loaded at address 0x0 that causes >> bus errors for architectures that do not have >> RAM starting at address zero. >> >> Signed-

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-13 Thread Albert ARIBAUD
Hi Tom, > I've re-worded this as: > common/image.c: Fix regression with ramdisk load/entry points in FIT > > A FIT image with a ramdisk that sets the entry or load points to 0x0 > must be treated as meaning "leave in place" and NOT "relocate to 0x0". > This regression was introduced in a51ec6

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-12 Thread Tom Rini
On Fri, Jul 12, 2013 at 03:09:23PM +0200, Stefano Babic wrote: > Booting a FIT image containing a ramdisk, > the ramdisk is loaded at address 0x0 that causes > bus errors for architectures that do not have > RAM starting at address zero. > > Signed-off-by: Stefano Babic I've re-worded this as:

Re: [U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-12 Thread Albert ARIBAUD
Hi Stefano, On Fri, 12 Jul 2013 15:09:23 +0200, Stefano Babic wrote: > Booting a FIT image containing a ramdisk, > the ramdisk is loaded at address 0x0 that causes > bus errors for architectures that do not have > RAM starting at address zero. Kind-of-minor nitpick: ARMs which have RAM at addre

[U-Boot] [PATCH] tools: fix FIT image with ramdisk

2013-07-12 Thread Stefano Babic
Booting a FIT image containing a ramdisk, the ramdisk is loaded at address 0x0 that causes bus errors for architectures that do not have RAM starting at address zero. Signed-off-by: Stefano Babic --- common/image.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/im