From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1VPT48-0001vY-Cx for barebox@lists.infradead.org; Fri, 27 Sep 2013 08:06:21 +0000 Date: Fri, 27 Sep 2013 10:05:58 +0200 From: Sascha Hauer Message-ID: <20130927080557.GK30088@pengutronix.de> References: <1380227429.10759.1.camel@lolumad> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1380227429.10759.1.camel@lolumad> 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: Wrong barebox_image_size value To: Rostislav Lisovy Cc: barebox@lists.infradead.org Hi Rostislav, On Thu, Sep 26, 2013 at 10:30:29PM +0200, Rostislav Lisovy wrote: > Dear barebox developers and users; > I am compiling barebox for freescale-imx53-smd board. My issue is, > that the image does not contain the correct size record. > The "bootloader size" needed by the flashing tool is initialized in > freescale-mx53-smd/flash_header.c, in structure flash_header, > field .boot_data.size. > Formerly there used to be a hardcoded value 0x40000, nowadays we use > DCD_BAREBOX_SIZE which expands into (barebox_image_size + 4095), where > "barebox_image_size" has real size computed by the linker. > When the value .boot_data.size is filled by hand (as 0x40000), correct > value is stored in the barebox image. > Proof: hexdump -s 0x24 -n 4 barebox-flash-image > 0000024 0000 0004 > > however if .boot_data.size = DCD_BAREBOX_SIZE, > $ hexdump -s 0x24 -n 4 barebox-flash-image > 0000024 0fff 0000 > where this 0xfff is the "+4095" added to round the value. (I am pretty > sure the offset into the image used by hexdump is correct). Probably you are not building the defconfig, right? Compiling the defconfig gives me the correct value and it's at offset 0x424 in the image, not at 0x24. Maybe you are compiling a relocatable image as Jean-Christophe mentioned, then you would see this problem. > > The barebox version I am using most recent git, branch master. > The compiler (maybe here is the point of failure) is > arm-cortex_a8-linux-gnueabi-gcc (crosstool-NG hg+default-86a8d1d467c8) > 4.4.6. That should be ok. 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