[U-Boot] [PATCH v1 2/2] imx: fix exception vectors relocation in i.MX27

2014-11-11 Thread Albert ARIBAUD
Commit 3ff46cc4 fixed exception vectors setting in
the general ARM case, by either copying the exception
and indirect vector tables to normal (0x) or
high (0x) vectors address, or setting VBAR to
U-Boot's base if applicable.

i.MX27 SoC is ARM926E-JS, thus has only normal and
high options, but does not provide RAM at 0x
and has only ROM at 0x; it is therefore not
possible to move or change its exception vectors.

Besides, i.MX27 ROM code does provide an indirect
vectors table but at a non-standard address and with
the reset and reserved vectors missing.

Turn the current vector relocation code into a weak
routine called after relocate_code from crt0, and add
strong version for i.MX27.

Signed-off-by: Albert ARIBAUD albert.u.b...@aribaud.net
---

 arch/arm/cpu/arm926ejs/mx27/Makefile   |  4 ++
 arch/arm/cpu/arm926ejs/mx27/relocate.S | 49 
 arch/arm/lib/crt0.S|  5 +++
 arch/arm/lib/relocate.S| 69 --
 4 files changed, 99 insertions(+), 28 deletions(-)
 create mode 100644 arch/arm/cpu/arm926ejs/mx27/relocate.S

diff --git a/arch/arm/cpu/arm926ejs/mx27/Makefile 
b/arch/arm/cpu/arm926ejs/mx27/Makefile
index 4976bbb..0edf144 100644
--- a/arch/arm/cpu/arm926ejs/mx27/Makefile
+++ b/arch/arm/cpu/arm926ejs/mx27/Makefile
@@ -5,3 +5,7 @@
 # SPDX-License-Identifier: GPL-2.0+
 
 obj-y  = generic.o reset.o timer.o
+
+ifndef CONFIG_SPL_BUILD
+obj-y  += relocate.o
+endif
diff --git a/arch/arm/cpu/arm926ejs/mx27/relocate.S 
b/arch/arm/cpu/arm926ejs/mx27/relocate.S
new file mode 100644
index 000..97003b3
--- /dev/null
+++ b/arch/arm/cpu/arm926ejs/mx27/relocate.S
@@ -0,0 +1,49 @@
+/*
+ *  relocate - i.MX27-specific vector relocation
+ *
+ *  Copyright (c) 2013  Albert ARIBAUD albert.u.b...@aribaud.net
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#include asm-offsets.h
+#include config.h
+#include linux/linkage.h
+
+/*
+ * The i.MX27 SoC is very specific with respect to exceptions: it
+ * does not provide RAM at the high vectors address (0x),
+ * thus only the low address (0x) is useable; but that is
+ * in ROM. Therefore, vectors cannot be changed at all.
+ *
+ * However, these ROM-based vectors actually just perform indirect
+ * calls through pointers located in RAM at SoC-specific addresses,
+ * as follows:
+ *
+ * Offset  Exception  Use by ROM code
+ * 0x  reset  indirect branch to [0x0014]
+ * 0x0004  undefined instruction  indirect branch to [0xfef0]
+ * 0x0008  software interrupt indirect branch to [0xfef4]
+ * 0x000c  prefetch abort indirect branch to [0xfef8]
+ * 0x0010  data abort indirect branch to [0xfefc]
+ * 0x0014  (reserved in ARMv5)vector to ROM reset: 0xc000
+ * 0x0018  IRQindirect branch to [0xff00]
+ * 0x001c  FIQindirect branch to [0xff04]
+ *
+ * In order to initialize exceptions on i.MX27, we must copy U-Boot's
+ * indirect (not exception!) vector table into 0xfef0..0xff04
+ * taking care not to copy vectors 0x00 (reset) and 0x14 (reserved).
+ */
+
+   .section.text.relocate_vectors,ax,%progbits
+
+ENTRY(relocate_vectors)
+
+   ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
+   ldr r1, =0xFEF0
+   ldmia   r0!, {r2-r8,r10} /* load all eight vectors */
+   stmia   r1!, {r3-r6,r8,r10} /* only write supported vectors */
+
+   bx  lr
+
+ENDPROC(relocate_vectors)
diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S
index 29cdad0..a33ad3e 100644
--- a/arch/arm/lib/crt0.S
+++ b/arch/arm/lib/crt0.S
@@ -104,6 +104,11 @@ clr_gd:
ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
b   relocate_code
 here:
