On Sun, Oct 21, 2007 at 07:57:39PM +0200, Geert Uytterhoeven wrote:
> On Fri, 19 Oct 2007, Sam Ravnborg wrote:
> > The cc-cross-prefix is new and developed on request from Geert Uytterhoeven.
> > With cc-cross-prefix it is now much easier to have a few default
> > cross compile prefixes and defaulting to none - if none of them were
> > present.
> > ARCH maintainers are expected to pick up this feature soon.
>
> Thanks, seems to work fine!
>
> m68k: Use cc-cross-prefix
>
> Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]>
> ---
> arch/m68k/Makefile | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> --- a/arch/m68k/Makefile
> +++ b/arch/m68k/Makefile
> @@ -13,16 +13,14 @@
> # Copyright (C) 1994 by Hamish Macdonald
> #
>
> -# test for cross compiling
> -COMPILE_ARCH = $(shell uname -m)
> -
> # override top level makefile
> AS += -m68020
> LDFLAGS := -m m68kelf
> LDFLAGS_MODULE += -T $(srctree)/arch/m68k/kernel/module.lds
> ifneq ($(COMPILE_ARCH),$(ARCH))
> - # prefix for cross-compiling binaries
> - CROSS_COMPILE = m68k-linux-gnu-
> + ifeq ($(CROSS_COMPILE),)
> + CROSS_COMPILE := $(call cc-cross-prefix, m68k-linux-gnu-
> m68k-linux-)
> + endif
> endif
Patch seems bogus.
You kill the assignment to COMPILE_ARCH yet you
refer to it in the line:
ifneq ($(COMPILE_ARCH),...
Did you mean to use $(SUBARCH) here?
Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
