[PATCH V2 0/3] arm: omap3: Make functions static when possible

Lokesh Vutla lokeshvutla at ti.com
Mon Apr 12 07:46:44 CEST 2021



On 18/03/21 6:10 pm, Adam Ford wrote:
> A few functions are defined without being labeled as static
> to that file.  In an effort to keep SPL as small as possible,
> one function needs to be moved into an #ifdef so it's only enabled
> when the calling function is available, and all of these can simply be
> marked as static.

I see some failures with this series. Can you take a look:
https://source.denx.de/u-boot/custodians/u-boot-ti/-/jobs/254794

Thanks and regards,
Lokesh

> 
> Before:
>    text	   data	    bss	    dec	    hex	filename
>   50988	   1053	   1888	  53929	   d2a9	spl/u-boot-spl
> 
> After:
>    text	   data	    bss	    dec	    hex	filename
>   50972	   1053	   1888	  53913	   d299	spl/u-boot-spl
> 
> While not significant, a few bytes can be very helpful when SPL is limited.
> 
> Adam Ford (3):
>   arm: omap3: Make try_unlock_memory() static
>   arm: omap3: Make secureworld_exit() static
>   arm: omap3: Make secure_unlock_mem() static
> 
>  arch/arm/include/asm/arch-omap3/sys_proto.h |  2 --
>  arch/arm/mach-omap2/omap3/board.c           | 21 +++++++++++----------
>  2 files changed, 11 insertions(+), 12 deletions(-)
> 


More information about the U-Boot mailing list