+/*
+ * now relocate vectors
+ */
+
+   bl  relocate_vectors
 
 /* Set up final (full) environment */
 
diff --git a/arch/arm/lib/relocate.S b/arch/arm/lib/relocate.S
index 6ede41c..92f5314 100644
--- a/arch/arm/lib/relocate.S
+++ b/arch/arm/lib/relocate.S
@@ -11,6 +11,47 @@
 #include linux/linkage.h
 
 /*
+ * Default/weak exception vectors relocation routine
+ *
+ * This routine covers the standard ARM cases: normal (0x),
+ * high (0x) and VBAR. SoCs which do not comply with any of
+ * the standard cases must provide their own, strong, version.
+ */
+
+   .section.text.relocate_vectors,ax,%progbits
+   .weak   relocate_vectors
+
+ENTRY(relocate_vectors)
+
+#ifdef CONFIG_HAS_VBAR
+   /*
+* If the ARM processor has the security extensions,
+* use VBAR to relocate the exception vectors.
+*/
+   ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
+   mcr p15, 0, r0, c12, c0, 0  /* Set VBAR */
+#else
+   /*
+* Copy the relocated exception vectors to the
+* correct address
+* CP15 c1 V bit 

Re: [U-Boot] [PATCH v1 2/2] imx: fix exception vectors relocation in i.MX27

2014-11-11 Thread trem

On 11/11/14 17:46, Albert ARIBAUD wrote:

Commit 3ff46cc4 fixed exception vectors setting in
the general ARM case, by either copying the exception
and indirect vector tables to normal (0x) or
high (0x) vectors address, or setting VBAR to
U-Boot's base if applicable.

i.MX27 SoC is ARM926E-JS, thus has only normal and
high options, but does not provide RAM at 0x
and has only ROM at 0x; it is therefore not
possible to move or change its exception vectors.

Besides, i.MX27 ROM code does provide an indirect
vectors table but at a non-standard address and with
the reset and reserved vectors missing.

Turn the current vector relocation code into a weak
routine called after relocate_code from crt0, and add
strong version for i.MX27.

Signed-off-by: Albert ARIBAUDalbert.u.b...@aribaud.net
---

  arch/arm/cpu/arm926ejs/mx27/Makefile   |  4 ++
  arch/arm/cpu/arm926ejs/mx27/relocate.S | 49 
  arch/arm/lib/crt0.S|  5 +++
  arch/arm/lib/relocate.S| 69 --
  4 files changed, 99 insertions(+), 28 deletions(-)
  create mode 100644 arch/arm/cpu/arm926ejs/mx27/relocate.S

diff --git a/arch/arm/cpu/arm926ejs/mx27/Makefile 
b/arch/arm/cpu/arm926ejs/mx27/Makefile
index 4976bbb..0edf144 100644
--- a/arch/arm/cpu/arm926ejs/mx27/Makefile
+++ b/arch/arm/cpu/arm926ejs/mx27/Makefile
@@ -5,3 +5,7 @@
  # SPDX-License-Identifier:GPL-2.0+

  obj-y = generic.o reset.o timer.o
+
+ifndef CONFIG_SPL_BUILD
+obj-y  += relocate.o
+endif
diff --git a/arch/arm/cpu/arm926ejs/mx27/relocate.S 
b/arch/arm/cpu/arm926ejs/mx27/relocate.S
new file mode 100644
index 000..97003b3
--- /dev/null
+++ b/arch/arm/cpu/arm926ejs/mx27/relocate.S
@@ -0,0 +1,49 @@
+/*
+ *  relocate - i.MX27-specific vector relocation
+ *
+ *  Copyright (c) 2013  Albert ARIBAUDalbert.u.b...@aribaud.net
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#includeasm-offsets.h
+#includeconfig.h
+#includelinux/linkage.h
+
+/*
+ * The i.MX27 SoC is very specific with respect to exceptions: it
+ * does not provide RAM at the high vectors address (0x),
+ * thus only the low address (0x) is useable; but that is
+ * in ROM. Therefore, vectors cannot be changed at all.
+ *
+ * However, these ROM-based vectors actually just perform indirect
+ * calls through pointers located in RAM at SoC-specific addresses,
+ * as follows:
+ *
+ * Offset  Exception  Use by ROM code
+ * 0x  reset  indirect branch to [0x0014]
+ * 0x0004  undefined instruction  indirect branch to [0xfef0]
+ * 0x0008  software interrupt indirect branch to [0xfef4]
+ * 0x000c  prefetch abort indirect branch to [0xfef8]
+ * 0x0010  data abort indirect branch to [0xfefc]
+ * 0x0014  (reserved in ARMv5)vector to ROM reset: 0xc000
+ * 0x0018  IRQindirect branch to [0xff00]
+ * 0x001c  FIQindirect branch to [0xff04]
+ *
+ * In order to initialize exceptions on i.MX27, we must copy U-Boot's
+ * indirect (not exception!) vector table into 0xfef0..0xff04
+ * taking care not to copy vectors 0x00 (reset) and 0x14 (reserved).
+ */
+
+   .section.text.relocate_vectors,ax,%progbits
+
+ENTRY(relocate_vectors)
+
+   ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
+   ldr r1, =0xFEF0
+   ldmia   r0!, {r2-r8,r10} /* load all eight vectors */
+   stmia   r1!, {r3-r6,r8,r10} /* only write supported vectors */
+
+   bx  lr
+
+ENDPROC(relocate_vectors)
diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S
index 29cdad0..a33ad3e 100644
--- a/arch/arm/lib/crt0.S
+++ b/arch/arm/lib/crt0.S
@@ -104,6 +104,11 @@ clr_gd:
ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
b   relocate_code
  here:
