public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Alexander Holler <holler@ahsoftware.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard	float	build	issues
Date: Sat, 19 Feb 2011 15:06:28 +0100	[thread overview]
Message-ID: <4D5FCE64.8030901@ahsoftware.de> (raw)
In-Reply-To: <4D5FCACA.1060606@free.fr>

Am 19.02.2011 14:51, schrieb Albert ARIBAUD:
> Le 19/02/2011 14:25, M?ns Rullg?rd a ?crit :
>
>>> So whats the reasoning to use -msoft-float as it is currently done? To
>>> confuse people? ;)
>>
>> I guess it's there to make sure no floating-point instructions make it
>> into u-boot, even though floats are not used in the code.  Perhaps
>> someone was paranoid.
>
> I think also that there is no choice but to have a float option for ARM
> C, either 'soft' or 'hard' (or 'softfp', actually, which is 'hard' with
> the 'soft' calling conventions), because the C compiler does not allow
> 'no floats', and anyway the C language *requires* to have some sort of
> float support.

As it is on every platform.

> Actually if you don't specify any float option, the C toolchain will
> choose one, which is just the same in the end: *some* float option is
> chosen.

Every compiler has dozens of options which are set to a default value.

> And since some option must be chosen, I prefer that U-Boot make the
> explicit decision, and choose soft float for the reasons I already exposed.

I don't see any reason to explicitly set an option for something which 
isn't used, but I know of many reasons to avoid such. ;)

Regards,

Alexander

  reply	other threads:[~2011-02-19 14:06 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-18 15:16 [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues Raghuveer Murthy
2011-02-18 17:01 ` Albert ARIBAUD
2011-02-18 21:11   ` Alexander Holler
2011-02-19 11:19   ` Måns Rullgård
2011-02-19 13:19     ` Alexander Holler
2011-02-19 13:25       ` Måns Rullgård
2011-02-19 13:51         ` Albert ARIBAUD
2011-02-19 14:06           ` Alexander Holler [this message]
2011-02-19 14:27             ` Albert ARIBAUD
2011-02-19 14:34               ` Måns Rullgård
2011-02-19 15:18                 ` Albert ARIBAUD
2011-02-19 18:36                   ` Måns Rullgård
2011-02-19 19:37                 ` Wolfgang Denk
2011-02-19 14:39               ` Alexander Holler
     [not found]                 ` <4D5FE29A.6050609@free.fr>
2011-02-19 15:40                   ` Alexander Holler
2011-02-19 15:59                     ` [U-Boot] reasons for armv5 as default Alexander Holler
2011-02-19 17:48                       ` Albert ARIBAUD
2011-02-20  2:20                         ` Alexander Holler
2011-02-19 19:46                       ` Wolfgang Denk
2011-02-19 19:39                     ` [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues Wolfgang Denk
2011-02-19 19:33             ` Wolfgang Denk
2011-02-19 19:32           ` Wolfgang Denk
2011-02-19 19:26         ` Wolfgang Denk
2011-02-18 22:41 ` Wolfgang Denk
2011-02-19 19:55 ` Wolfgang Denk
2011-02-21 10:36   ` Raghuveer Murthy

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=4D5FCE64.8030901@ahsoftware.de \
    --to=holler@ahsoftware.de \
    --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