Re: [PATCH 2/7] scripts: new kwbimage manipulation tool for Marvell SoC boot images

[Sascha Hauer <s.hauer@pengutronix.de>]

On Sat, May 04, 2013 at 04:32:13PM -0400, Jason Cooper wrote:
> On Sat, May 04, 2013 at 09:51:25PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 18:51 Fri 03 May     , Thomas Petazzoni wrote:
> > > The Marvell EBU SoCs (Kirkwood, Armada 370, Armada XP) have a BootROM
> > > that understand a specific image format, composed of a main header,
> > > several extension headers and a paylod. This image can be booted from
> > > NAND, SPI, SATA, UART, NOR, etc.
> > > 
> > > This patch adds a tool that allows to extract the components and
> > > configuration of existing images, and to create new images.
> > 
> > I don't like this
> 
> Why?
> 
> > please as on imx in barebox directly
> 
> what?  Could you please explain?

I can only guess that he means that he doesn't want to have a separate
tool for generating images but instead wants to generate the images
using C structures in the original binary like we do on i.MX currently.
Anyway, what we do on i.MX doesn't really scale anymore since the more
complicated features of the image format can't be used with C
structures. I'm working on replacing this with a imx-image tool.
You seem to use checksums in your images. These can't be generated
in C or CPP anyway.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox