All of lore.kernel.org
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.aribaud@free.fr>
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:27:41 +0100	[thread overview]
Message-ID: <4D5FD35D.3090609@free.fr> (raw)
In-Reply-To: <4D5FCE64.8030901@ahsoftware.de>

Le 19/02/2011 15:06, Alexander Holler a ?crit :
> 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.

Indeed.

>> 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.

Granted. But not all of them relate to hardware which may or may not be 
available.

>> 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. ;)

At least there is one reason for having -msoft-float even though no 
floating point should be used in U-Boot : it helps catching cases where 
some code in U-Boot tries to use hard float. :)

Think of it as choosing the least dangerous of Charybdis and Scylla, 
while trying our best not to get near any of the two.

> Regards,
>
> Alexander

Amicalement,
-- 
Albert.

  reply	other threads:[~2011-02-19 14:27 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
2011-02-19 14:27             ` Albert ARIBAUD [this message]
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=4D5FD35D.3090609@free.fr \
    --to=albert.aribaud@free.fr \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.