diff mbox series

[U-Boot,1/5] sh: Use -m2a-nofpu only

Message ID 1518906642-13401-1-git-send-email-trini@konsulko.com
State Accepted
Commit 036c9679d25cba87e7b82dc18d6f5f939d9806dd
Delegated to: Marek Vasut
Headers show
Series [U-Boot,1/5] sh: Use -m2a-nofpu only | expand

Commit Message

Tom Rini Feb. 17, 2018, 10:30 p.m. UTC
Based on reading over the GCC manual, passing both -m2a and -m2a-nofpu
are redundant, as -m2a-nofpu will provide functional code for both.  As
-m2a-nofpu functions with more toolchains and does what is expected,
switch.

Cc: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
Signed-off-by: Tom Rini <trini@konsulko.com>
---
 arch/sh/cpu/sh2/config.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marek Vasut Feb. 19, 2018, 4:43 a.m. UTC | #1
On 02/17/2018 11:30 PM, Tom Rini wrote:
> Based on reading over the GCC manual, passing both -m2a and -m2a-nofpu
> are redundant, as -m2a-nofpu will provide functional code for both.  As
> -m2a-nofpu functions with more toolchains and does what is expected,
> switch.
> 
> Cc: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>  arch/sh/cpu/sh2/config.mk | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/sh/cpu/sh2/config.mk b/arch/sh/cpu/sh2/config.mk
> index 12e202d539a0..a063e8361c98 100644
> --- a/arch/sh/cpu/sh2/config.mk
> +++ b/arch/sh/cpu/sh2/config.mk
> @@ -8,7 +8,7 @@
>  ENDIANNESS += -EB
>  
>  ifdef CONFIG_CPU_SH2A
> -PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb
> +PLATFORM_CPPFLAGS += -m2a-nofpu -mb
>  else # SH2
>  PLATFORM_CPPFLAGS += -m3e -mb
>  endif
> 
Applied, thanks.
diff mbox series

Patch

diff --git a/arch/sh/cpu/sh2/config.mk b/arch/sh/cpu/sh2/config.mk
index 12e202d539a0..a063e8361c98 100644
--- a/arch/sh/cpu/sh2/config.mk
+++ b/arch/sh/cpu/sh2/config.mk
@@ -8,7 +8,7 @@ 
 ENDIANNESS += -EB
 
 ifdef CONFIG_CPU_SH2A
-PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb
+PLATFORM_CPPFLAGS += -m2a-nofpu -mb
 else # SH2
 PLATFORM_CPPFLAGS += -m3e -mb
 endif