From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1753957AbdGCMfg (ORCPT <rfc822;w@1wt.eu>);
        Mon, 3 Jul 2017 08:35:36 -0400
Received: from mail-oi0-f67.google.com ([209.85.218.67]:34405 "EHLO
        mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1753588AbdGCMfd (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 3 Jul 2017 08:35:33 -0400
MIME-Version: 1.0
In-Reply-To: <20170701212930.606-1-afaerber@suse.de>
References: <CAK8P3a1vCKkZ8i=MLQXHX5ZowWQGbqiMo3gQz1E9RV+8sSGfpA@mail.gmail.com>
 <20170701212930.606-1-afaerber@suse.de>
From: Arnd Bergmann <arnd@arndb.de>
Date: Mon, 3 Jul 2017 14:35:31 +0200
X-Google-Sender-Auth: E6rQ9W8G59Jqk1ypIde0EJ6q3vY
Message-ID: <CAK8P3a3R9g2S4cPDhgYK2rPmLxjvs0ki1kg5vY_1zPZwcbttQQ@mail.gmail.com>
Subject: Re: [PATCH] ARM: owl: smp: Drop bogus holding pen
To: =?UTF-8?Q?Andreas_F=C3=A4rber?= <afaerber@suse.de>
Cc: Linux ARM <linux-arm-kernel@lists.infradead.org>,
        mp-cs@actions-semi.com, Thomas Liau <thomas.liau@actions-semi.com>,
        =?UTF-8?B?5byg5Lic6aOO?= <zhangdf@actions-semi.com>,
        =?UTF-8?B?5YiY54Kc?= <liuwei@actions-semi.com>,
        =?UTF-8?B?5byg5aSp55uK?= <tyzhang@actions-semi.com>,
        =?UTF-8?B?5qKF5Yip?= <harrymei@actions-semi.com>, support@lemaker.org,
        support@cubietech.com, lee@cubietech.com,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Russell King <linux@armlinux.org.uk>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v63Cdf3w029373

On Sat, Jul 1, 2017 at 11:29 PM, Andreas Färber <afaerber@suse.de> 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@arndb.de>
> Cc: David Liu <liuwei@actions-semi.com>
> Signed-off-by: Andreas Färber <afaerber@suse.de>

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
well.

Thanks!

        Arnd

From mboxrd@z Thu Jan  1 00:00:00 1970
From: arnd@arndb.de (Arnd Bergmann)
Date: Mon, 3 Jul 2017 14:35:31 +0200
Subject: [PATCH] ARM: owl: smp: Drop bogus holding pen
In-Reply-To: <20170701212930.606-1-afaerber@suse.de>
References: <CAK8P3a1vCKkZ8i=MLQXHX5ZowWQGbqiMo3gQz1E9RV+8sSGfpA@mail.gmail.com>
 <20170701212930.606-1-afaerber@suse.de>
Message-ID: <CAK8P3a3R9g2S4cPDhgYK2rPmLxjvs0ki1kg5vY_1zPZwcbttQQ@mail.gmail.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

On Sat, Jul 1, 2017 at 11:29 PM, Andreas F?rber <afaerber@suse.de> 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@arndb.de>
> Cc: David Liu <liuwei@actions-semi.com>
> Signed-off-by: Andreas F?rber <afaerber@suse.de>

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
well.

Thanks!

        Arnd