Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Albert ARIBAUD
Hello Stefan, On Wed, 15 Jul 2015 14:36:16 +0200, Stefan Agner wrote: > >> As far as I see we have these two patches which fix Vybrid booting: > >> > >> - one which changes the boot_data_t /size/ which keeps the offset > >> of dcd_v2_t and the image aligned. > >> > >> - one which changes the d

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Stefan Agner
On 2015-07-15 13:44, Albert ARIBAUD wrote: > Hello Stefan (and sorry for the duplicate), > > On Wed, 15 Jul 2015 12:41:59 +0200, Stefan Agner > wrote: >> Hi Albert, >> >> On 2015-07-15 09:54, Albert ARIBAUD wrote: >> > Hello Stefano, >> > >> > On Wed, 15 Jul 2015 09:19:55 +0200, Stefano Babic >>

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Albert ARIBAUD
Hello Stefan (and sorry for the duplicate), On Wed, 15 Jul 2015 12:41:59 +0200, Stefan Agner wrote: > Hi Albert, > > On 2015-07-15 09:54, Albert ARIBAUD wrote: > > Hello Stefano, > > > > On Wed, 15 Jul 2015 09:19:55 +0200, Stefano Babic > > wrote: > > > >> > I think my patch back then solves

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Stefan Agner
Hi Albert, On 2015-07-15 09:54, Albert ARIBAUD wrote: > Hello Stefano, > > On Wed, 15 Jul 2015 09:19:55 +0200, Stefano Babic > wrote: > >> > I think my patch back then solves the issue nicer. The struct dcd_v2_t >> > is not the reason the whole header is not aligned by 8-byte, it is a >> > prob

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Albert ARIBAUD
Hello Stefano, On Wed, 15 Jul 2015 09:19:55 +0200, Stefano Babic wrote: > > I think my patch back then solves the issue nicer. The struct dcd_v2_t > > is not the reason the whole header is not aligned by 8-byte, it is a > > problem of the boot_data_t header which is 12 bytes long. So inserting >

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Albert ARIBAUD
Hello Stefan, On Tue, 14 Jul 2015 12:29:52 +0200, Stefan Agner wrote: > Hi Stefano, > > On 2015-07-10 10:14, Stefano Babic wrote: > > On 19/06/2015 14:18, Albert ARIBAUD (3ADEV) wrote: > >> imximage header size is 4-byte, not 8-byte aligned. > >> This produces .imx images that a Vybrid cannot bo

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-15 Thread Stefano Babic
Hi Stefan, On 14/07/2015 12:29, Stefan Agner wrote: >> >> Applied to u-boot-imx, thanks ! > > Sorry, just stumbled over this message now. > > We discussed exactly this issue already more than a year ago, see: > http://lists.denx.de/pipermail/u-boot/2014-April/177580.html I admit that I have fo

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-14 Thread Stefan Agner
Hi Stefano, On 2015-07-10 10:14, Stefano Babic wrote: > On 19/06/2015 14:18, Albert ARIBAUD (3ADEV) wrote: >> imximage header size is 4-byte, not 8-byte aligned. >> This produces .imx images that a Vybrid cannot boot >> on. >> >> Fix by adding a "padding" field in header. >> >> Signed-off-by: Albe

Re: [U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-07-10 Thread Stefano Babic
On 19/06/2015 14:18, Albert ARIBAUD (3ADEV) wrote: > imximage header size is 4-byte, not 8-byte aligned. > This produces .imx images that a Vybrid cannot boot > on. > > Fix by adding a "padding" field in header. > > Signed-off-by: Albert ARIBAUD (3ADEV) > --- > > tools/imximage.h | 1 + > 1 fi

[U-Boot] [PATCH 4/5] tools: mkimage: fix imximage header size

2015-06-19 Thread Albert ARIBAUD (3ADEV)
imximage header size is 4-byte, not 8-byte aligned. This produces .imx images that a Vybrid cannot boot on. Fix by adding a "padding" field in header. Signed-off-by: Albert ARIBAUD (3ADEV) --- tools/imximage.h | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/imximage.h b/tools/imximag