[U-Boot] [PATCH 2/2] SPL: Allow ARM926EJS to avoid compiling in the CPU support code

Albert ARIBAUD
2011-10-18T21:33:59Z

Hi Marek,

Le 06/10/2011 02:13, Marek Vasut a ?crit :
> This allows the SPL to avoid compiling in the CPU support code.
>
> Signed-off-by: Marek Vasut<marek.vasut@gmail.com>
> Cc: Stefano Babic<sbabic@denx.de>
> Cc: Wolfgang Denk<wd@denx.de>
> Cc: Detlev Zundel<dzu@denx.de>
> Cc: Scott Wood<scottwood@freescale.com>
> ---
> arch/arm/cpu/arm926ejs/Makefile | 7 +++++++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/arch/arm/cpu/arm926ejs/Makefile b/arch/arm/cpu/arm926ejs/Makefile
> index 930e0d1..3f9b0f1 100644
> --- a/arch/arm/cpu/arm926ejs/Makefile
> +++ b/arch/arm/cpu/arm926ejs/Makefile
> @@ -28,6 +28,13 @@ LIB = $(obj)lib$(CPU).o
> START = start.o
> COBJS = cpu.o
>
> +ifdef CONFIG_SPL_BUILD
> +ifdef CONFIG_SPL_NO_CPU_SUPPORT_CODE
> +START :=
> +COBJS :=
> +endif
> +endif
> +
> SRCS := $(START:.o=.S) $(SOBJS:.o=.S) $(COBJS:.o=.c)
> OBJS := $(addprefix $(obj),$(COBJS) $(SOBJS))
> START := $(addprefix $(obj),$(START))

cpu.c basically contains one cache management function and one
linux-boot-related function probably better suited in bootm... Rather
than adding a config option to avoid compiling cpu.c, should we not
simply move the functions where they belong?

Amicalement,
--
Albert.