On 11:03 Fri 23 Apr , Sascha Hauer wrote: > Lets translate the startup code to a language we all understand better. > Tested on pcm038 (arm v5) and pcm043 (arm v6). > > Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> > --- > arch/arm/cpu/Makefile | 6 +- > arch/arm/cpu/start-arm.S | 248 ------------------------------------ > arch/arm/cpu/start.c | 107 ++++++++++++++++ > arch/arm/include/asm/barebox-arm.h | 3 + > arch/arm/lib/barebox.lds.S | 2 +- > 5 files changed, 112 insertions(+), 254 deletions(-) > delete mode 100644 arch/arm/cpu/start-arm.S > create mode 100644 arch/arm/cpu/start.c > > diff --git a/arch/arm/cpu/Makefile b/arch/arm/cpu/Makefile > index ae1f762..036768e 100644 > --- a/arch/arm/cpu/Makefile > +++ b/arch/arm/cpu/Makefile > @@ -1,15 +1,11 @@ > obj-y += cpu.o > obj-y += exceptions.o > obj-y += interrupts.o > +obj-y += start.o > > # > # Any variants can be called as start-armxyz.S > # > -obj-$(CONFIG_CPU_ARM920T) += start-arm.o > -obj-$(CONFIG_CPU_ARM926T) += start-arm.o > -obj-$(CONFIG_CPU_V7) += start-arm.o > -obj-$(CONFIG_ARCH_IMX31) += start-arm.o > -obj-$(CONFIG_ARCH_IMX35) += start-arm.o > obj-$(CONFIG_CMD_ARM_CPUINFO) += cpuinfo.o > obj-$(CONFIG_MMU) += mmu.o > obj-$(CONFIG_CPU_32v4T) += cache-armv4.o > diff --git a/arch/arm/cpu/start-arm.S b/arch/arm/cpu/start-arm.S > deleted file mode 100644 > index 4d61cee..0000000 > --- a/arch/arm/cpu/start-arm.S > +++ /dev/null > @@ -1,248 +0,0 @@ > -/* > + > + /* disable MMU stuff and caches */ > + __asm__ __volatile__("mrc p15, 0, %0, c1, c0, 0":"=r"(r)); > + r &= ~0x2387; /* clear bits 13, 9:8, 7, 2:0 (--V- --RS B--- -CAM) */ > + r |= 0x1002; /* set bit 2 (A) Align, set bit 12 (I) I-Cache */ can we use here the CP15 macro as CR_M and get_cr and set_cr? > + __asm__ __volatile__("mcr p15, 0, %0, c1, c0, 0" : : "r"(r)); > + > +#ifdef CONFIG_MACH_DO_LOWLEVEL_INIT > + board_init_lowlevel(); > +#endif why not always avoid the ifdef CONFIG_ in the code and define an inline? > + board_init_lowlevel_return(); > +} > + otherwise it's really easier than the asm to maintain evenif I prefer to write it in asm personnaly Best Regards, J. _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox