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 ste...@agner.ch 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

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 sba...@denx.de 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

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 forgotten

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 sba...@denx.de 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

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 ste...@agner.ch 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 sba...@denx.de wrote: I think my patch back

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 ste...@agner.ch 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, On Wed, 15 Jul 2015 14:36:16 +0200, Stefan Agner ste...@agner.ch 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

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: Albert ARIBAUD

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) albert.arib...@3adev.fr --- tools/imximage.h |

[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) albert.arib...@3adev.fr --- tools/imximage.h | 1 + 1 file changed, 1 insertion(+) diff --git