Re: [U-Boot] [PATCH] nios2: consolidate reset initialization

2010-04-22 Thread Scott McNutt
Applied. Thanks.
--Scott

Thomas Chou wrote:
> Global interrupt should be disabled from the beginning.
> 
> Signed-off-by: Thomas Chou 
> ---
>  arch/nios2/cpu/start.S |9 -
>  1 files changed, 4 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/nios2/cpu/start.S b/arch/nios2/cpu/start.S
> index 31cd5b0..d1016ea 100644
> --- a/arch/nios2/cpu/start.S
> +++ b/arch/nios2/cpu/start.S
> @@ -34,6 +34,7 @@
>   .global _start
>  
>  _start:
> + wrctl   status, r0  /* Disable interrupts */
>   /* ICACHE INIT -- only the icache line at the reset address
>* is invalidated at reset. So the init must stay within
>* the cache line size (8 words). If GERMS is used, we'll
> @@ -43,10 +44,9 @@ _start:
>   ori r4, r0, %lo(CONFIG_SYS_ICACHELINE_SIZE)
>   movhi   r5, %hi(CONFIG_SYS_ICACHE_SIZE)
>   ori r5, r5, %lo(CONFIG_SYS_ICACHE_SIZE)
> - mov r6, r0
> -0:   initi   r6
> - add r6, r6, r4
> - bltur6, r5, 0b
> +0:   initi   r5
> + sub r5, r5, r4
> + bgt r5, r0, 0b
>   br  _except_end /* Skip the tramp */
>  
>   /* EXCEPTION TRAMPOLINE -- the following gets copied
> @@ -62,7 +62,6 @@ _except_end:
>   /* INTERRUPTS -- for now, all interrupts masked and globally
>* disabled.
>*/
> - wrctl   status, r0  /* Disable interrupts */
>   wrctl   ienable, r0 /* All disabled */
>  
>   /* DCACHE INIT -- if dcache not implemented, initd behaves as
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] nios2: consolidate reset initialization

2010-04-19 Thread Thomas Chou
Global interrupt should be disabled from the beginning.

Signed-off-by: Thomas Chou 
---
 arch/nios2/cpu/start.S |9 -
 1 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/arch/nios2/cpu/start.S b/arch/nios2/cpu/start.S
index 31cd5b0..d1016ea 100644
--- a/arch/nios2/cpu/start.S
+++ b/arch/nios2/cpu/start.S
@@ -34,6 +34,7 @@
.global _start
 
 _start:
+   wrctl   status, r0  /* Disable interrupts */
/* ICACHE INIT -- only the icache line at the reset address
 * is invalidated at reset. So the init must stay within
 * the cache line size (8 words). If GERMS is used, we'll
@@ -43,10 +44,9 @@ _start:
ori r4, r0, %lo(CONFIG_SYS_ICACHELINE_SIZE)
movhi   r5, %hi(CONFIG_SYS_ICACHE_SIZE)
ori r5, r5, %lo(CONFIG_SYS_ICACHE_SIZE)
-   mov r6, r0
-0: initi   r6
-   add r6, r6, r4
-   bltur6, r5, 0b
+0: initi   r5
+   sub r5, r5, r4
+   bgt r5, r0, 0b
br  _except_end /* Skip the tramp */
 
/* EXCEPTION TRAMPOLINE -- the following gets copied
@@ -62,7 +62,6 @@ _except_end:
/* INTERRUPTS -- for now, all interrupts masked and globally
 * disabled.
 */
-   wrctl   status, r0  /* Disable interrupts */
wrctl   ienable, r0 /* All disabled */
 
/* DCACHE INIT -- if dcache not implemented, initd behaves as
-- 
1.6.6.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot