diff mbox

[U-Boot,5/8] sh: Do not define -ffreestanding twice

Message ID 1393393865-11608-6-git-send-email-yamada.m@jp.panasonic.com
State Accepted
Delegated to: Tom Rini
Headers show

Commit Message

Masahiro Yamada Feb. 26, 2014, 5:51 a.m. UTC
-ffreestanding is defined at the top Makefile for all architectures.
Do not define it twice for SH2A.

Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Cc: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.com>
---

 arch/sh/cpu/sh2/config.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Nobuhiro Iwamatsu Feb. 27, 2014, 9:24 p.m. UTC | #1
Applied, thanks.

Best regards,
  Nobuhiro

2014-02-26 14:51 GMT+09:00 Masahiro Yamada <yamada.m@jp.panasonic.com>:
> -ffreestanding is defined at the top Makefile for all architectures.
> Do not define it twice for SH2A.
>
> Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> Cc: Nobuhiro Iwamatsu <nobuhiro.iwamatsu.yj@renesas.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 69273b4..25abcdd 100644
> --- a/arch/sh/cpu/sh2/config.mk
> +++ b/arch/sh/cpu/sh2/config.mk
> @@ -8,7 +8,7 @@
>  ENDIANNESS += -EB
>
>  ifdef CONFIG_SH2A
> -PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb -ffreestanding
> +PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb
>  else # SH2
>  PLATFORM_CPPFLAGS += -m3e -mb
>  endif
> --
> 1.8.3.2
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
Masahiro Yamada Feb. 28, 2014, 3:39 a.m. UTC | #2
Hello Nobihiro,

Thanks.

I want this and 6/8 on u-boot/master.

Can you please send pull-request now?


Best Regards
Masahiro Yamada
Tom Rini March 4, 2014, 8:37 p.m. UTC | #3
On Fri, Feb 28, 2014 at 12:39:35PM +0900, Masahiro Yamada wrote:

> Hello Nobihiro,
> 
> Thanks.
> 
> I want this and 6/8 on u-boot/master.
> 
> Can you please send pull-request now?

Yes, soon please :)
Nobuhiro Iwamatsu March 5, 2014, 12:45 a.m. UTC | #4
Hi,

These were already applied to u-boot/master.
Thanks, Tom,.

Nobuhiro

2014-03-05 5:37 GMT+09:00 Tom Rini <trini@ti.com>:
> On Fri, Feb 28, 2014 at 12:39:35PM +0900, Masahiro Yamada wrote:
>
>> Hello Nobihiro,
>>
>> Thanks.
>>
>> I want this and 6/8 on u-boot/master.
>>
>> Can you please send pull-request now?
>
> Yes, soon please :)
>
> --
> Tom
>
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>
Tom Rini March 5, 2014, 2:10 a.m. UTC | #5
On Wed, Mar 05, 2014 at 09:45:26AM +0900, Nobuhiro Iwamatsu wrote:
> Hi,
> 
> These were already applied to u-boot/master.
> Thanks, Tom,.

Bah, I was intending to not apply them, but I guess dropped them too
late.  So long as u-boot-sh ends up being sane, oh well :)

> 
> Nobuhiro
> 
> 2014-03-05 5:37 GMT+09:00 Tom Rini <trini@ti.com>:
> > On Fri, Feb 28, 2014 at 12:39:35PM +0900, Masahiro Yamada wrote:
> >
> >> Hello Nobihiro,
> >>
> >> Thanks.
> >>
> >> I want this and 6/8 on u-boot/master.
> >>
> >> Can you please send pull-request now?
> >
> > Yes, soon please :)
> >
> > --
> > Tom
> >
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot@lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
> >
> 
> 
> 
> -- 
> Nobuhiro Iwamatsu
> _______________________________________________
> U-Boot mailing list
> U-Boot@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
diff mbox

Patch

diff --git a/arch/sh/cpu/sh2/config.mk b/arch/sh/cpu/sh2/config.mk
index 69273b4..25abcdd 100644
--- a/arch/sh/cpu/sh2/config.mk
+++ b/arch/sh/cpu/sh2/config.mk
@@ -8,7 +8,7 @@ 
 ENDIANNESS += -EB
 
 ifdef CONFIG_SH2A
-PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb -ffreestanding
+PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb
 else # SH2
 PLATFORM_CPPFLAGS += -m3e -mb
 endif