From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail.free-electrons.com ([94.23.35.102]) by merlin.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1UZM3s-000552-A8 for barebox@lists.infradead.org; Mon, 06 May 2013 14:06:40 +0000 Date: Mon, 6 May 2013 16:06:14 +0200 From: Thomas Petazzoni Message-ID: <20130506160614.054b5e1a@skate> In-Reply-To: <20130506135435.GL13393@game.jcrosoft.org> References: <1367599871-28479-1-git-send-email-thomas.petazzoni@free-electrons.com> <1367599871-28479-3-git-send-email-thomas.petazzoni@free-electrons.com> <20130504195125.GJ13393@game.jcrosoft.org> <20130504203213.GS31290@titan.lakedaemon.net> <20130505111927.GK13393@game.jcrosoft.org> <20130506155352.15f23d3a@skate> <20130506135435.GL13393@game.jcrosoft.org> Mime-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "barebox" Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 2/7] scripts: new kwbimage manipulation tool for Marvell SoC boot images To: Jean-Christophe PLAGNIOL-VILLARD Cc: Lior Amsalem , barebox@lists.infradead.org, Jason Cooper , Ezequiel Garcia , Willy Tarreau Dear Jean-Christophe PLAGNIOL-VILLARD, On Mon, 6 May 2013 15:54:35 +0200, Jean-Christophe PLAGNIOL-VILLARD wrote: > > 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 The image is pushed to the target through X-modem, directly talking to the ROM code of the SoC. Do you want the kwboot tool to prepare the image? It seems to me like it makes a lot more sense to have a tool to prepare the image. Remember also that this tool is needed to *extract* existing images. And you haven't explained how you intend to do this without a proper userspace tool such as kwbimage. And by the way, the tool is a simple userspace tool written entirely in C, I'm not sure to see where the problem is. Quoting Sascha, an e-mail for this thread: """ 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. """ Thanks, 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@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox