[U-Boot] [PATCHv1 12/22] arm: socfpga: spl: Use common lowlevel_init

Marek Vasut
2015-01-14T23:51:33Z

On Wednesday, January 14, 2015 at 05:40:52 PM, dinguyen at opensource.altera.com
wrote:
> From: Dinh Nguyen <dinguyen@opensource.altera.com>
>
> For SoCFGPA, use the common ARMv7 lowlevel_init.
>
> Signed-off-by: Dinh Nguyen <dinguyen@opensource.altera.com>
> ---
> arch/arm/cpu/armv7/Makefile | 2 +-
> arch/arm/cpu/armv7/socfpga/lowlevel_init.S | 4 ----
> 2 files changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/arch/arm/cpu/armv7/Makefile b/arch/arm/cpu/armv7/Makefile
> index 409e6f5..d0434b1 100644
> --- a/arch/arm/cpu/armv7/Makefile
> +++ b/arch/arm/cpu/armv7/Makefile
> @@ -12,7 +12,7 @@ obj-y += cache_v7.o
> obj-y += cpu.o
> obj-y += syslib.o
>
> -ifneq
> ($(CONFIG_AM43XX)$(CONFIG_AM33XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX)$(CON
> FIG_TEGRA)$(CONFIG_MX6)$(CONFIG_TI81XX)$(CONFIG_AT91FAMILY)$(CONFIG_SUNXI),
> ) +ifneq
> ($(CONFIG_AM43XX)$(CONFIG_AM33XX)$(CONFIG_OMAP44XX)$(CONFIG_OMAP54XX)$(CON
> FIG_TEGRA)$(CONFIG_MX6)$(CONFIG_TI81XX)$(CONFIG_AT91FAMILY)$(CONFIG_SUNXI)$
> (CONFIG_SOCFPGA),) ifneq ($(CONFIG_SKIP_LOWLEVEL_INIT),y)
> obj-y += lowlevel_init.o
> endif
> diff --git a/arch/arm/cpu/armv7/socfpga/lowlevel_init.S
> b/arch/arm/cpu/armv7/socfpga/lowlevel_init.S index 2f2e9fc..f251f07 100644
> --- a/arch/arm/cpu/armv7/socfpga/lowlevel_init.S
> +++ b/arch/arm/cpu/armv7/socfpga/lowlevel_init.S
> @@ -14,10 +14,6 @@ save_boot_params:
> bx lr
>
>
> -/* Set up the platform, once the cpu has been initialized */
> -.globl lowlevel_init
> -lowlevel_init:
> -

You want to remove the whole lowlevel_init.S file , since what you removed here
is only the symbol, not the whole function body. The function body is this stuff
below here and it's now both unreachable code and useless as well :)

Otherwise this is cool, thanks!

> /* Remap */
> #ifdef CONFIG_SPL_BUILD
> /*

Best regards,
Marek Vasut