Wrong cache flush function used in arch/arm/boot/compressed/head.S?

Catalin Marinas catalin.marinas at arm.com
Fri Mar 12 10:51:56 EST 2010


On Fri, 2010-03-12 at 10:05 +0000, Sascha Hauer wrote:
> 0e056f20f18d0efa5da920f3cf8532adc56d5779 contains the following hunk:
> 
>                 .word   0x00060000              @ ARMv5TEJ
>                 .word   0x000f0000
> -               b       __armv4_mmu_cache_on
> -               b       __armv4_mmu_cache_off
> -               b       __armv5tej_mmu_cache_flush
> +               W(b)    __armv4_mmu_cache_on
> +               W(b)    __armv4_mmu_cache_off
> +               W(b)    __armv4_mmu_cache_flush
> 
> Is this intentional? I do not see any problems in my kernel, but I
> use the same code in my bootloader and it doesn't seem to work correctly
> on my arm926 based system.

It wasn't intentional. I think I got it wrong when trying to solve a
merge conflict and I just added the original hunk.

Thanks.

-- 
Catalin




More information about the linux-arm-kernel mailing list