[PATCH] arm: mm: Fix infinite looping issue

Girish KS girishks2000 at gmail.com
Sat Oct 8 01:24:53 EDT 2011


On Tue, Oct 4, 2011 at 4:08 PM, Girish K S
<girish.shivananjappa at linaro.org> wrote:
> This patch fixes the problem of infinite looping while booting.
>
> The bne instruction expects the z flag to be set to break the loop.
> The (mov r1, r1, lsr #1) doesn't behave in the expected way. If
> replaced with the movs instruction then the execution will exit
> the loop.
>
> Signed-off-by: Girish K S <girish.shivananjappa at linaro.org>
> ---
>  arch/arm/mm/proc-arm740.S |    4 ++--
>  arch/arm/mm/proc-arm940.S |    4 ++--
>  arch/arm/mm/proc-arm946.S |    4 ++--
>  3 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/mm/proc-arm740.S b/arch/arm/mm/proc-arm740.S
> index 4506be3..9284f10 100644
> --- a/arch/arm/mm/proc-arm740.S
> +++ b/arch/arm/mm/proc-arm740.S
> @@ -77,7 +77,7 @@ __arm740_setup:
>        ldr     r1, =(CONFIG_DRAM_SIZE >> 12)   @ size of RAM (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the area register value
>        orr     r0, r0, #1                      @ set enable bit
> @@ -87,7 +87,7 @@ __arm740_setup:
>        ldr     r1, =(CONFIG_FLASH_SIZE >> 12)  @ size of FLASH (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the area register value
>        orr     r0, r0, #1                      @ set enable bit
> diff --git a/arch/arm/mm/proc-arm940.S b/arch/arm/mm/proc-arm940.S
> index ac750d5..e639034 100644
> --- a/arch/arm/mm/proc-arm940.S
> +++ b/arch/arm/mm/proc-arm940.S
> @@ -296,7 +296,7 @@ __arm940_setup:
>        ldr     r1, =(CONFIG_DRAM_SIZE >> 12)   @ size of RAM (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the area register value
>        orr     r0, r0, #1                      @ set enable bit
> @@ -307,7 +307,7 @@ __arm940_setup:
>        ldr     r1, =(CONFIG_FLASH_SIZE >> 12)  @ size of FLASH (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the area register value
>        orr     r0, r0, #1                      @ set enable bit
> diff --git a/arch/arm/mm/proc-arm946.S b/arch/arm/mm/proc-arm946.S
> index 683af3a..731388a 100644
> --- a/arch/arm/mm/proc-arm946.S
> +++ b/arch/arm/mm/proc-arm946.S
> @@ -341,7 +341,7 @@ __arm946_setup:
>        ldr     r1, =(CONFIG_DRAM_SIZE >> 12)   @ size of RAM (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the region register value
>        orr     r0, r0, #1                      @ set enable bit
> @@ -351,7 +351,7 @@ __arm946_setup:
>        ldr     r1, =(CONFIG_FLASH_SIZE >> 12)  @ size of FLASH (must be >= 4KB)
>        mov     r2, #10                         @ 11 is the minimum (4KB)
>  1:     add     r2, r2, #1                      @ area size *= 2
> -       mov     r1, r1, lsr #1
> +       movs    r1, r1, lsr #1                  @ set zero bit if r1=0
>        bne     1b                              @ count not zero r-shift
>        orr     r0, r0, r2, lsl #1              @ the region register value
>        orr     r0, r0, #1                      @ set enable bit
> --
> 1.7.1
>
>Hello Mr choi,
 Sorry for my previous top posting.
If there is no issue with this patch can you apply to your for-next tree
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>



More information about the linux-arm-kernel mailing list