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 1UaCaY-0003Hf-NH for barebox@lists.infradead.org; Wed, 08 May 2013 22:11:55 +0000 Message-ID: <518ACD8B.30009@free-electrons.com> Date: Thu, 09 May 2013 00:11:23 +0200 From: Gregory CLEMENT MIME-Version: 1.0 References: <1368047422-3397-1-git-send-email-thomas.petazzoni@free-electrons.com> <518AC964.50304@free-electrons.com> <20130508235926.39101806@skate> In-Reply-To: <20130508235926.39101806@skate> 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 v2 0/7] Basic support for Armada 370/XP SOCs To: Thomas Petazzoni Cc: Lior Amsalem , barebox@lists.infradead.org, Willy Tarreau , Ezequiel Garcia On 05/08/2013 11:59 PM, Thomas Petazzoni wrote: > Dear Gregory CLEMENT, > > On Wed, 08 May 2013 23:53:40 +0200, Gregory CLEMENT wrote: > >>> - Add an integration of kwbimage to the Barebox build process. Now, >>> when an ARCH_MVEBU platform is selected, a barebox.kwb file is >>> automatically generated using the board kwbimage.cfg, and a >>> barebox.kwbuart is generated using the board kwbimage.cfg + an >>> override of the boot media to be UART. >> >> It is a very nice improvement however, you should maybe mention somwhere >> that you expect to find a file named binary.0 at the root of the barebox >> sources. But it is not very important. > > Indeed, but which "somewhere" do you suggest? My first thought was in the help of the menuconfig, but in the case of someone just do a make board_configs, make, he won't see it. So maybe the best option should just to make the error message more verbose. Instead of "Cannot stat binary file binary.0" some thing like "Didn't find the file binary.0 at /absolute/path/to/barebox/source which is mandatory to generate the kwboot image" > >> However you didn't manage the out of tree building when KBUILD_OUTPUT is >> set. This is more annoying but it should be easily fixed. > > Aah, I haven't thought of out-of-tree build. I'll test that and I'll > have a look. > >> It didn't work for me when I tested the on the OpenBlocks AX3. The boot >> media didn't seem to have been overridden ie it still try to boot on the >> NOR. And if I modify the file arch/arm/boards/plathome-openblocks-ax3//kwbimage.cfg >> and replace spi by uart, then it works. > > Are you sure you used barebox.kwbuart and not barebox.kwb? Oh I didn't notice this one! Indeed if I use barebox.kwbuart it works. Sorry for the noise. Gregory _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox