All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables
@ 2018-10-01 12:52 Peter Korsgaard
  2018-10-01 14:12 ` Mark Corbin
  0 siblings, 1 reply; 4+ messages in thread
From: Peter Korsgaard @ 2018-10-01 12:52 UTC (permalink / raw)
  To: buildroot

commit: https://git.buildroot.net/buildroot/commit/?id=e0d14fb21bc1bd3468e3b4ddc6bf20cce13dcdec
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

Since the introduction of the GCC_TARGET_<foo> variables in
arch/arch.mk in commit bd0640a2139119e2fdb4b384ebf32d1edcb0fdaa
("arch: allow GCC target options to be optionally overwritten") and
the removal of the BR2_GCC_TARGET_CPU_REVISION, the CC_TARGET_<foo>_
variables in pkg-toolchain-external.mk map 1:1 with the corresponding
GCC_TARGET_<foo> variables.

So let's drop the CC_TARGET_<foo>_ variables, and use directly the
GCC_TARGET_<foo> ones.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 .../toolchain-external/pkg-toolchain-external.mk   | 56 ++++++++++------------
 1 file changed, 24 insertions(+), 32 deletions(-)

diff --git a/toolchain/toolchain-external/pkg-toolchain-external.mk b/toolchain/toolchain-external/pkg-toolchain-external.mk
index 241a39a5ee..b1c41b0d97 100644
--- a/toolchain/toolchain-external/pkg-toolchain-external.mk
+++ b/toolchain/toolchain-external/pkg-toolchain-external.mk
@@ -151,14 +151,6 @@ TOOLCHAIN_EXTERNAL_LIBS += $(call qstrip,$(BR2_TOOLCHAIN_EXTRA_EXTERNAL_LIBS))
 # Definition of the CFLAGS to use with the external toolchain, as well as the
 # common toolchain wrapper build arguments
 #
-CC_TARGET_CPU_ := $(GCC_TARGET_CPU)
-CC_TARGET_ARCH_ := $(GCC_TARGET_ARCH)
-CC_TARGET_ABI_ := $(GCC_TARGET_ABI)
-CC_TARGET_NAN_ := $(GCC_TARGET_NAN)
-CC_TARGET_FP32_MODE_ := $(GCC_TARGET_FP32_MODE)
-CC_TARGET_FPU_ := $(GCC_TARGET_FPU)
-CC_TARGET_FLOAT_ABI_ := $(GCC_TARGET_FLOAT_ABI)
-CC_TARGET_MODE_ := $(GCC_TARGET_MODE)
 
 # march/mtune/floating point mode needs to be passed to the external toolchain
 # to select the right multilib variant
@@ -166,39 +158,39 @@ ifeq ($(BR2_x86_64),y)
 TOOLCHAIN_EXTERNAL_CFLAGS += -m64
 TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_64
 endif
-ifneq ($(CC_TARGET_ARCH_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(CC_TARGET_ARCH_)"'
+ifneq ($(GCC_TARGET_ARCH),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(GCC_TARGET_ARCH)"'
 endif
-ifneq ($(CC_TARGET_CPU_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mcpu=$(CC_TARGET_CPU_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_CPU='"$(CC_TARGET_CPU_)"'
+ifneq ($(GCC_TARGET_CPU),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mcpu=$(GCC_TARGET_CPU)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_CPU='"$(GCC_TARGET_CPU)"'
 endif
-ifneq ($(CC_TARGET_ABI_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mabi=$(CC_TARGET_ABI_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ABI='"$(CC_TARGET_ABI_)"'
+ifneq ($(GCC_TARGET_ABI),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mabi=$(GCC_TARGET_ABI)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ABI='"$(GCC_TARGET_ABI)"'
 endif
 ifeq ($(BR2_TOOLCHAIN_HAS_MNAN_OPTION),y)
-ifneq ($(CC_TARGET_NAN_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mnan=$(CC_TARGET_NAN_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_NAN='"$(CC_TARGET_NAN_)"'
+ifneq ($(GCC_TARGET_NAN),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mnan=$(GCC_TARGET_NAN)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_NAN='"$(GCC_TARGET_NAN)"'
 endif
 endif
-ifneq ($(CC_TARGET_FP32_MODE_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mfp$(CC_TARGET_FP32_MODE_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FP32_MODE='"$(CC_TARGET_FP32_MODE_)"'
+ifneq ($(GCC_TARGET_FP32_MODE),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mfp$(GCC_TARGET_FP32_MODE)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FP32_MODE='"$(GCC_TARGET_FP32_MODE)"'
 endif
-ifneq ($(CC_TARGET_FPU_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mfpu=$(CC_TARGET_FPU_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FPU='"$(CC_TARGET_FPU_)"'
+ifneq ($(GCC_TARGET_FPU),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mfpu=$(GCC_TARGET_FPU)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FPU='"$(GCC_TARGET_FPU)"'
 endif
-ifneq ($(CC_TARGET_FLOAT_ABI_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -mfloat-abi=$(CC_TARGET_FLOAT_ABI_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FLOAT_ABI='"$(CC_TARGET_FLOAT_ABI_)"'
+ifneq ($(GCC_TARGET_FLOAT_ABI),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -mfloat-abi=$(GCC_TARGET_FLOAT_ABI)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FLOAT_ABI='"$(GCC_TARGET_FLOAT_ABI)"'
 endif
-ifneq ($(CC_TARGET_MODE_),)
-TOOLCHAIN_EXTERNAL_CFLAGS += -m$(CC_TARGET_MODE_)
-TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_MODE='"$(CC_TARGET_MODE_)"'
+ifneq ($(GCC_TARGET_MODE),)
+TOOLCHAIN_EXTERNAL_CFLAGS += -m$(GCC_TARGET_MODE)
+TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_MODE='"$(GCC_TARGET_MODE)"'
 endif
 ifeq ($(BR2_BINFMT_FLAT),y)
 TOOLCHAIN_EXTERNAL_CFLAGS += -Wl,-elf2flt

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables
  2018-10-01 12:52 [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables Peter Korsgaard
@ 2018-10-01 14:12 ` Mark Corbin
  2018-10-01 15:33   ` Peter Korsgaard
  2018-10-02 14:29   ` Thomas Petazzoni
  0 siblings, 2 replies; 4+ messages in thread
From: Mark Corbin @ 2018-10-01 14:12 UTC (permalink / raw)
  To: buildroot

Hello Peter

I've just noticed something with this patch - see below...

On 01/10/18 13:52, Peter Korsgaard wrote:
> commit: https://git.buildroot.net/buildroot/commit/?id=e0d14fb21bc1bd3468e3b4ddc6bf20cce13dcdec
> branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master
>
> Since the introduction of the GCC_TARGET_<foo> variables in
> arch/arch.mk in commit bd0640a2139119e2fdb4b384ebf32d1edcb0fdaa
> ("arch: allow GCC target options to be optionally overwritten") and
> the removal of the BR2_GCC_TARGET_CPU_REVISION, the CC_TARGET_<foo>_
> variables in pkg-toolchain-external.mk map 1:1 with the corresponding
> GCC_TARGET_<foo> variables.
>
> So let's drop the CC_TARGET_<foo>_ variables, and use directly the
> GCC_TARGET_<foo> ones.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> ---
>  .../toolchain-external/pkg-toolchain-external.mk   | 56 ++++++++++------------
>  1 file changed, 24 insertions(+), 32 deletions(-)
>
> diff --git a/toolchain/toolchain-external/pkg-toolchain-external.mk b/toolchain/toolchain-external/pkg-toolchain-external.mk
> index 241a39a5ee..b1c41b0d97 100644
> --- a/toolchain/toolchain-external/pkg-toolchain-external.mk
> +++ b/toolchain/toolchain-external/pkg-toolchain-external.mk
> @@ -151,14 +151,6 @@ TOOLCHAIN_EXTERNAL_LIBS += $(call qstrip,$(BR2_TOOLCHAIN_EXTRA_EXTERNAL_LIBS))
>  # Definition of the CFLAGS to use with the external toolchain, as well as the
>  # common toolchain wrapper build arguments
>  #
> -CC_TARGET_CPU_ := $(GCC_TARGET_CPU)
> -CC_TARGET_ARCH_ := $(GCC_TARGET_ARCH)
> -CC_TARGET_ABI_ := $(GCC_TARGET_ABI)
> -CC_TARGET_NAN_ := $(GCC_TARGET_NAN)
> -CC_TARGET_FP32_MODE_ := $(GCC_TARGET_FP32_MODE)
> -CC_TARGET_FPU_ := $(GCC_TARGET_FPU)
> -CC_TARGET_FLOAT_ABI_ := $(GCC_TARGET_FLOAT_ABI)
> -CC_TARGET_MODE_ := $(GCC_TARGET_MODE)
>  
>  # march/mtune/floating point mode needs to be passed to the external toolchain
>  # to select the right multilib variant
> @@ -166,39 +158,39 @@ ifeq ($(BR2_x86_64),y)
>  TOOLCHAIN_EXTERNAL_CFLAGS += -m64
>  TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_64
>  endif
> -ifneq ($(CC_TARGET_ARCH_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(CC_TARGET_ARCH_)"'
> +ifneq ($(GCC_TARGET_ARCH),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(GCC_TARGET_ARCH)"'

...isn't there a 'G' missing in front of the 'CC_TARGET_ARCH' in the
inserted TOOLCHAIN_EXTERNAL_CFLAGS line above?

Regards

Mark

>  endif
> -ifneq ($(CC_TARGET_CPU_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mcpu=$(CC_TARGET_CPU_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_CPU='"$(CC_TARGET_CPU_)"'
> +ifneq ($(GCC_TARGET_CPU),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mcpu=$(GCC_TARGET_CPU)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_CPU='"$(GCC_TARGET_CPU)"'
>  endif
> -ifneq ($(CC_TARGET_ABI_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mabi=$(CC_TARGET_ABI_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ABI='"$(CC_TARGET_ABI_)"'
> +ifneq ($(GCC_TARGET_ABI),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mabi=$(GCC_TARGET_ABI)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ABI='"$(GCC_TARGET_ABI)"'
>  endif
>  ifeq ($(BR2_TOOLCHAIN_HAS_MNAN_OPTION),y)
> -ifneq ($(CC_TARGET_NAN_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mnan=$(CC_TARGET_NAN_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_NAN='"$(CC_TARGET_NAN_)"'
> +ifneq ($(GCC_TARGET_NAN),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mnan=$(GCC_TARGET_NAN)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_NAN='"$(GCC_TARGET_NAN)"'
>  endif
>  endif
> -ifneq ($(CC_TARGET_FP32_MODE_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mfp$(CC_TARGET_FP32_MODE_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FP32_MODE='"$(CC_TARGET_FP32_MODE_)"'
> +ifneq ($(GCC_TARGET_FP32_MODE),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mfp$(GCC_TARGET_FP32_MODE)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FP32_MODE='"$(GCC_TARGET_FP32_MODE)"'
>  endif
> -ifneq ($(CC_TARGET_FPU_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mfpu=$(CC_TARGET_FPU_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FPU='"$(CC_TARGET_FPU_)"'
> +ifneq ($(GCC_TARGET_FPU),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mfpu=$(GCC_TARGET_FPU)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FPU='"$(GCC_TARGET_FPU)"'
>  endif
> -ifneq ($(CC_TARGET_FLOAT_ABI_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -mfloat-abi=$(CC_TARGET_FLOAT_ABI_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FLOAT_ABI='"$(CC_TARGET_FLOAT_ABI_)"'
> +ifneq ($(GCC_TARGET_FLOAT_ABI),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -mfloat-abi=$(GCC_TARGET_FLOAT_ABI)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_FLOAT_ABI='"$(GCC_TARGET_FLOAT_ABI)"'
>  endif
> -ifneq ($(CC_TARGET_MODE_),)
> -TOOLCHAIN_EXTERNAL_CFLAGS += -m$(CC_TARGET_MODE_)
> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_MODE='"$(CC_TARGET_MODE_)"'
> +ifneq ($(GCC_TARGET_MODE),)
> +TOOLCHAIN_EXTERNAL_CFLAGS += -m$(GCC_TARGET_MODE)
> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_MODE='"$(GCC_TARGET_MODE)"'
>  endif
>  ifeq ($(BR2_BINFMT_FLAT),y)
>  TOOLCHAIN_EXTERNAL_CFLAGS += -Wl,-elf2flt
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
Mark Corbin
Embecosm Ltd.
https://www.embecosm.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables
  2018-10-01 14:12 ` Mark Corbin
@ 2018-10-01 15:33   ` Peter Korsgaard
  2018-10-02 14:29   ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2018-10-01 15:33 UTC (permalink / raw)
  To: buildroot

>>>>> "Mark" == Mark Corbin <mark.corbin@embecosm.com> writes:

 > Hello Peter
 > I've just noticed something with this patch - see below...

 > On 01/10/18 13:52, Peter Korsgaard wrote:
 >> commit: https://git.buildroot.net/buildroot/commit/?id=e0d14fb21bc1bd3468e3b4ddc6bf20cce13dcdec
 >> branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master
 >> 
 >> Since the introduction of the GCC_TARGET_<foo> variables in
 >> arch/arch.mk in commit bd0640a2139119e2fdb4b384ebf32d1edcb0fdaa
 >> ("arch: allow GCC target options to be optionally overwritten") and
 >> the removal of the BR2_GCC_TARGET_CPU_REVISION, the CC_TARGET_<foo>_
 >> variables in pkg-toolchain-external.mk map 1:1 with the corresponding
 >> GCC_TARGET_<foo> variables.
 >> 
 >> So let's drop the CC_TARGET_<foo>_ variables, and use directly the
 >> GCC_TARGET_<foo> ones.
 >> 
 >> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
 >> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
 >> ---
 >> .../toolchain-external/pkg-toolchain-external.mk   | 56 ++++++++++------------
 >> 1 file changed, 24 insertions(+), 32 deletions(-)
 >> 
 >> diff --git a/toolchain/toolchain-external/pkg-toolchain-external.mk b/toolchain/toolchain-external/pkg-toolchain-external.mk
 >> index 241a39a5ee..b1c41b0d97 100644
 >> --- a/toolchain/toolchain-external/pkg-toolchain-external.mk
 >> +++ b/toolchain/toolchain-external/pkg-toolchain-external.mk
 >> @@ -151,14 +151,6 @@ TOOLCHAIN_EXTERNAL_LIBS += $(call qstrip,$(BR2_TOOLCHAIN_EXTRA_EXTERNAL_LIBS))
 >> # Definition of the CFLAGS to use with the external toolchain, as well as the
 >> # common toolchain wrapper build arguments
 >> #
 >> -CC_TARGET_CPU_ := $(GCC_TARGET_CPU)
 >> -CC_TARGET_ARCH_ := $(GCC_TARGET_ARCH)
 >> -CC_TARGET_ABI_ := $(GCC_TARGET_ABI)
 >> -CC_TARGET_NAN_ := $(GCC_TARGET_NAN)
 >> -CC_TARGET_FP32_MODE_ := $(GCC_TARGET_FP32_MODE)
 >> -CC_TARGET_FPU_ := $(GCC_TARGET_FPU)
 >> -CC_TARGET_FLOAT_ABI_ := $(GCC_TARGET_FLOAT_ABI)
 >> -CC_TARGET_MODE_ := $(GCC_TARGET_MODE)
 >> 
 >> # march/mtune/floating point mode needs to be passed to the external toolchain
 >> # to select the right multilib variant
 >> @@ -166,39 +158,39 @@ ifeq ($(BR2_x86_64),y)
 >> TOOLCHAIN_EXTERNAL_CFLAGS += -m64
 >> TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_64
 >> endif
 >> -ifneq ($(CC_TARGET_ARCH_),)
 >> -TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH_)
 >> -TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(CC_TARGET_ARCH_)"'
 >> +ifneq ($(GCC_TARGET_ARCH),)
 >> +TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH)
 >> +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(GCC_TARGET_ARCH)"'

 > ...isn't there a 'G' missing in front of the 'CC_TARGET_ARCH' in the
 > inserted TOOLCHAIN_EXTERNAL_CFLAGS line above?

Hmm, correct - I'll fix. Thanks!

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables
  2018-10-01 14:12 ` Mark Corbin
  2018-10-01 15:33   ` Peter Korsgaard
@ 2018-10-02 14:29   ` Thomas Petazzoni
  1 sibling, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2018-10-02 14:29 UTC (permalink / raw)
  To: buildroot

Hello,

On Mon, 1 Oct 2018 15:12:35 +0100, Mark Corbin wrote:

> > +ifneq ($(GCC_TARGET_ARCH),)
> > +TOOLCHAIN_EXTERNAL_CFLAGS += -march=$(CC_TARGET_ARCH)
> > +TOOLCHAIN_EXTERNAL_TOOLCHAIN_WRAPPER_ARGS += -DBR_ARCH='"$(GCC_TARGET_ARCH)"'  
> 
> ...isn't there a 'G' missing in front of the 'CC_TARGET_ARCH' in the
> inserted TOOLCHAIN_EXTERNAL_CFLAGS line above?

Indeed, sorry for the mess here. Seems like I messed up the
search/replace. Good that we have people reviewing the patches! :-)

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-10-02 14:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-10-01 12:52 [Buildroot] [git commit] toolchain-external: drop no longer needed CC_TARGET_<foo>_ variables Peter Korsgaard
2018-10-01 14:12 ` Mark Corbin
2018-10-01 15:33   ` Peter Korsgaard
2018-10-02 14:29   ` Thomas Petazzoni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.