Re: [PATCH v2] ARM: vexpress: Use wfi macro in platform_do_lowpower.

From: Jamie Iles
Date: Mon Aug 22 2011 - 10:00:02 EST


On Mon, Aug 22, 2011 at 09:54:19AM -0400, Nick Bowler wrote:
> Current Versatile Express CPU hotplug code includes a hardcoded WFI
> instruction, in ARM encoding. When the kernel is compiled in Thumb-2
> mode, this is invalid and causes the machine to hang hard when a CPU
> is offlined.
>
> Using the wfi macro (which uses the appropriate assembler mnemonic)
> causes the correct instruction to be emitted in either case. As a
> consequence of this change, an apparently vestigial "cc" clobber is
> dropped from the asm (the macro uses "memory" only).
>
> Signed-off-by: Nick Bowler <nbowler@xxxxxxxxxxxxxxxx>
> ---

Reviewed-by: Jamie Iles <jamie@xxxxxxxxxxxxx>

if it helps!

Jamie
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/