public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V5] libgcc inclusion from common Makefile overwritable from platform configs files
Date: Wed, 8 Jul 2009 23:19:02 +0200	[thread overview]
Message-ID: <20090708211902.GH12394@game.jcrosoft.org> (raw)
In-Reply-To: <20090708205535.C0CF2832E416@gemini.denx.de>

On 22:55 Wed 08 Jul     , Wolfgang Denk wrote:
> Dear Jean-Christophe PLAGNIOL-VILLARD,
> 
> In message <1247085496-21754-1-git-send-email-plagnioj@jcrosoft.com> you wrote:
> > This patch allow to overwrit the libgcc Makefile inclusion from the toplevel
> 
> Please mind the hints: "overwrit" is a typo. You  meant  "overwrite",
> and  Mike  just  explained  that  "override" is even more appropriate
> here.
> 
> > Makefile by the arch_config.mk files. This is in preparation for the ARM
> > architecture to move away from including libgcc function and only using
> > self-contained U-Boot functions as done in Linux.
> > 
> > Currently all the ARM boards that use the nand are broken due to the adding of
> > the 64 Bit device size support. In the past we have seen problems with different
> > toolchains due to EABI, FPU as example.
> 
> Correct spelling is "tool chains", not "toolchains".
> 
> > With this patch and the following one we move away from all these problems and
> > we will be able to have full control to have a functions embedded into u-boot.
> 
> Please fix the comment.
> 
> In git history, there will be no "following one" patch, and here you
> did not post one either. If you don't have it ready yet, then please
> use a description of what it will do.
the precedent patch still apply cleany so I do not resend it
> 
> And "we will be able  to  have  full  control  to  have  a  functions
> embedded  into  u-boot"  makes  no  sense.  Either it is "a function"
> (singular), or it is "functions" (plural, without "a"). But  what  do
> you  want  to  tell  us? "have functions embedded into U-Boot"? We do
> this all the time by linking them in - we already have  full  control
> over  it  (except  that  it's not working as expected sometimes), and
> your patch does not really change anything of this.
gcc for arm is really boring about abi support and float support
so on arm we do need to control it and be sure about what we use for basic
function and do not use the libgcc provide by the tool chains as we can not
trust them
so for linux as example we implement it correctly and are able to manage
against what armv we want to compile or be comptatible
ditto when we will be able to support Thumb2 as needed for cortex-m3
> 
> 
> You wrote: "gcc does not provide necessarely cored functions". Sorry,
> but I have no idea what you are talking about here. What is a "cored
> function"?
correct function
> 
> Best regards,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> When some people discover the truth, they just can't  understand  why
> everybody isn't eager to hear it.

  reply	other threads:[~2009-07-08 21:19 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 [this message]
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
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=20090708211902.GH12394@game.jcrosoft.org \
    --to=plagnioj@jcrosoft.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