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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 15:53 Mon 06 May     , Thomas Petazzoni wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> On Sun, 5 May 2013 13:19:27 +0200, Jean-Christophe PLAGNIOL-VILLARD
> wrote:
> 
> > > > > 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?
> > to geneate a simple header do it in C not with a tools
> 
> Then it seems like you haven't read the code at all. The 'simple
> header' has the following properties:
> 
>  * Is of variable length: it contains a main header, and an optional
>    arbitrary number of binary headers, security headers and register
>    headers.
> 
>  * A 8-bit checksum that covers the entire size of headers. This cannot
>    be easily computed directly by compiling C code and using linker
>    scrips.
> 
>  * A 32-bit checksum that covers the binary payload.
> 
>  * The contents of the binary header are extracted from existing boot
>    images for Armada 370/XP platforms because this binary header
>    contains some DDR3 training code that I haven't worked on for now.
>    And since this binary header is cleanly isolated from the bootloader
>    itself, it is easier to re-use such binary code for now.
> 
> The tool also allows to extract binary images, which is needed to
> extract the binary header mentioned above, and re-use when building a
> bootable image that includes Barebox.
> 
> > read the imx code we do it in C and place it at the begenning of the binary so
> > no need such tools and we can use linker scirpt and CONFIG_xx
> > 
> > Stop to think u-boot here
> 
> Could you please adopt a nicer language? You are very aggressive... and
> at that the same time completely wrong. Your comments make it entirely
> clear that you haven't even read the comments at the top of file.
I did but still think we can handle it in C

and need to handle by barebox itself when flashing
> 
> Thomas
> -- 
> Thomas Petazzoni, Free Electrons
> Kernel, drivers, real-time and embedded Linux
> development, consulting, training and support.
> http://free-electrons.com

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




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux