On Wednesday, January 14, 2015 at 05:40:52 PM, dingu...@opensource.altera.com wrote: > From: Dinh Nguyen <dingu...@opensource.altera.com> > > For SoCFGPA, use the common ARMv7 lowlevel_init. > > Signed-off-by: Dinh Nguyen <dingu...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot