Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.debian.kernel > #62853

Re: [PATCH] powerpc: Fix -mcpu= options for SPE-only compiler

From Segher Boessenkool <segher@kernel.crashing.org>
Newsgroups linux.debian.kernel
Subject Re: [PATCH] powerpc: Fix -mcpu= options for SPE-only compiler
Date 2018-12-27 02:20 +0100
Message-ID <x9sv7-1jv-1@gated-at.bofh.it> (permalink)
References <x9icp-3CE-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


Hi!

On Wed, Dec 26, 2018 at 02:15:11PM +0000, Ben Hutchings wrote:
> GCC for Debian's "powerpcspe" architecture only supports 32-bit
> SPE targets, and using -mcpu=powerpc or -mcpu=powerpc64 is a fatal
> error.

I don't see where your patch touches -mcpu=powerpc, and I don't see how
it would make a difference either?

> +ifdef CONFIG_PPC64
>  ifdef CONFIG_PPC_BOOK3S_64
>  ifdef CONFIG_CPU_LITTLE_ENDIAN
>  CFLAGS-$(CONFIG_GENERIC_CPU) += -mcpu=power8
> @@ -177,6 +178,7 @@ endif
>  else
>  CFLAGS-$(CONFIG_GENERIC_CPU) += -mcpu=powerpc64
>  endif
> +endif

How can CONFIG_PPC_BOOK3S_64 be true while CONFIG_PPC64 is fals?  That
does not make a lot of sense.


Segher

Back to linux.debian.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] powerpc: Fix -mcpu= options for SPE-only compiler Ben Hutchings <ben@decadent.org.uk> - 2018-12-26 15:20 +0100
  Re: [PATCH] powerpc: Fix -mcpu= options for SPE-only compiler Segher Boessenkool <segher@kernel.crashing.org> - 2018-12-27 02:20 +0100
    Re: [PATCH] powerpc: Fix -mcpu= options for SPE-only compiler Ben Hutchings <ben@decadent.org.uk> - 2018-12-27 03:40 +0100
      Re: [PATCH] powerpc: Fix -mcpu= options for SPE-only compiler Segher Boessenkool <segher@kernel.crashing.org> - 2018-12-29 16:40 +0100

csiph-web