[Buildroot] [PATCH 1/1] Fix redundant usage of -mcpu and -march/-mtune

Károly Kasza kaszak at gmail.com
Sat Nov 1 21:24:28 UTC 2014


Hi Thomas,

thanks for the notice, I've seen the changes since then.

Best regards,
Karoly

On Sat, Nov 1, 2014 at 10:22 PM, Thomas Petazzoni <
thomas.petazzoni at free-electrons.com> wrote:

> Dear Karoly Kasza,
>
> On Tue, 13 May 2014 15:58:14 +0200, Karoly Kasza wrote:
> > Disable adding -mtune or -march to the external toolchain parameters
> > if -mcpu is present.
> >
> > Signed-off-by: Karoly Kasza <kaszak at gmail.com>
> > ---
> >  toolchain/toolchain-external/toolchain-external.mk |    7 ++++---
> >  1 file changed, 4 insertions(+), 3 deletions(-)
>
> We discussed this patch at the last Buildroot meeting, and our feeling
> is that it's not the correct approach. I've since then sent a patch
> series that fix the same problem. Our approach is:
>
>  * On ARM, to no longer specify -march, but only -mcpu, since the
>    latter is used by gcc to deduce the former.
>
>  * In general, get rid of -mtune, since it doesn't make much sense in
>    the context of Buildroot where we target one specific system, and
>    therefore using -mcpu is sufficient.
>
> So, we've marked your patch as Rejected in our patch tracking system.
>
> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
>



-- 

Üdv,
KK
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20141101/604084a2/attachment-0002.html>


More information about the buildroot mailing list