public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Dirk Behme <dirk.behme@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/2 v6] Make libgcc inclusion from common	Makefile overridable by platform config file
Date: Sun, 12 Jul 2009 16:55:22 +0200	[thread overview]
Message-ID: <4A59F95A.6060803@googlemail.com> (raw)
In-Reply-To: <20090712143607.8A901832E416@gemini.denx.de>

Dear Wolfgang,

Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <20090712120655.GA21713@game.jcrosoft.org> you wrote:
>>> It will hush up the current errors, but that's actually a  bad  thing
>>> here  -  the  errors  are  an indication that Jean-Christophe's patch
>>> might not be working as it is supposed to.
>> They do fix what they are suppose to , fix FPU and EABI problem which was
>> re-introduce by the 64 bit mtd support
>> here the problem is different you try to div64 which is not supported on arm
>> you do need to do_div
> 
> What do you mean - not supported by ARM?  Of course ARM supports 64
> bit division.
> 
> Compiling this little test code:
> 
> 	long long div(long long x, long long y)
> 	{
> 	        return x / y;
> 	}
> 
> will result in a call to  __aeabi_ldivmod  using  an  EABI  compliant
> version  of  GCC,  resp.  to __divdi3 using an older compiler. So GCC
> knows how to handle this, and it provides adequate  functions  to  do
> it.
> 
>> please apply this patch so I'll be able to send a pull request with the arm
>> specific part and other patch be go in vacancy for one week this night
> 
> I really hesitate to do that. It seems that not  using  the  compiler
> provided library is not such a clever thing to do. The compile writes
> probably  know  better  what  a  specific  version  of GCC needs that
> anybody else.

Yes, you are basically right. But ;)

But, as Jean-Christophe mentioned above, it's a pain with the various 
ARM tool chains floating around. Some are older, some are newer, some 
are configured for EABI, some not, some are configured for software 
floating point, some for hardware floating point, etc., etc.

While I as developer might be able to find a recent tool chain with a 
libgcc compatible with U-Boot, I think we should avoid this pain for 
our users. Users which like to "just compile U-Boot" and then we tell 
them "well, your tool chain you seem to be happy with doesn't link 
U-Boot, for U-Boot you have to install an other one" I think wouldn't 
make them happy.

Regarding not using the compilers library and if this clever: No, it 
isn't clever, you are right again. The compiler's library version is 
most probably better optimized. But, we are dealing with a boot loader 
here. So for the topic we discuss here, I think avoiding some pain for 
us ("my tool chain doesn't compile U-Boot, help!" mails at this list) 
and our users (see above) is the stronger argument than some 
optimization/performance issues in some (seldom?) used math functions.

Best regards

Dirk

  reply	other threads:[~2009-07-12 14:55 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-08 19:36 [U-Boot] [PATCH v3] libgcc inclusion from common Makefile overwritable from platform configs files Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 19:42 ` Scott Wood
2009-07-08 20:09   ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 20:14 ` [U-Boot] [PATCH V4] " Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 20:26   ` Scott Wood
2009-07-08 20:33     ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 20:38     ` [U-Boot] [PATCH V5] " Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 20:38       ` [U-Boot] [PATCH 2/2] netstar/voiceblue: remove no-need libgcc link for eeprom standalone Jean-Christophe PLAGNIOL-VILLARD
2009-07-20 22:03         ` Wolfgang Denk
2009-07-08 20:55       ` [U-Boot] [PATCH V5] libgcc inclusion from common Makefile overwritable from platform configs files Wolfgang Denk
2009-07-08 21:19         ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 21:29           ` Wolfgang Denk
2009-07-08 20:30   ` [U-Boot] [PATCH V4] " Wolfgang Denk
2009-07-08 20:45     ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-08 20:47     ` Mike Frysinger
2009-07-09 10:24 ` [U-Boot] [PATCH 1/2 v6] Make libgcc inclusion from common Makefile overridable by platform config file Jean-Christophe PLAGNIOL-VILLARD
2009-07-12  7:54   ` Dirk Behme
2009-07-12  8:02     ` Stefan Roese
2009-07-12  8:15       ` Dirk Behme
2009-07-12 10:29       ` Wolfgang Denk
2009-07-12 12:06         ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-12 12:13           ` Dirk Behme
2009-07-12 12:39             ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-12 14:36           ` Wolfgang Denk
2009-07-12 14:55             ` Dirk Behme [this message]
2009-07-12 15:50               ` Wolfgang Denk
2009-07-12 16:12                 ` Dirk Behme
2009-07-12 18:17                   ` Wolfgang Denk
2009-07-12 19:22                     ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-12 19:35                       ` Wolfgang Denk
2009-07-12 19:51                         ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-12 21:27                           ` Wolfgang Denk
2009-07-12 16:17                 ` Jean-Christophe PLAGNIOL-VILLARD
2009-07-12 18:29                   ` Wolfgang Denk
2009-07-12 19:06                     ` Dirk Behme
2009-07-12 19:30                       ` Wolfgang Denk
2009-07-13  9:25                     ` Mike Frysinger
2009-07-13 16:00                       ` Dirk Behme
2009-07-15 22:18                       ` Scott Wood
2009-07-15 22:43                         ` Mike Frysinger
2009-07-15 23:03                           ` Scott Wood
2009-07-15 23:54                             ` Mike Frysinger
2009-07-16 15:36                               ` Scott Wood
2009-07-16 15:42                                 ` Wolfgang Denk
2009-07-16 15:56                                   ` Scott Wood
2009-07-17 11:27                                     ` Detlev Zundel
2009-07-17 11:37                                       ` Wolfgang Denk
2009-07-17 11:41                                         ` Wolfgang Denk
2009-07-17 15:24                                         ` Scott Wood
2009-07-16 11:11                         ` Wolfgang Denk
2009-07-13  7:36         ` Stefan Roese
2009-07-13 15:46           ` Dirk Behme
2009-07-13 18:16           ` Mike Frysinger
2009-07-23  9:36   ` Wolfgang Denk
2009-07-23 11:09     ` [U-Boot] [PATCH] Make linking against libgcc configurable Wolfgang Denk
2009-07-23 11:15       ` [U-Boot] [PATCH v2] " Wolfgang Denk
2009-07-23 11:27         ` [U-Boot] [PATCH] arm: add _lshrdi3.S Heiko Schocher
2009-07-23 11:41           ` Wolfgang Denk
2009-07-23 12:16             ` Heiko Schocher
2009-07-26 22:11           ` Wolfgang Denk
2009-07-23 13:28         ` [U-Boot] [PATCH v2] Make linking against libgcc configurable Daniel Gorsulowski
2009-07-23 14:12           ` Heiko Schocher
2009-07-23 14:43             ` Daniel Gorsulowski
2009-07-23 14:48               ` Daniel Gorsulowski
2009-07-23 15:33                 ` Heiko Schocher
2009-07-24  6:07                   ` Daniel Gorsulowski
2009-07-27  6:26                     ` Heiko Schocher
2009-07-23 16:45           ` Wolfgang Denk
2009-07-26 22:11         ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4A59F95A.6060803@googlemail.com \
    --to=dirk.behme@googlemail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox