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 Rieschchristian.rie...@omicron.at
Cc: Tom Rinitr...@ti.com
Cc: Sergey Kubushynk...@koi8.net
Acked-by: Heiko Schocherh...@denx.de
Tested-by: Heiko Schocherh...@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 @@
.globllowlevel_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 0x8000
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
