[PATCH 52/74] ST SPEAr13xx: Adding CPU hotplug support added for SMP platforms
Sundar
sunder.svit at gmail.com
Fri Sep 3 02:00:12 EDT 2010
Hi,
On Mon, Aug 30, 2010 at 4:09 PM, Viresh KUMAR <viresh.kumar at st.com> wrote:
> From: Deepak Sikri <deepak.sikri at st.com>
>
> Signed-off-by: Deepak Sikri <deepak.sikri at st.com>
> Signed-off-by: shiraz hashim <shiraz.hashim at st.com>
> Signed-off-by: Viresh Kumar <viresh.kumar at st.com>
> ---
> + flush_cache_all();
> + asm volatile(
> + " mcr p15, 0, %1, c7, c5, 0\n"
> + " mcr p15, 0, %1, c7, c10, 4\n"
Isnt this a Cortex-A9 platform? If yes, then this is not implemented
on the Cortex-A9 if I am right.
You should use a dsb() here.
> + /*
> + * here's the WFI
> + */
> + asm(".word 0xe320f003\n"
> + :
Also, why not a simple asm volatile("wfi" : : : "memory"); instead of
hard coded values.
Regards,
Sundar
More information about the linux-arm-kernel
mailing list