From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Mon, 22 Jun 2009 14:01:34 +0200
Subject: [U-Boot] [PATCH] mtd: Update MTD infrastructure to
	support	64bit device size
In-Reply-To: <4A3D2044.6090106@googlemail.com>
References: <1242050635-7152-1-git-send-email-sr@denx.de>
	<20090620085207.GC6870@game.jcrosoft.org>
	<4A3D2044.6090106@googlemail.com>
Message-ID: <200906221401.35203.sr@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Dirk,

On Saturday 20 June 2009 19:45:40 Dirk Behme wrote:
> Jean-Christophe PLAGNIOL-VILLARD wrote:
> >> 2. C++ style comment. Found by Wolfgang ;)

This is because it's a copy from the Linux code. Not sure if
we should change the coding style here. I really prefer to be
in sync with the Linux version. This makes future updates easier.

> >> 3. The major one: By the new 64bit variables, depending on tool chain,
> >> there are now calls to libgcc introduced. Depending on tool chain,
> >> this might work, or fail. As it does with one of my tool chains, which
> >> worked totally fine until here. It was my understanding that U-Boot
> >> shall not rely on libgcc, i.e. proper tool chain libraries.
> >>
> >> Having CONFIG_SYS_64BIT_VSPRINTF not defined, the linker needs
> >> "_lshrdi3", which comes from nand_base.o and nand_bbt.o. E.g. from
> >> "len = mtd->size >> (this->bbt_erase_shift + 2);" from nand_bbt.c.
> >>
> >> Having CONFIG_SYS_64BIT_VSPRINTF enabled, the linker requests
> >> _lshrdi3, _udivdi3, _umoddi3 and _clz from libgcc.
> >>
> >> Looking into U-Boot's lib_arm/ directory, we have already some low
> >> level math functions there to avoid libgcc. E.g. nand_bbt.c has
> >> references to _ashrdi3, too, but this is resolved by lib_arm's version
> >> what is fine.
> >>
> >> I'd like that NAND code is modified that only math is used which
> >> U-Boot provides and no libgcc is needed.
> >
> > this is a already known problem by Stefan and I and solved
> >
> > two patchs was send to the ML
>
> Thanks for the hint! Do you kindly have the subjects or better links
> to mailing list archive?

http://git.denx.de/?p=u-boot/u-boot-arm.git;a=commit;h=cf96e690cb9840b4a5dc7e750be863228550b5ce

and 

http://git.denx.de/?p=u-boot/u-boot-arm.git;a=commit;h=c5ae538c9bd359c137d306e431b2589ba91fbc08

both available in the "testing" branch of u-boot-arm.

>From you comments on the list I see that you found those patches here
too. :)

Do these patches fix the building problem for you?

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================