diff mbox series

[U-Boot] mpc85xx: Add support for -msingle-pic-base

Message ID 20181206162053.30926-1-joakim.tjernlund@infinera.com
State Accepted
Commit 45e81f9ab3e97aec8e8744b23589bc21086cfd43
Delegated to: York Sun
Headers show
Series [U-Boot] mpc85xx: Add support for -msingle-pic-base | expand

Commit Message

Joakim Tjernlund Dec. 6, 2018, 4:20 p.m. UTC
-msingle-pic-base is a new gcc(from 4.6) option for ppc and
it reduces the size of my u-boot with about 4-5 KB.
While at it, add -fno-jump-tables too to save a
few more bytes.

e5500 core:
size u-boot.bef
   text	   data	    bss	    dec	    hex	filename
 473043	  23772	 307104	 803919	  c444f	u-boot.bef
size u-boot.aft
   text	   data	    bss	    dec	    hex	filename
 453195	  23772	 307104	 784071	  bf6c7	u-boot.aft

e500 core:
size u-boot.bef
   text	   data	    bss	    dec	    hex	filename
 292998	  17868	  24968	 335834	  51fda	u-boot.bef
size u-boot.aft
   text	   data	    bss	    dec	    hex	filename
 288002	  17868	  24968	 330838	  50c56	u-boot.aft

Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
---
 arch/powerpc/cpu/mpc85xx/config.mk | 1 +
 arch/powerpc/cpu/mpc85xx/start.S   | 3 +++
 2 files changed, 4 insertions(+)

Comments

York Sun Dec. 7, 2018, 5:47 p.m. UTC | #1
On 12/6/18 8:20 AM, Joakim Tjernlund wrote:
> -msingle-pic-base is a new gcc(from 4.6) option for ppc and
> it reduces the size of my u-boot with about 4-5 KB.
> While at it, add -fno-jump-tables too to save a
> few more bytes.
>
> e5500 core:
> size u-boot.bef
>    text	   data	    bss	    dec	    hex	filename
>  473043	  23772	 307104	 803919	  c444f	u-boot.bef
> size u-boot.aft
>    text	   data	    bss	    dec	    hex	filename
>  453195	  23772	 307104	 784071	  bf6c7	u-boot.aft
>
> e500 core:
> size u-boot.bef
>    text	   data	    bss	    dec	    hex	filename
>  292998	  17868	  24968	 335834	  51fda	u-boot.bef
> size u-boot.aft
>    text	   data	    bss	    dec	    hex	filename
>  288002	  17868	  24968	 330838	  50c56	u-boot.aft
>
> Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
> ---

Tested on e500mc, e5500, e6500 platforms.

Acked-by: York Sun <york.sun@nxp.com>
York Sun Jan. 24, 2019, 4:25 p.m. UTC | #2
On 12/6/18 8:20 AM, Joakim Tjernlund wrote:
> -msingle-pic-base is a new gcc(from 4.6) option for ppc and
> it reduces the size of my u-boot with about 4-5 KB.
> While at it, add -fno-jump-tables too to save a
> few more bytes.
> 
> e5500 core:
> size u-boot.bef
>    text	   data	    bss	    dec	    hex	filename
>  473043	  23772	 307104	 803919	  c444f	u-boot.bef
> size u-boot.aft
>    text	   data	    bss	    dec	    hex	filename
>  453195	  23772	 307104	 784071	  bf6c7	u-boot.aft
> 
> e500 core:
> size u-boot.bef
>    text	   data	    bss	    dec	    hex	filename
>  292998	  17868	  24968	 335834	  51fda	u-boot.bef
> size u-boot.aft
>    text	   data	    bss	    dec	    hex	filename
>  288002	  17868	  24968	 330838	  50c56	u-boot.aft
> 
> Signed-off-by: Joakim Tjernlund <joakim.tjernlund@infinera.com>
> ---

Applied to u-boot-mpc85xx, awaiting upstream. Thanks.

York
diff mbox series

Patch

diff --git a/arch/powerpc/cpu/mpc85xx/config.mk b/arch/powerpc/cpu/mpc85xx/config.mk
index 44d69add68..7a1d81cf2d 100644
--- a/arch/powerpc/cpu/mpc85xx/config.mk
+++ b/arch/powerpc/cpu/mpc85xx/config.mk
@@ -4,6 +4,7 @@ 
 # Xianghua Xiao, X.Xiao@motorola.com
 
 PLATFORM_CPPFLAGS += -Wa,-me500 -msoft-float -mno-string
+PLATFORM_RELFLAGS += -msingle-pic-base -fno-jump-tables
 
 # -mspe=yes is needed to have -mno-spe accepted by a buggy GCC;
 # see "[PATCH,rs6000] make -mno-spe work as expected" on
diff --git a/arch/powerpc/cpu/mpc85xx/start.S b/arch/powerpc/cpu/mpc85xx/start.S
index 932aa08b27..dbc705388c 100644
--- a/arch/powerpc/cpu/mpc85xx/start.S
+++ b/arch/powerpc/cpu/mpc85xx/start.S
@@ -1216,6 +1216,9 @@  _start_cont:
 	mr	r1,r3		/* Transfer to SP(r1) */
 
 	GET_GOT
+	/* Needed for -msingle-pic-base */
+	bl	_GLOBAL_OFFSET_TABLE_@local-4
+	mflr	r30
 
 	/* Pass our potential ePAPR device tree pointer to cpu_init_early_f */
 	mr	r3, r24