[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20071012102124.GA18959@uranus.ravnborg.org>
Date: Fri, 12 Oct 2007 12:21:24 +0200
From: Sam Ravnborg <sam@...nborg.org>
To: Geert Uytterhoeven <geert@...ux-m68k.org>
Cc: Finn Thain <fthain@...egraphics.com.au>,
Rob Landley <rob@...dley.net>,
LKML <linux-kernel@...r.kernel.org>,
Linux/m68k <linux-m68k@...r.kernel.org>
Subject: Re: [PATCH] Make m68k cross compile like every other architecture.
On Fri, Oct 12, 2007 at 10:51:00AM +0200, Geert Uytterhoeven wrote:
> On Fri, 12 Oct 2007, Finn Thain wrote:
> > > If your cross-compiler is called differently than the default on in
> > > arch/*/Makefile,
> >
> > Part of the problem is that there is no compelling default. The name of
> > the cross-compiler can vary depending on the chosen executable prefix or
> > the chosen target tuple.
> >
> > Let's say I create new cross toolchains for both m68k and powerpc. I use
> > the "m68k-linux-gnu" to follow the m68k default, but let's assume powerpc
> > users prefer their default "powerpc-linux" so I follow that too. The
> > result is that I now have gcc's -B option working (or not working) in
> > suprising ways too.
> >
> > Uniformity is more helpful downstream than a different arbitrary default
> > for each architecture. Debian should patch their default into their kernel
> > source packages if they've standardised on cross compiler prefixes.
> >
> > > what's the problem with calling?
> > >
> > > make ARCH=myarch CROSS_COMPILE=my-cross-compile-prefx-
> > >
> > > This has been working for +10 years on all non-ia32 platforms I ever worked on.
> >
> > But there is no problem with calling that (?). Rob's patch doesn't change
> > this.
>
> But it makes life harder for the people who use it daily.
>
> If the consensus is to not provide default cross compiler prefixes in
> arch/*/Makefile, fine for me, but IFF it's done for all architectures
> (i.e. check with the blackfin, h8300, mips, parisc, and xtensa people
> first).
Test if the expected gcc is available and set CROSS_COMPILE
if it is present. Otherwise do not touch CROSS_COMPILE.
m68kgcc := $(shell "test if m68k-gcc is present and echo 'y' if so")
ifeq ($(m68kgcc,y)
CROSS_COMPILE := m68k-foo
endif
Then everyone is happy.
parisc does something similar.
Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists