Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-21 Thread Raghuveer Murthy
On Sunday 20 February 2011 01:25 AM, Wolfgang Denk wrote: Dear Raghuveer Murthy, In message1298042212-12260-1-git-send-email-raghuveer.mur...@ti.com you wrote: U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id:

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Måns Rullgård
Albert ARIBAUD albert.arib...@free.fr writes: Le 18/02/2011 16:16, Raghuveer Murthy a écrit : U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id: http://bugs.meego.com/show_bug.cgi?id=13140 Is this the official

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Alexander Holler
Hello, Am 19.02.2011 12:19, schrieb Måns Rullgård: In no case, however, should -msoft-float be *removed*: that would make float support in U-Boot implicit and dependent on the toolchain in use, and I think U-Boot float support should always be explicit and not depend on the toolchain.

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Måns Rullgård
Alexander Holler hol...@ahsoftware.de writes: Hello, Am 19.02.2011 12:19, schrieb Måns Rullgård: In no case, however, should -msoft-float be *removed*: that would make float support in U-Boot implicit and dependent on the toolchain in use, and I think U-Boot float support should always be

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread 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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Alexander Holler
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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Albert ARIBAUD
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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Måns Rullgård
Albert ARIBAUD albert.arib...@free.fr writes: 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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Alexander Holler
Am 19.02.2011 15:27, schrieb Albert ARIBAUD: 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 don't see any

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Albert ARIBAUD
Le 19/02/2011 15:34, Måns Rullgård a écrit : Albert ARIBAUDalbert.arib...@free.fr writes: 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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Alexander Holler
Hello, Am 19.02.2011 16:32, schrieb Albert ARIBAUD: Granted :) -- the best option would be to have -float-abi=none, but that does not exist, and I prefer to be sure that if some poor soul puts floating point code in U-Boot then at least it is going to work for all platforms, so I'll keep

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Måns Rullgård
Albert ARIBAUD albert.arib...@free.fr writes: Le 19/02/2011 15:34, Måns Rullgård a écrit : Albert ARIBAUDalbert.arib...@free.fr writes: 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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?=, In message yw1x7hcw5f6w@unicorn.mansr.com you wrote: 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

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear Albert ARIBAUD, In message 4d5fcaca.1060...@free.fr you wrote: 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 explained this several times before. There are good reasons to have

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear Alexander Holler, In message 4d5fce64.8030...@ahsoftware.de you wrote: 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. ;) Read my previous messages, and/or the archives. There _are_ good reasons that

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear =?iso-8859-1?Q?M=E5ns_Rullg=E5rd?=, In message yw1xy65c3xfi@unicorn.mansr.com you wrote: There is one further complication here. If the compiler was built with -mfpu=3Dneon as default, it might emit neon instructions for pure integer code. For the purpose of this discussion, such

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear Alexander Holler, In message 4d5fe485.5030...@ahsoftware.de you wrote: Hmm, but as we've seen, that doesn't work for all platforms because you can't mix softfloat and hardfloat. So using softfloat breaks building on (default-) hardfloat platforms if some poor soul puts floating point

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-19 Thread Wolfgang Denk
Dear Raghuveer Murthy, In message 1298042212-12260-1-git-send-email-raghuveer.mur...@ti.com you wrote: U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id: http://bugs.meego.com/show_bug.cgi?id=13140 Removing the

[U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-18 Thread Raghuveer Murthy
U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id: http://bugs.meego.com/show_bug.cgi?id=13140 Removing the -msoft-float options in the config.mk files, allowed it to be built for both armv7hl and armv7el compilers on

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-18 Thread Albert ARIBAUD
Le 18/02/2011 16:16, Raghuveer Murthy a écrit : U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id: http://bugs.meego.com/show_bug.cgi?id=13140 Is this the official U-Boot? Why is it still at 2010.09 ? Removing the

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-18 Thread Alexander Holler
Hello, Am 18.02.2011 18:01, schrieb Albert ARIBAUD: In no case, however, should -msoft-float be *removed*: that would make float support in U-Boot implicit and dependent on the toolchain in use, and I think U-Boot float support should always be explicit and not depend on the toolchain.

Re: [U-Boot] [RFC][PATCH] ARMV7: Patch to fix hard float build issues

2011-02-18 Thread Wolfgang Denk
Dear Raghuveer Murthy, In message 1298042212-12260-1-git-send-email-raghuveer.mur...@ti.com you wrote: U-boot built for MeeGo on PandaBoard, with compiler option -mfloat-abi=hard, caused a build break. Please refer to the bug id: Well, isn't this a typical Don't do that then! situation? (Cf.