Hi Christian,
Le 02/02/2012 11:44, Christian Riesch a écrit :
The low level initialization code in
arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S was written for
DM644X SoCs only. This patch makes the lowlevel_init function in this
file a dummy function for SoCs other than DM644X.
Signed-off-by: Christian Riesch<christian.rie...@omicron.at>
Cc: Tom Rini<tr...@ti.com>
Cc: Sergey Kubushyn<k...@koi8.net>
Acked-by: Heiko Schocher<h...@denx.de>
Tested-by: Heiko Schocher<h...@denx.de>
---
arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S
b/arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S
index 7a169b1..5b39484 100644
--- a/arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S
+++ b/arch/arm/cpu/arm926ejs/davinci/lowlevel_init.S
@@ -49,6 +49,7 @@
.globl lowlevel_init
lowlevel_init:
+#ifdef CONFIG_SOC_DM644X
/*-------------------------------------------------------*
* Mask all IRQs by setting all bits in the EINT default *
@@ -707,3 +708,6 @@ DDR2_START_ADDR:
.word 0x80000000
DUMMY_VAL:
.word 0xa55aa55a
+#else /* CONFIG_SOC_DM644X */
+ mov pc, lr
+#endif
Nitpicking here, but:
I don't link the "mov pc, lr" added in #else while it already exists in
the #if part. Can we have a cleaner pair of #if/#else/#endif sections,
one for code, ending right *before* the 'mov pc,lr' and one for the
literals that follow it?
Amicalement,
--
Albert.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
