Re: [PATCH] ARM: owl: smp: Drop bogus holding pen

From: Arnd Bergmann
Date: Mon Jul 03 2017 - 08:35:43 EST

On Sat, Jul 1, 2017 at 11:29 PM, Andreas FÃrber <afaerber@xxxxxxx> wrote:
> The S500 SoC can start secondary CPUs without busy-looping for pen_release,
> so simplify the SMP code compared to the LeMaker kernel tree.
> Fixes: 172067e0bc87 ("ARM: owl: Implement CPU enable-method for S500")
> Suggested-by: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: David Liu <liuwei@xxxxxxxxxxxxxxxx>
> Signed-off-by: Andreas FÃrber <afaerber@xxxxxxx>

Looks good to me. I applied both on top of next/soc before tagging the
branch for 4.13,
so this version is going to be what we merge.

I saw that David had another suggestion for simplifying it further.
That also looks
good, but it's less urgent, so please implement that as an add-on patch and send
it for 4.14, unless you decide there is a reason it needs to be
included in 4.13 as