+/*
+ * now relocate vectors
+ */
+
+   bl  relocate_vectors

  /* Set up final (full) environment */

diff --git a/arch/arm/lib/relocate.S b/arch/arm/lib/relocate.S
index 6ede41c..92f5314 100644
--- a/arch/arm/lib/relocate.S
+++ b/arch/arm/lib/relocate.S
@@ -11,6 +11,47 @@
  #includelinux/linkage.h

  /*
+ * Default/weak exception vectors relocation routine
+ *
+ * This routine covers the standard ARM cases: normal (0x),
+ * high (0x) and VBAR. SoCs which do not comply with any of
+ * the standard cases must provide their own, strong, version.
+ */
+
+   .section.text.relocate_vectors,ax,%progbits
+   .weak   relocate_vectors
+
+ENTRY(relocate_vectors)
+
+#ifdef CONFIG_HAS_VBAR
+   /*
+* If the ARM processor has the security extensions,
+* use VBAR to relocate the exception vectors.
+*/
+   ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
+   mcr p15, 0, r0, c12, c0, 0  /* Set VBAR */
+#else
+   /*
+* Copy the relocated exception vectors to the
+* 

Re: [U-Boot] [PATCH v1 2/2] imx: fix exception vectors relocation in i.MX27

2014-11-11 Thread Albert ARIBAUD
Hello trem,

On Tue, 11 Nov 2014 19:02:21 +0100, trem trem...@yahoo.fr wrote:
 On 11/11/14 17:46, Albert ARIBAUD wrote:
  Commit 3ff46cc4 fixed exception vectors setting in
  the general ARM case, by either copying the exception
  and indirect vector tables to normal (0x) or
  high (0x) vectors address, or setting VBAR to
  U-Boot's base if applicable.
 
  i.MX27 SoC is ARM926E-JS, thus has only normal and
  high options, but does not provide RAM at 0x
  and has only ROM at 0x; it is therefore not
  possible to move or change its exception vectors.
 
  Besides, i.MX27 ROM code does provide an indirect
  vectors table but at a non-standard address and with
  the reset and reserved vectors missing.
 
  Turn the current vector relocation code into a weak
  routine called after relocate_code from crt0, and add
  strong version for i.MX27.
 
  Signed-off-by: Albert ARIBAUDalbert.u.b...@aribaud.net
  ---
 
arch/arm/cpu/arm926ejs/mx27/Makefile   |  4 ++
arch/arm/cpu/arm926ejs/mx27/relocate.S | 49 
arch/arm/lib/crt0.S|  5 +++
arch/arm/lib/relocate.S| 69 
  --
4 files changed, 99 insertions(+), 28 deletions(-)
create mode 100644 arch/arm/cpu/arm926ejs/mx27/relocate.S
 
  diff --git a/arch/arm/cpu/arm926ejs/mx27/Makefile 
  b/arch/arm/cpu/arm926ejs/mx27/Makefile
  index 4976bbb..0edf144 100644
  --- a/arch/arm/cpu/arm926ejs/mx27/Makefile
  +++ b/arch/arm/cpu/arm926ejs/mx27/Makefile
  @@ -5,3 +5,7 @@
# SPDX-License-Identifier:GPL-2.0+
 
obj-y = generic.o reset.o timer.o
  +
  +ifndef CONFIG_SPL_BUILD
  +obj-y  += relocate.o
  +endif
  diff --git a/arch/arm/cpu/arm926ejs/mx27/relocate.S 
  b/arch/arm/cpu/arm926ejs/mx27/relocate.S
  new file mode 100644
  index 000..97003b3
  --- /dev/null
  +++ b/arch/arm/cpu/arm926ejs/mx27/relocate.S
  @@ -0,0 +1,49 @@
  +/*
  + *  relocate - i.MX27-specific vector relocation
  + *
  + *  Copyright (c) 2013  Albert ARIBAUDalbert.u.b...@aribaud.net
  + *
  + * SPDX-License-Identifier:GPL-2.0+
  + */
  +
  +#includeasm-offsets.h
  +#includeconfig.h
  +#includelinux/linkage.h
  +
  +/*
  + * The i.MX27 SoC is very specific with respect to exceptions: it
  + * does not provide RAM at the high vectors address (0x),
  + * thus only the low address (0x) is useable; but that is
  + * in ROM. Therefore, vectors cannot be changed at all.
  + *
  + * However, these ROM-based vectors actually just perform indirect
  + * calls through pointers located in RAM at SoC-specific addresses,
  + * as follows:
  + *
  + * Offset  Exception  Use by ROM code
  + * 0x  reset  indirect branch to [0x0014]
  + * 0x0004  undefined instruction  indirect branch to [0xfef0]
  + * 0x0008  software interrupt indirect branch to [0xfef4]
  + * 0x000c  prefetch abort indirect branch to [0xfef8]
  + * 0x0010  data abort indirect branch to [0xfefc]
  + * 0x0014  (reserved in ARMv5)vector to ROM reset: 0xc000
  + * 0x0018  IRQindirect branch to [0xff00]
  + * 0x001c  FIQindirect branch to [0xff04]
  + *
  + * In order to initialize exceptions on i.MX27, we must copy U-Boot's
  + * indirect (not exception!) vector table into 0xfef0..0xff04
  + * taking care not to copy vectors 0x00 (reset) and 0x14 (reserved).
  + */
  +
  +   .section.text.relocate_vectors,ax,%progbits
  +
  +ENTRY(relocate_vectors)
  +
  +   ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
  +   ldr r1, =0xFEF0
  +   ldmia   r0!, {r2-r8,r10} /* load all eight vectors */
  +   stmia   r1!, {r3-r6,r8,r10} /* only write supported vectors */
  +
  +   bx  lr
  +
  +ENDPROC(relocate_vectors)
  diff --git a/arch/arm/lib/crt0.S b/arch/arm/lib/crt0.S
  index 29cdad0..a33ad3e 100644
  --- a/arch/arm/lib/crt0.S
  +++ b/arch/arm/lib/crt0.S
  @@ -104,6 +104,11 @@ clr_gd:
  ldr r0, [r9, #GD_RELOCADDR] /* r0 = gd-relocaddr */
  b   relocate_code
here:
  +/*
  + * now relocate vectors
  + */
  +
  +   bl  relocate_vectors
 
/* Set up final (full) environment */
 
  diff --git a/arch/arm/lib/relocate.S b/arch/arm/lib/relocate.S
  index 6ede41c..92f5314 100644
  --- a/arch/arm/lib/relocate.S
  +++ b/arch/arm/lib/relocate.S
  @@ -11,6 +11,47 @@
#includelinux/linkage.h
 
/*
  + * Default/weak exception vectors relocation routine
  + *
  + * This routine covers the standard ARM cases: normal (0x),
  + * high (0x) and VBAR. SoCs which do not comply with any of
  + * the standard cases must provide their own, strong, version.
  + */
  +
  +   .section.text.relocate_vectors,ax,%progbits
  +   .weak   relocate_vectors
  +
  +ENTRY(relocate_vectors)
  +
  +#ifdef CONFIG_HAS_VBAR
  +   /*
  +* If the ARM