[U-Boot] [PATCH] arm: armv7-a: Enable mtune when CONFIG_CPU_V7A

[U-Boot] [PATCH] arm: armv7-a: Enable mtune when CONFIG_CPU_V7A

[Adam Ford]

arch/arm/Makefile has a list of options for mtune, but the
entry for CONFIG_CPU_V7A is empty.  By tuning for generic-armv7-a
both the size of SPL and u-boot shrink a little.  For those
with limited resources in SPL, every little bit helps.

The following size changed apply to omap3_logic

Original:
   text	   data	    bss	    dec	    hex	filename
 541198	  22692	 327040	 890930	  d9832	u-boot

   text	   data	    bss	    dec	    hex	filename
  50964	    429	  67580	 118973	  1d0bd	spl/u-boot-spl

With -mtune=generic-armv7-a
   text	   data	    bss	    dec	    hex	filename
 540934	  22692	 327056	 890682	  d973a	u-boot

   text	   data	    bss	    dec	    hex	filename
  50972	    429	  67580	 118981	  1d0c5	spl/u-boot-spl

Signed-off-by: Adam Ford <aford173@gmail.com>

diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index cac58bdc4d..b9f0903f9b 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -40,7 +40,7 @@ tune-$(CONFIG_CPU_SA1100)	=-mtune=strongarm1100
 tune-$(CONFIG_CPU_PXA)		=-mcpu=xscale
 tune-$(CONFIG_CPU_ARM1136)	=
 tune-$(CONFIG_CPU_ARM1176)	=
-tune-$(CONFIG_CPU_V7A)		=
+tune-$(CONFIG_CPU_V7A)		=-mtune=generic-armv7-a
 tune-$(CONFIG_CPU_V7R)		=
 tune-$(CONFIG_ARM64)		=
 
-- 
2.17.1

[U-Boot] [PATCH] arm: armv7-a: Enable mtune when CONFIG_CPU_V7A

[Tom Rini]

On Fri, Aug 03, 2018 at 05:23:21AM -0500, Adam Ford wrote:

> arch/arm/Makefile has a list of options for mtune, but the
> entry for CONFIG_CPU_V7A is empty.  By tuning for generic-armv7-a
> both the size of SPL and u-boot shrink a little.  For those
> with limited resources in SPL, every little bit helps.
> 
> The following size changed apply to omap3_logic
> 
> Original:
>    text	   data	    bss	    dec	    hex	filename
>  541198	  22692	 327040	 890930	  d9832	u-boot
> 
>    text	   data	    bss	    dec	    hex	filename
>   50964	    429	  67580	 118973	  1d0bd	spl/u-boot-spl
> 
> With -mtune=generic-armv7-a
>    text	   data	    bss	    dec	    hex	filename
>  540934	  22692	 327056	 890682	  d973a	u-boot
> 
>    text	   data	    bss	    dec	    hex	filename
>   50972	    429	  67580	 118981	  1d0c5	spl/u-boot-spl
> 
> Signed-off-by: Adam Ford <aford173@gmail.com>
> 
> diff --git a/arch/arm/Makefile b/arch/arm/Makefile
> index cac58bdc4d..b9f0903f9b 100644
> --- a/arch/arm/Makefile
> +++ b/arch/arm/Makefile
> @@ -40,7 +40,7 @@ tune-$(CONFIG_CPU_SA1100)	=-mtune=strongarm1100
>  tune-$(CONFIG_CPU_PXA)		=-mcpu=xscale
>  tune-$(CONFIG_CPU_ARM1136)	=
>  tune-$(CONFIG_CPU_ARM1176)	=
> -tune-$(CONFIG_CPU_V7A)		=
> +tune-$(CONFIG_CPU_V7A)		=-mtune=generic-armv7-a
>  tune-$(CONFIG_CPU_V7R)		=
>  tune-$(CONFIG_ARM64)		=

I think we might need to use cc-option here.  We have that around the
arch portion due to llvm using 'armv7' not 'armv7-a'.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180803/6a2f6dca/attachment.sig>